fix(ui): tuck notification stack below modal and dropdown menus#5779
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Toasts stay ambient: above page content and Reviewed by Cursor Bugbot for commit fd8b8ab. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit fd8b8ab. Configure here.
Greptile SummaryThis PR changes the shared notification layer ordering in the Sim UI. The main changes are:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (1): Last reviewed commit: "fix(ui): tuck notification stack below m..." | Re-trigger Greptile |

Summary
--z-toast(500) sat above--z-modal(200) and--z-popover(300) in the shared z-index scale--z-toast: 150so it's ambient: above page content and editor drawers (--z-dropdown: 100), but below the modal and every transient popper (menus, selects, popovers, tooltips)globals.css— all toasts, modals, and dropdown menus consume these tokens as the single source of truth, so no component code changesType of Change
Testing
Traced every
--z-toast,--z-modal, and--z-popoverconsumer: the toast stack and modal are both portaled todocument.body(shared stacking context, so pure z compares), and the workspace switcher usesDropdownMenu(--z-popover, 300). Verified the only other--z-toastconsumers — the landing skip-link (still above thez-50navbar) and an off-screen drag ghost — are unaffected at 150.Checklist