Skip to content

[WC-3047] Gallery: Improve preview with limited rows and fade out - #1833

Merged
yordan-st merged 3 commits into
mainfrom
fix/WC-2924_gallery-preview-columns
Sep 9, 2025
Merged

[WC-3047] Gallery: Improve preview with limited rows and fade out#1833
yordan-st merged 3 commits into
mainfrom
fix/WC-2924_gallery-preview-columns

Conversation

@yordan-st

Copy link
Copy Markdown
Contributor

Pull request type

Bug fix (non-breaking change which fixes an issue)


Description

Fixed an issue where the displayed number of columns in preview mode wouldn't match the configuration
Added functionality where the row count doesn't exceed 3 - in cases where it should because of the column count x page size
Added functionality where the rows of the widget slowly fade out in a vertical gradient

What should be covered while testing?

If the rows in no circumstance exceed the count of 3.
Therefore configure a page size and column count which would normally cause the row count to exceed 3.
Test this on al viewports
Test if there is a fade-away gradient overlaying the grid items

@yordan-st
yordan-st requested a review from a team as a code owner August 14, 2025 14:20
Comment thread packages/pluggableWidgets/gallery-web/CHANGELOG.md Outdated
Comment thread packages/pluggableWidgets/gallery-web/CHANGELOG.md Outdated
Comment thread packages/pluggableWidgets/gallery-web/CHANGELOG.md Outdated
Comment thread packages/pluggableWidgets/gallery-web/src/features/useGridPositionsPreview.ts Outdated
Comment thread packages/pluggableWidgets/gallery-web/src/features/useGridPositionsPreview.ts Outdated
@yordan-st
yordan-st force-pushed the fix/WC-2924_gallery-preview-columns branch 2 times, most recently from c863e67 to 78dd7b2 Compare September 1, 2025 10:11
r0b1n
r0b1n previously approved these changes Sep 1, 2025
leonardomendix
leonardomendix previously approved these changes Sep 4, 2025
@yordan-st
yordan-st dismissed stale reviews from leonardomendix and r0b1n via 7f303c4 September 4, 2025 14:48
gjulivan
gjulivan previously approved these changes Sep 4, 2025
leonardomendix
leonardomendix previously approved these changes Sep 4, 2025
@gjulivan
gjulivan dismissed stale reviews from leonardomendix and themself via c35dc33 September 6, 2025 20:09
@gjulivan
gjulivan force-pushed the fix/WC-2924_gallery-preview-columns branch from 7f303c4 to c35dc33 Compare September 6, 2025 20:09
@gjulivan
gjulivan force-pushed the fix/WC-2924_gallery-preview-columns branch from c35dc33 to 66d4843 Compare September 7, 2025 14:39
@yordan-st
yordan-st merged commit 98f9a1b into main Sep 9, 2025
14 checks passed
@yordan-st
yordan-st deleted the fix/WC-2924_gallery-preview-columns branch September 9, 2025 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants