Bug 132205

Summary: Web Inspector: DOM Tree debugger popover obscures dashboard
Product: WebKit Reporter: jonowells
Component: Web InspectorAssignee: 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 Flags
[PATCH] Proposed Fix: Change to DashboardContainerView.css
none
[PATCH] Proposed Fix: Change to DashboardContainerView.css, Popover.css none

Description jonowells 2014-04-25 14:34:03 PDT
<rdar://problem/16622125>
Comment 1 jonowells 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;
}
Comment 2 jonowells 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.
Comment 3 Joseph Pecoraro 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
Comment 4 jonowells 2014-04-25 15:17:57 PDT
Created attachment 230212 [details]
[PATCH] Proposed Fix: Change to DashboardContainerView.css, Popover.css
Comment 5 WebKit Commit Bot 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>
Comment 6 WebKit Commit Bot 2014-04-25 16:43:50 PDT
All reviewed patches have been landed.  Closing bug.
Comment 7 Antoine Quint 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.