Skip to content

fix(carousel): change icon to chevron - #4681

Merged
FelixBlaisThon merged 1 commit into
masterfrom
ADUI-11651-change-carousel-icon
Oct 30, 2026
Merged

FelixBlaisThon merged 1 commit into
masterfrom
ADUI-11651-change-carousel-icon

Conversation

@FelixBlaisThon

@FelixBlaisThon FelixBlaisThon commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Proposed Changes

Mistake I made when adding the Carousel component. I put the Arrow icon instead of Chevron icon.

That's it

Before

image

After

image

Potential Breaking Changes

Acceptance Criteria

  • The proposed changes are covered by unit tests
  • The potential breaking changes are clearly identified
  • A changeset is added for releasable changes, following CONTRIBUTING.md
  • README.md is adjusted to reflect the proposed changes (if relevant)

Copilot AI balanced review requested due to automatic review settings October 30, 2026 20:47
@FelixBlaisThon
FelixBlaisThon requested a review from a team as a code owner October 30, 2026 20:47
@FelixBlaisThon
FelixBlaisThon requested review from GermainBergeron and gdostie and removed request for a team October 30, 2026 20:47
@svcsnykcoveo

svcsnykcoveo commented Oct 30, 2026 •

Copy link
Copy Markdown

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues
✅ Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@github-actions

Copy link
Copy Markdown
Contributor

Release Preview

This PR would bump 1 package version if it were merged and released from master.

Patch

Package Next
@coveord/plasma-mantine 61.6.1

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

There are no active review issues for the Carousel default-navigation icon update.

No open issues 🎉

What changed in this PR

Corrects Plasma’s Carousel default navigation controls to use chevrons instead of arrows.

Changes:

  • Replaces default icons for both horizontal and vertical orientations while preserving custom icon support.
  • Adds a patch changeset for @coveord/plasma-mantine.
File Description
packages/​mantine/​src/​components/​Carousel/​Carousel.tsx Replaces the four directional arrow icons with chevrons.
.changeset/​proud-donkeys-wink.md Documents the icon correction for a patch release.

Review effort: Balanced

@github-actions

Copy link
Copy Markdown
Contributor

Live demo of your latest successful build available here:

@FelixBlaisThon
FelixBlaisThon merged commit 1b5ffbc into master Oct 30, 2026
19 checks passed
@FelixBlaisThon
FelixBlaisThon deleted the ADUI-11651-change-carousel-icon branch October 30, 2026 20:56
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.

4 participants