RESOLVED FIXED 221246
Web Inspector: remove experimental setting and enable Layout sidebar 
https://bugs.webkit.org/show_bug.cgi?id=221246
Summary Web Inspector: remove experimental setting and enable Layout sidebar
Blaze Burg
Reported 2021-02-01 22:57:28 PST
Placeholder bug for later.
Attachments
Patch (4.33 KB, patch)
2021-03-15 12:41 PDT, Nikita Vasilyev 		no flags
DetailsFormatted DiffDiff
Patch (5.31 KB, patch)
2021-03-15 12:47 PDT, Nikita Vasilyev 		no flags
DetailsFormatted DiffDiff
Patch (5.17 KB, patch)
2021-03-15 16:59 PDT, Nikita Vasilyev 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Blaze Burg
Comment 1 2021-02-01 23:00:56 PST
As part of this, the engineering-only context menu items for showing/hiding grid overlays should be removed. At that point they will no longer be necessary.
Radar WebKit Bug Importer
Comment 2 2021-02-08 22:58:12 PST
<rdar://problem/74130753>
Nikita Vasilyev
Comment 3 2021-03-15 12:37:45 PDT
(In reply to BJ Burg from comment #1) > As part of this, the engineering-only context menu items for showing/hiding > grid overlays should be removed. At that point they will no longer be > necessary. This was removed in Web Inspector: CSS Grid Inspector clean-up https://trac.webkit.org/changeset/274229/webkit
Nikita Vasilyev
Comment 4 2021-03-15 12:41:13 PDT Comment hidden (obsolete)
Nikita Vasilyev
Comment 5 2021-03-15 12:47:54 PDT
Created attachment 423223 [details] Patch
Devin Rousso
Comment 6 2021-03-15 12:50:35 PDT
Comment on attachment 423223 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=423223&action=review > Source/WebInspectorUI/UserInterface/Views/ElementsTabContentView.js:33 > + WI.LayoutDetailsSidebarPanel, Shouldn't this only be added if the backend has `CSS.nodeLayoutContextTypeChanged` (and/or any of the other new protocol things)?
Nikita Vasilyev
Comment 7 2021-03-15 16:59:08 PDT
Created attachment 423264 [details] Patch
Nikita Vasilyev
Comment 8 2021-03-15 17:00:45 PDT
Comment on attachment 423264 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=423264&action=review > Source/WebInspectorUI/UserInterface/Views/ElementsTabContentView.js:36 > + // COMPATIBILITY (iOS 14.5): `DOM.showGridOverlay` did not exist yet. > + if (InspectorBackend.hasCommand("DOM.showGridOverlay")) I don't know if should check for anything else here. Other commands were added around the same time, and it looks like we usually check for only one command elsewhere in our codebase.
Blaze Burg
Comment 9 2021-03-15 22:44:20 PDT
Comment on attachment 423264 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=423264&action=review r=me >> Source/WebInspectorUI/UserInterface/Views/ElementsTabContentView.js:36 >> + if (InspectorBackend.hasCommand("DOM.showGridOverlay")) > > I don't know if should check for anything else here. Other commands were added around the same time, and it looks like we usually check for only one command elsewhere in our codebase. This is fine.
EWS
Comment 10 2021-03-15 22:55:24 PDT
Committed r274466: <https://commits.webkit.org/r274466> All reviewed patches have been landed. Closing bug and clearing flags on attachment 423264 [details].
Note You need to log in before you can comment on or make changes to this bug.