132205 2014-04-25 14:34:03 -0700 Web Inspector: DOM Tree debugger popover obscures dashboard 2014-04-26 01:54:23 -0700 1 1 1 Unclassified WebKit Web Inspector 528+ (Nightly build) All All RESOLVED FIXED InRadar P2 Normal --- 1 jonowells webkit-unassigned commit-queue graouts joepeck timothy webkit-bug-importer oldest_to_newest 1004213 0 jonowells 2014-04-25 14:34:03 -0700 <rdar://problem/16622125> 1004214 1 jonowells 2014-04-25 14:35:16 -0700 This is a z-index issue. I think this selector in DashboardContainerView.css isn't needed: .toolbar .dashboard.visible { z-index: 100; } 1004222 2 230207 jonowells 2014-04-25 14:43:35 -0700 Created attachment 230207 [PATCH] Proposed Fix: Change to DashboardContainerView.css I believe this z-index property to be unnecessary. 1004228 3 230207 joepeck 2014-04-25 14:47:46 -0700 Comment on attachment 230207 [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 1004238 4 230212 jonowells 2014-04-25 15:17:57 -0700 Created attachment 230212 [PATCH] Proposed Fix: Change to DashboardContainerView.css, Popover.css 1004282 5 230212 commit-queue 2014-04-25 16:43:47 -0700 Comment on attachment 230212 [PATCH] Proposed Fix: Change to DashboardContainerView.css, Popover.css Clearing flags on attachment: 230212 Committed r167833: <http://trac.webkit.org/changeset/167833> 1004283 6 commit-queue 2014-04-25 16:43:50 -0700 All reviewed patches have been landed. Closing bug. 1004328 7 graouts 2014-04-26 01:54:23 -0700 Was there a screenshot of the original issue? Sounds to me like a popover should be laid out over the dashboard. 230207 2014-04-25 14:43:35 -0700 2014-04-25 15:17:57 -0700 [PATCH] Proposed Fix: Change to DashboardContainerView.css popover-overlap.diff text/plain 1298 jonowells ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2Vi SW5zcGVjdG9yVUkvQ2hhbmdlTG9nCmluZGV4IDQ3YzdkYTUuLjE1Y2U1YWMgMTAwNjQ0Ci0tLSBh L1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkluc3BlY3Rv clVJL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDE0LTA0LTI1ICBKb25hdGhhbiBXZWxs cyAgPGpvbm93ZWxsc0BhcHBsZS5jb20+CisKKyAgICAgICAgV2ViIEluc3BlY3RvcjogRE9NIFRy ZWUgZGVidWdnZXIgcG9wb3ZlciBvYnNjdXJlcyBkYXNoYm9hcmQKKyAgICAgICAgaHR0cHM6Ly9i dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEzMjIwNQorCisgICAgICAgIFJldmlld2Vk IGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogVXNlckludGVyZmFjZS9WaWV3cy9EYXNo Ym9hcmRDb250YWluZXJWaWV3LmNzczoKKyAgICAgICAgKC50b29sYmFyIC5kYXNoYm9hcmQudmlz aWJsZSk6IERlbGV0ZWQuCisKIDIwMTQtMDQtMjQgIFRpbW90aHkgSGF0Y2hlciAgPHRpbW90aHlA YXBwbGUuY29tPgoKICAgICAgICAgV2ViIEluc3BlY3RvcjogRGFzaGJvYXJkIHdoZW4gcGF1c2Vk IGluIGRlYnVnZ2VyIHNob3dzIHNvbWUgbWlzc2luZyBnbHlwaCBib3hlcwpkaWZmIC0tZ2l0IGEv U291cmNlL1dlYkluc3BlY3RvclVJL1VzZXJJbnRlcmZhY2UvVmlld3MvRGFzaGJvYXJkQ29udGFp bmVyVmlldy5jc3MgYi9Tb3VyY2UvV2ViSW5zcGVjdG9yVUkvVXNlckludGVyZmFjZS9WaWV3cy9E YXNoYm9hcmRDb250YWluZXJWaWV3LmNzcwppbmRleCAwMzI0ZjMwLi5mMmE5MjI0IDEwMDY0NAot LS0gYS9Tb3VyY2UvV2ViSW5zcGVjdG9yVUkvVXNlckludGVyZmFjZS9WaWV3cy9EYXNoYm9hcmRD b250YWluZXJWaWV3LmNzcworKysgYi9Tb3VyY2UvV2ViSW5zcGVjdG9yVUkvVXNlckludGVyZmFj ZS9WaWV3cy9EYXNoYm9hcmRDb250YWluZXJWaWV3LmNzcwpAQCAtOTAsMTAgKzkwLDYgQEAgYm9k eS53aW5kb3ctaW5hY3RpdmUgLnRvb2xiYXIgLmRhc2hib2FyZC1jb250YWluZXIgewogICAgIC13 ZWJraXQtYW5pbWF0aW9uLWR1cmF0aW9uOiAwLjRzOwogfQoKLS50b29sYmFyIC5kYXNoYm9hcmQu dmlzaWJsZSB7Ci0gICAgei1pbmRleDogMTAwOwotfQotCiAudG9vbGJhciAuZGFzaGJvYXJkLnNs aWRlLW91dC11cCB7CiAgICAgLXdlYmtpdC1hbmltYXRpb24tbmFtZTogc2xpZGUtdG9wLWVkZ2U7 CiAgICAgLXdlYmtpdC1hbmltYXRpb24tZGlyZWN0aW9uOiByZXZlcnNlOwo= 230212 2014-04-25 15:17:57 -0700 2014-04-25 16:43:47 -0700 [PATCH] Proposed Fix: Change to DashboardContainerView.css, Popover.css popover-overlap-rev1.diff text/plain 1755 jonowells ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2Vi SW5zcGVjdG9yVUkvQ2hhbmdlTG9nCmluZGV4IDQ3YzdkYTUuLmM1MGI4N2YgMTAwNjQ0Ci0tLSBh L1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkluc3BlY3Rv clVJL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDE0LTA0LTI1ICBKb25hdGhhbiBXZWxs cyAgPGpvbm93ZWxsc0BhcHBsZS5jb20+CisKKyAgICAgICAgV2ViIEluc3BlY3RvcjogRE9NIFRy ZWUgZGVidWdnZXIgcG9wb3ZlciBvYnNjdXJlcyBkYXNoYm9hcmQKKyAgICAgICAgaHR0cHM6Ly9i dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEzMjIwNQorCisgICAgICAgIFJldmlld2Vk IGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogVXNlckludGVyZmFjZS9WaWV3cy9EYXNo Ym9hcmRDb250YWluZXJWaWV3LmNzczoKKyAgICAgICAgKC50b29sYmFyIC5kYXNoYm9hcmQudmlz aWJsZSk6IENoYW5nZSB6LWluZGV4IHRvIDEuIEZvcmNlIHN0YWNraW5nIGNvbnRleHQuCisgICAg ICAgICogVXNlckludGVyZmFjZS9WaWV3cy9Qb3BvdmVyLmNzczoKKyAgICAgICAgKC5wb3BvdmVy KTogQWRkIHotaW5kZXg6IDEwMC4KKwogMjAxNC0wNC0yNCAgVGltb3RoeSBIYXRjaGVyICA8dGlt b3RoeUBhcHBsZS5jb20+CgogICAgICAgICBXZWIgSW5zcGVjdG9yOiBEYXNoYm9hcmQgd2hlbiBw YXVzZWQgaW4gZGVidWdnZXIgc2hvd3Mgc29tZSBtaXNzaW5nIGdseXBoIGJveGVzCmRpZmYgLS1n aXQgYS9Tb3VyY2UvV2ViSW5zcGVjdG9yVUkvVXNlckludGVyZmFjZS9WaWV3cy9EYXNoYm9hcmRD b250YWluZXJWaWV3LmNzcyBiL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1Zp ZXdzL0Rhc2hib2FyZENvbnRhaW5lclZpZXcuY3NzCmluZGV4IDAzMjRmMzAuLjI2OTg2ZGEgMTAw NjQ0Ci0tLSBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1ZpZXdzL0Rhc2hi b2FyZENvbnRhaW5lclZpZXcuY3NzCisrKyBiL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50 ZXJmYWNlL1ZpZXdzL0Rhc2hib2FyZENvbnRhaW5lclZpZXcuY3NzCkBAIC05MSw3ICs5MSw3IEBA IGJvZHkud2luZG93LWluYWN0aXZlIC50b29sYmFyIC5kYXNoYm9hcmQtY29udGFpbmVyIHsKIH0K CiAudG9vbGJhciAuZGFzaGJvYXJkLnZpc2libGUgewotICAgIHotaW5kZXg6IDEwMDsKKyAgICB6 LWluZGV4OiAxOyAvKiBFc3RhYmxpc2ggYSBzdGFja2luZyBjb250ZXh0LiAqLwogfQoKIC50b29s YmFyIC5kYXNoYm9hcmQuc2xpZGUtb3V0LXVwIHsKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNw ZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1ZpZXdzL1BvcG92ZXIuY3NzIGIvU291cmNlL1dlYkluc3Bl Y3RvclVJL1VzZXJJbnRlcmZhY2UvVmlld3MvUG9wb3Zlci5jc3MKaW5kZXggZTA3YWU3MC4uMDg2 YjJjMCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkluc3BlY3RvclVJL1VzZXJJbnRlcmZhY2UvVmll d3MvUG9wb3Zlci5jc3MKKysrIGIvU291cmNlL1dlYkluc3BlY3RvclVJL1VzZXJJbnRlcmZhY2Uv Vmlld3MvUG9wb3Zlci5jc3MKQEAgLTI5LDYgKzI5LDcgQEAKICAgICBtaW4taGVpZ2h0OiAyMHB4 OwogICAgIGJveC1zaXppbmc6IGJvcmRlci1ib3g7CiAgICAgcG9pbnRlci1ldmVudHM6IG5vbmU7 CisgICAgei1pbmRleDogMTAwOwogfQoKIC5wb3BvdmVyLmFycm93LXVwIHsK