Skip to content

feat(Icon): added new move-left and move-right icons#3102

Merged
codecademydev merged 1 commit intomainfrom
move-left-right-icons
Jun 12, 2025
Merged

feat(Icon): added new move-left and move-right icons#3102
codecademydev merged 1 commit intomainfrom
move-left-right-icons

Conversation

@NarsimhaRaj
Copy link
Copy Markdown
Contributor

Overview

icons added for menu move left and right

Screenshot 2025-06-06 at 10 14 43 AM Screenshot 2025-06-06 at 10 14 32 AM

PR Checklist

Testing Instructions

Go to storybook Icon ?path=/docs/atoms-icons-regular--docs
Search for MoveLeftIcon, MoveRightIcon
Screenshot 2025-06-02 at 3 23 07 PM

Don't make me tap the sign.

  1. Go to story X
  2. Do something
  3. Do that something in dark mode
  4. Check it with VO
  5. Finish and do a celebratory dance

PR Links and Envs

Repository PR Link
Monolith Monolith PR
Mono Mono PR

@NarsimhaRaj NarsimhaRaj requested a review from a team as a code owner June 6, 2025 04:50
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jun 6, 2025

CLA assistant check
All committers have signed the CLA.

@nx-cloud
Copy link
Copy Markdown

nx-cloud bot commented Jun 6, 2025

View your CI Pipeline Execution ↗ for commit e8fd0f7.

Command Status Duration Result
nx run-many --target=verify --parallel=3 --all ✅ Succeeded 2s View ↗
nx run-many --target=build --all ✅ Succeeded 2s View ↗

☁️ Nx Cloud last updated this comment at 2025-06-06 04:51:39 UTC

Copy link
Copy Markdown
Contributor

@dreamwasp dreamwasp left a comment

Choose a reason for hiding this comment

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

tested locally, lgtm

@NarsimhaRaj NarsimhaRaj added the Ship It :shipit: Ready 2 ship label Jun 12, 2025
@codecademydev codecademydev merged commit 50ef4e7 into main Jun 12, 2025
23 of 25 checks passed
@codecademydev codecademydev removed the Ship It :shipit: Ready 2 ship label Jun 12, 2025
@codecademydev codecademydev deleted the move-left-right-icons branch June 12, 2025 15:00
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