Bug 132205

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:

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

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	Details Formatted Diff Diff
[PATCH] Proposed Fix: Change to DashboardContainerView.css, Popover.css (1.71 KB, patch) 2014-04-25 15:17 PDT, Jonathan Wells	no flags	Details Formatted Diff Diff
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.