Bug 224206

Summary: Web Inspector: remove duplicate Box Model section from Layout panel in Elements Tab
Product: WebKit Reporter: BJ Burg <bburg>
Component: Web InspectorAssignee: BJ Burg <bburg>
Status: RESOLVED FIXED    
Severity: Normal CC: bburg, ews-watchlist, hi, inspector-bugzilla-changes, nvasilyev, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch v1.0
none
Patch v1.1 hi: review+

Description BJ Burg 2021-04-05 14:42:48 PDT 
After 220987 was fixed, we now have two Box Model locations. There should only be one.

Option A. Leave in the "Computed" section alongside Computed Properties. This section is of questionable value as it is, and due for rethinking. The list of computed properties is not particularly helpful.

Option B. Leave it in Layout section alongside Grid Inspector tools. This would leave the Computed section rather empty, just with the Computed Properties pane.
Comment 1 Radar WebKit Bug Importer 2021-04-05 14:43:04 PDT 
<rdar://problem/76235731>
Comment 2 BJ Burg 2021-04-06 09:51:24 PDT 
Created attachment 425290 [details]
Patch v1.0
Comment 3 Nikita Vasilyev 2021-04-06 09:57:16 PDT 
Nit: can we retitle the bug "remove Box Model section from Computed" or something like that? This goes to STP changelogs.
Comment 4 BJ Burg 2021-04-06 10:03:41 PDT 
Created attachment 425292 [details]
Patch v1.1
Comment 5 Devin Rousso 2021-04-06 11:44:31 PDT 
Comment on attachment 425292 [details]
Patch v1.1

r=me
Comment 6 BJ Burg 2021-04-06 13:26:48 PDT 
Committed r275545 (236201@main): <https://commits.webkit.org/236201@main>