fix(ui): make toolbar mobile friendly#3650
Conversation
Co-authored-by: Gabriel Miranda <gabrielmfern@outlook.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🦋 Changeset detectedLatest commit: 9a80959 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
commit: |
There was a problem hiding this comment.
No issues found across 4 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Auto-approved: CSS-only mobile responsiveness for toolbar tabs and actions; no logic changes, low risk.
Re-trigger cubic
Co-authored-by: Gabriel Miranda <gabrielmfern@outlook.com>
There was a problem hiding this comment.
0 issues found across 3 files (changes from recent commits).
Auto-approved: CSS-only mobile responsiveness changes: horizontal scroll for tabs, no-wrap on labels, viewport-constrained menu. No logic or data changes.
Re-trigger cubic
Co-authored-by: Gabriel Miranda <gabrielmfern@outlook.com>
Co-authored-by: Gabriel Miranda <gabrielmfern@outlook.com>
Co-authored-by: Gabriel Miranda <gabrielmfern@outlook.com>
There was a problem hiding this comment.
0 issues found across 2 files (changes from recent commits).
Auto-approved: UI responsiveness improvements: horizontal scrolling tabs, popover for toolbar info, and text wrapping fixes. Isolated to UI package with no impact on core logic.
Re-trigger cubic
Co-authored-by: Gabriel Miranda <gabrielmfern@outlook.com>
Co-authored-by: Gabriel Miranda <gabrielmfern@outlook.com>
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Auto-approved: Responsive layout and popover additions for toolbar, with tested CSS and logic changes limited to UI components.
Re-trigger cubic
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
Dismissed because Cubic found issues in a newer review.
Co-authored-by: Gabriel Miranda <gabrielmfern@outlook.com>
There was a problem hiding this comment.
All reported issues were addressed across 2 files (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
Co-authored-by: Gabriel Miranda <gabrielmfern@outlook.com>
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
Co-authored-by: Gabriel Miranda <gabrielmfern@outlook.com>
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
Co-authored-by: Gabriel Miranda <gabrielmfern@outlook.com>
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
Co-authored-by: Gabriel Miranda <gabrielmfern@outlook.com>
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
Co-authored-by: Gabriel Miranda <gabrielmfern@outlook.com>
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Auto-approved: Adds responsive toolbar with mobile dropdown, updates info popover and Copy for AI menu constraints. Low-risk UI changes with tests passing.
Re-trigger cubic

Summary
onOpenChange/onClick/onPointerDownhandlers (no extracted top-level handler functions); the pointer-type tracking removes both itspointerup/pointercancellisteners on either terminal event so repeated interactions don't accumulate stale window listeners@react-email/uiTesting
pnpm exec biome checkon changed filespnpm --filter @react-email/ui test(105 tests passed)bluraddEventListener/removeEventListener) that 5 real clicks and 5 simulated interrupted presses in a row leave no straypointerup/pointercancellisteners onwindowMobile dropdown panel switcher open
Resend panel selected from the dropdown
To show artifacts inline, enable in settings.
Slack Thread
Summary by cubic
Make the preview toolbar and
Copy for AImenu responsive. On small screens, tabs collapse into a dropdown that shows the current panel; the info button uses hover on desktop and a persistent popover on touch/keyboard.Bug Fixes
Copy for AIon one line; keep the active-tab underline aligned.Copy for AIand info popover to the viewport with max size, scrolling, and collision padding.pointerdownfor Safari, registers a blur listener on open, closes when the preview iframe gains focus, and cleans up blur and pointer listeners on close and unmount.Dependencies
@react-email/ui.Written for commit 9a80959. Summary will update on new commits.