Summary: | Web Inspector: Decrease the padding of each rule in the Rules sidebar to allow more content to show | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Devin Rousso <hi> | ||||||||||||
Component: | Web Inspector | Assignee: | Devin Rousso <hi> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | commit-queue, graouts, joepeck, jonowells, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||
Hardware: | All | ||||||||||||||
OS: | All | ||||||||||||||
Bug Depends on: | |||||||||||||||
Bug Blocks: | 145982 | ||||||||||||||
Attachments: |
|
Description
Devin Rousso
2015-07-28 09:56:56 PDT
Created attachment 257651 [details]
Patch
A before/after screenshot would be great. (In reply to comment #3) > A before/after screenshot would be great. Whoops. Sorry, I completely forgot to upload them. I'll do that now. Created attachment 257652 [details]
After Patch is applied
Created attachment 257653 [details]
Current Functionality
Created attachment 257659 [details]
[Image] Before/after side-by-side
Comment on attachment 257651 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=257651&action=review > Source/WebInspectorUI/UserInterface/Views/CSSStyleDeclarationSection.css:-56 > -.style-declaration-section.last-in-group + .style-declaration-section { > - margin-top: 15px; > - > - border-top-color: hsla(0, 0%, 0%, 0.3); > -} Why is this no longer needed? > Source/WebInspectorUI/UserInterface/Views/CSSStyleDetailsSidebarPanel.css:94 > + content: url(../Images/Plus.svg); You should add opacity: 0.7; Comment on attachment 257651 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=257651&action=review >> Source/WebInspectorUI/UserInterface/Views/CSSStyleDeclarationSection.css:-56 >> -} > > Why is this no longer needed? That was what was used as spacing between rules without media queries before "Media: all" was added. Now that "Media: all" is put before any rule that doesn't have a media query, this rule will never match as there will be a label between .last-in-group and the next .style-declaratio-section. You can set commit-queue + on reviewed patches since you are a commiter. You just can't set review + yet. Comment on attachment 257651 [details] Patch Rejecting attachment 257651 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'apply-attachment', '--no-update', '--non-interactive', 257651, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: torUI/UserInterface/Views/RulesStyleDetailsPanel.js Hunk #1 succeeded at 178 (offset 18 lines). Hunk #2 succeeded at 294 (offset 18 lines). Hunk #3 succeeded at 395 with fuzz 1 (offset 23 lines). Hunk #4 FAILED at 449. 1 out of 4 hunks FAILED -- saving rejects to file Source/WebInspectorUI/UserInterface/Views/RulesStyleDetailsPanel.js.rej Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Timothy Hatcher']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/29611 (In reply to comment #10) > You can set commit-queue + on reviewed patches since you are a commiter. You > just can't set review + yet. Oh haha sorry I didn't know that. Thanks for the info! In this case, add me to the ChangeLog as the reviewer after you rebase. Then upload the patch and cq+ it and leave r blank. Created attachment 258559 [details]
Patch
Comment on attachment 258559 [details] Patch Clearing flags on attachment: 258559 Committed r188188: <http://trac.webkit.org/changeset/188188> All reviewed patches have been landed. Closing bug. |