Skip to content

Add navigation arrows to product carousel.#15025

Merged
oliverabrahams merged 22 commits intomainfrom
oa/navigation-arrows
Jan 20, 2026
Merged

Add navigation arrows to product carousel.#15025
oliverabrahams merged 22 commits intomainfrom
oa/navigation-arrows

Conversation

@oliverabrahams
Copy link
Copy Markdown
Contributor

@oliverabrahams oliverabrahams commented Dec 17, 2025

What does this change?

Adds a Heading and navigation arrows to the product carousel.
Adds an interaction story to to the carousel storybook.

builds off this PR #14968

Screenshots

Screenshot 2025-12-17 at 12 25 21 Screenshot 2025-12-17 at 12 29 07

@oliverabrahams oliverabrahams marked this pull request as ready for review December 17, 2025 14:23
@github-actions
Copy link
Copy Markdown

Hello 👋! When you're ready to run Chromatic, please apply the run_chromatic label to this PR.

You will need to reapply the label each time you want to run Chromatic.

Click here to see the Chromatic project.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 17, 2025

… subgrid, and link to the product text heading using the H2. The product element heading h2Id is added using the enhancers in dcr
…f the card using Visable slides on either mobile or tablet. Remove the css in the carousel that is front specific and put that behind the !isArticle boolean
…l carousel. There will be a future PR for carousels with 2 cards + stories
@oliverabrahams oliverabrahams force-pushed the mob/scrollable-products branch from f20853d to a064393 Compare December 19, 2025 17:09
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 22, 2025

Comment thread dotcom-rendering/src/components/ScrollableProducts.stories.tsx Outdated
@charleycampbell
Copy link
Copy Markdown
Contributor

Looks good! Have we tested this for accessibility e.g. keyboard navigation etc?

@oliverabrahams oliverabrahams added feature Departmental tracking: work on a new feature run_chromatic Runs chromatic when label is applied labels Jan 15, 2026
@github-actions github-actions Bot removed the run_chromatic Runs chromatic when label is applied label Jan 15, 2026
@oliverabrahams oliverabrahams added the run_chromatic Runs chromatic when label is applied label Jan 15, 2026
@github-actions github-actions Bot removed the run_chromatic Runs chromatic when label is applied label Jan 15, 2026
@oliverabrahams
Copy link
Copy Markdown
Contributor Author

Looks good! Have we tested this for accessibility e.g. keyboard navigation etc?

I have tried keyboard navigation and it all seems to work 👍

Base automatically changed from mob/scrollable-products to main January 19, 2026 14:59
# Conflicts:
#	dotcom-rendering/src/components/ScrollableProduct.importable.tsx
#	dotcom-rendering/src/components/ScrollableProducts.stories.tsx
@oliverabrahams oliverabrahams added the run_chromatic Runs chromatic when label is applied label Jan 19, 2026
@github-actions github-actions Bot removed the run_chromatic Runs chromatic when label is applied label Jan 19, 2026
@oliverabrahams oliverabrahams merged commit 2998946 into main Jan 20, 2026
26 checks passed
@oliverabrahams oliverabrahams deleted the oa/navigation-arrows branch January 20, 2026 10:20
@gu-prout
Copy link
Copy Markdown

gu-prout Bot commented Jan 20, 2026

Seen on PROD (merged by @oliverabrahams 7 minutes and 57 seconds ago) Please check your changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Departmental tracking: work on a new feature Seen-on-PROD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants