fix(ui): Increase dropdown z-index to appear above sidebar#116139
Conversation
- Set theme.zIndex.dropdown to 1020 (sidebarPanel + 1) - Fixes widget context menu appearing behind dashboard navigation - Reordered dropdown after sidebarPanel to maintain sorted zIndex list Co-authored-by: James Keane <james.keane.github@gmail.com>
github-actions
Bot
added
the
Scope: Frontend
📊 Type Coverage Diff✅ No new type safety issues introduced. Coverage: 93.58% |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes 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 3b8bcf8. Configure here.
| widgetBuilderDrawer: 1016, | ||
|
|
||
| sidebarPanel: 1019, | ||
| dropdown: 1020, |
There was a problem hiding this comment.
Dropdown z-index equals sidebar causing potential overlap
Medium Severity
dropdown is now set to 1020, which is identical to sidebar (also 1020). Since dropdown menus are not portaled by default (usePortal = false), they render inline within the content area, which typically precedes the navigation in DOM order. When two positioned elements share the same z-index, the one later in the DOM wins. This means the mobile navigation bar (which uses theme.zIndex.sidebar) can still appear above non-portaled dropdowns depending on DOM structure—the same class of bug this PR aims to fix.
Reviewed by Cursor Bugbot for commit 3b8bcf8. Configure here.
DominikB2014
left a comment
DominikB2014
left a comment
There was a problem hiding this comment.
I think the cursor comment is probably valid, but the changes should still address the issue that we're trying to fix
3 participants
Fixes DAIN-1690
Problem
Widget context menus (triggered by clicking the '...' buttons on dashboard widgets) were appearing behind the dashboard navigation sidebar because
theme.zIndex.dropdown(1001) was lower thantheme.zIndex.sidebarPanel(1019).Before
After
Solution
Updated
theme.zIndex.dropdownfrom 1001 to 1020 (sidebarPanel + 1) in the theme definition. This ensures all dropdown menus across the application appear above the sidebar panel.Linear Issue: DAIN-1690