Bug 224206 - Web Inspector: remove duplicate Box Model section from Layout panel in Elements Tab
Summary: Web Inspector: remove duplicate Box Model section from Layout panel in Elemen...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: BJ Burg
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-04-05 14:42 PDT by BJ Burg
Modified: 2021-04-06 13:26 PDT (History)
6 users (show)

See Also:


Attachments
Patch v1.0 (3.06 KB, patch)
2021-04-06 09:51 PDT, BJ Burg
no flags Details | Formatted Diff | Diff
Patch v1.1 (3.09 KB, patch)
2021-04-06 10:03 PDT, BJ Burg
drousso: review+
bburg: commit-queue?
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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>