Skip to content

feat: add pageSize prop to carousels#1165

Merged
jorgemoya merged 1 commit intorefactor/ui-componentsfrom
add-pagesize-caroulse
Jul 26, 2024
Merged

feat: add pageSize prop to carousels#1165
jorgemoya merged 1 commit intorefactor/ui-componentsfrom
add-pagesize-caroulse

Conversation

@jorgemoya
Copy link
Copy Markdown
Contributor

What/Why?

Add pageSize prop to show less products per page. 4 is default.

Testing

Locally.

@jorgemoya jorgemoya requested a review from a team July 26, 2024 18:44
@vercel
Copy link
Copy Markdown

vercel Bot commented Jul 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
catalyst-latest ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 26, 2024 6:46pm
6 Skipped Deployments
Name Status Preview Comments Updated (UTC)
catalyst ⬜️ Ignored (Inspect) Jul 26, 2024 6:46pm
catalyst-1millionproducts-store ⬜️ Ignored (Inspect) Jul 26, 2024 6:46pm
catalyst-au ⬜️ Ignored (Inspect) Jul 26, 2024 6:46pm
catalyst-test-store ⬜️ Ignored (Inspect) Jul 26, 2024 6:46pm
catalyst-uk ⬜️ Ignored (Inspect) Jul 26, 2024 6:46pm
catalyst-unstable ⬜️ Ignored (Inspect) Jul 26, 2024 6:46pm

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jul 26, 2024

⚠️ No Changeset found

Latest commit: b1422a4

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Copy Markdown
Contributor

⚡️🏠 Lighthouse report

Lighthouse ran against https://catalyst-latest-c947fevy8-bigcommerce-platform.vercel.app

🖥️ Desktop

We ran Lighthouse against the changes on a desktop and produced this report. Here's the summary:

Category Score
🟢 Performance 99
🟢 Accessibility 95
🟢 Best practices 96
🟠 SEO 82

📱 Mobile

We ran Lighthouse against the changes on a mobile and produced this report. Here's the summary:

Category Score
🟢 Performance 95
🟢 Accessibility 95
🟢 Best practices 96
🟠 SEO 85

@jorgemoya jorgemoya merged this pull request into refactor/ui-components Jul 26, 2024
@jorgemoya jorgemoya deleted the add-pagesize-caroulse branch July 26, 2024 19:56
jorgemoya added a commit that referenced this pull request Jul 30, 2024
github-merge-queue Bot pushed a commit that referenced this pull request Jul 30, 2024
* refactor: tag + usages (#1107)

* refactor(core): radio-group (#1106)

* fix: merge with main (#1105)

* refactor: remove file-chooser, no usages (#1110)

* refactor: carousel (#1109)

* refactor: badge (#1108)

* fix: eslint issues in tabs and badge

* refactor: gallery refactor (#1104)

* fix(core): lint issues from gallery refactor

* refactor(core): slideshow (#1111)

* refactor: message component (#1112)

* refactor: badge takes children (#1114)

* chore(core): delete skeleton (#1113)

* refactor: modal (#1115)

* refactor: button (#1116)

* fix: render cart count in badge

* refactor: datepicker uses popover primitives (#1118)

* refactor: accordions component (#1101)

* refactor(core): product card (#1117)

* refactor: input (#1131)

* refactor: textarea (#1134)

* refactor: checkbox (#1135)

* refactor: date picker (#1130)

* fix: radio group error state in pdp (#1133)

* refactor: pick list (#1139)

* fix: forward ref to button (#1140)

* refactor: select (#1143)

* fix: use correct font weight in badge (#1151)

* refactor: swatch (#1144)

* refactor: rectangle list (#1145)

* refactor: pick list props (#1146)

* refactor: radio group props (#1147)

* refactor: breadcrumbs (#1148)

* refactor: footer (#1154)

* fix: forward DatePicker ref (#1155)

* refactor: label (#1153)

* refactor: blog post card (#1152)

* refactor: rating (#1150)

* refactor: sheet (#1149)

* refactor: counter (#1136)

* refactor: header & navigation menu (#1160)

* fix: accordions props (#1162)

* fix: remove transparent overlay prop in Sheet (#1163)

* fix: gallery props (#1164)

* feat: add pageSize prop to carousels (#1165)

* chore: add changeset

* chore: update changeset

* fix: missing padding for sm viewport in quick search

* fix: close mobile nav on navigation

* fix: add aria-label to account tabs

* fix: skip blog post card regression

* fix: p wrapper for cart link

* fix: update header visual regression test

* fix: correct capitlization for tabs

* fix: change hover for on click in test

* fix: swatch test

* fix: changeset examples

* feat: add label prop to tabs

* fix: update account and register spec

* fix: update tests with latest changes

---------

Co-authored-by: Matthew Volk <matt.volk@bigcommerce.com>
Co-authored-by: Drew Pledger <drew@makeswift.com>
Co-authored-by: Daniel Almaguer <daniel.almaguer@bigcommerce.com>
Co-authored-by: Matthew Volk <volkmattj@gmail.com>
Co-authored-by: Anudeep Vattipalli <anudeep.vattipalli@bigcommerce.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants