Bug 178047 - Web Inspector: decrease outside spacing of content in NetworkResourceDetailsView
Summary: Web Inspector: decrease outside spacing of content in NetworkResourceDetailsView
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Devin Rousso
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-10-07 00:22 PDT by Devin Rousso
Modified: 2017-10-11 17:57 PDT (History)
3 users (show)

See Also:

Attachments
Patch (1.95 KB, patch)
2017-10-07 00:23 PDT, Devin Rousso 		joepeck: review-
Details | Formatted Diff | Diff
[Image] After Patch is applied (650.92 KB, image/png)
2017-10-07 00:23 PDT, Devin Rousso 		no flags Details
[Image] Before (252.39 KB, image/png)
2017-10-07 15:59 PDT, Devin Rousso 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Devin Rousso 2017-10-07 00:22:57 PDT 
As an example, the padding around the content of the Headers view wastes a lot of space.  I think we can tighten this and still have a nice UI :)
Comment 1 Devin Rousso 2017-10-07 00:23:27 PDT 
Created attachment 323090 [details]
Patch
Comment 2 Devin Rousso 2017-10-07 00:23:40 PDT 
Created attachment 323091 [details]
[Image] After Patch is applied
Comment 3 Joseph Pecoraro 2017-10-07 02:06:06 PDT 
Comment on attachment 323090 [details]
Patch

Only way to tell if this is better is to compare to a before screenshot.

I personally don't like these changes. The extra space between groups was intentional, on advise from someone with UI chops, to make the break between sections more prominent.
Comment 4 Devin Rousso 2017-10-07 15:59:06 PDT 
Created attachment 323104 [details]
[Image] Before
Comment 5 Matt Baker 2017-10-11 17:57:45 PDT 
I agree that we should keep the extra space. Closing.