RESOLVED FIXED 132205
Web Inspector: DOM Tree debugger popover obscures dashboard 
https://bugs.webkit.org/show_bug.cgi?id=132205
Summary Web Inspector: DOM Tree debugger popover obscures dashboard
Jonathan Wells
Reported 2014-04-25 14:34:03 PDT
<rdar://problem/16622125>
Attachments
[PATCH] Proposed Fix: Change to DashboardContainerView.css (1.27 KB, patch)
2014-04-25 14:43 PDT, Jonathan Wells 		no flags
DetailsFormatted DiffDiff
[PATCH] Proposed Fix: Change to DashboardContainerView.css, Popover.css (1.71 KB, patch)
2014-04-25 15:17 PDT, Jonathan Wells 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Jonathan Wells
Comment 1 2014-04-25 14:35:16 PDT
This is a z-index issue. I think this selector in DashboardContainerView.css isn't needed: .toolbar .dashboard.visible { z-index: 100; }
Jonathan Wells
Comment 2 2014-04-25 14:43:35 PDT
Created attachment 230207 [details] [PATCH] Proposed Fix: Change to DashboardContainerView.css I believe this z-index property to be unnecessary.
Joseph Pecoraro
Comment 3 2014-04-25 14:47:46 PDT
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
Jonathan Wells
Comment 4 2014-04-25 15:17:57 PDT
Created attachment 230212 [details] [PATCH] Proposed Fix: Change to DashboardContainerView.css, Popover.css
WebKit Commit Bot
Comment 5 2014-04-25 16:43:47 PDT
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>
WebKit Commit Bot
Comment 6 2014-04-25 16:43:50 PDT
All reviewed patches have been landed. Closing bug.
Antoine Quint
Comment 7 2014-04-26 01:54:23 PDT
Was there a screenshot of the original issue? Sounds to me like a popover should be laid out over the dashboard.
Note You need to log in before you can comment on or make changes to this bug.