[webkit-reviews] review denied: [Bug 178047] Web Inspector: decrease outside spacing of content in NetworkResourceDetailsView : [Attachment 323090] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Oct 7 02:06:06 PDT 2017


Joseph Pecoraro <joepeck at webkit.org> has denied Devin Rousso
<webkit at devinrousso.com>'s request for review:
Bug 178047: Web Inspector: decrease outside spacing of content in
NetworkResourceDetailsView
https://bugs.webkit.org/show_bug.cgi?id=178047

Attachment 323090: Patch

https://bugs.webkit.org/attachment.cgi?id=323090&action=review




--- Comment #3 from Joseph Pecoraro <joepeck at webkit.org> ---
Comment on attachment 323090
  --> https://bugs.webkit.org/attachment.cgi?id=323090
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.


More information about the webkit-reviews mailing list