Skip to content

fix(toolbar): use matching rotate icons#70

Merged
DjDeveloperr merged 1 commit into
mainfrom
update-rotate-toolbar-icons
May 31, 2026
Merged

fix(toolbar): use matching rotate icons#70
DjDeveloperr merged 1 commit into
mainfrom
update-rotate-toolbar-icons

Conversation

@DjDeveloperr
Copy link
Copy Markdown
Collaborator

Summary

  • Replace the rotate toolbar glyphs with Radix’s native rotate icon style so they match the rest of the toolbar.
  • Add a Rotate Left action alongside Rotate Right, with the left button hidden below the wider toolbar breakpoint.
  • Keep the shared rotation control path in AppShell so both buttons use the existing simulator rotation behavior.

Testing

  • UI tested in the in-app browser at http://127.0.0.1:5173/.
  • Verified both rotate buttons render at wide widths, the left button hides at narrower widths, and clicking each button updates then restores the simulator rotation state.
  • No fresh browser console errors were emitted during the final interaction pass.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 31, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7aaf9872-0f68-4fb1-b98d-94b30fa52cfc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update-rotate-toolbar-icons

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@DjDeveloperr DjDeveloperr merged commit 876e6ec into main May 31, 2026
16 of 17 checks passed
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.

1 participant