Summary: | Web Inspector: DOM Tree debugger popover obscures dashboard | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jonathan Wells <jonowells> | ||||||
Component: | Web Inspector | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, graouts, joepeck, timothy, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Jonathan Wells
2014-04-25 14:34:03 PDT
This is a z-index issue. I think this selector in DashboardContainerView.css isn't needed: .toolbar .dashboard.visible { z-index: 100; } Created attachment 230207 [details]
[PATCH] Proposed Fix: Change to DashboardContainerView.css
I believe this z-index property to be unnecessary.
Comment on attachment 230207 [details] [PATCH] Proposed Fix: Change to DashboardContainerView.css Do we know why this z-index was added originally? I think Brian Burg added it with: <https://webkit.org/b/129898> Web Inspector: convert the dashboard toolbar item to support multiple dashboards Created attachment 230212 [details]
[PATCH] Proposed Fix: Change to DashboardContainerView.css, Popover.css
Comment on attachment 230212 [details] [PATCH] Proposed Fix: Change to DashboardContainerView.css, Popover.css Clearing flags on attachment: 230212 Committed r167833: <http://trac.webkit.org/changeset/167833> All reviewed patches have been landed. Closing bug. Was there a screenshot of the original issue? Sounds to me like a popover should be laid out over the dashboard. |