Summary: | Web Inspector Console should have scope bars | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brian Weinstein <bweinstein> | ||||||
Component: | Web Inspector (Deprecated) | Assignee: | Brian Weinstein <bweinstein> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | rik, timothy | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 14353 | ||||||||
Attachments: |
|
Description
Brian Weinstein
2009-09-11 11:18:17 PDT
Created attachment 39452 [details]
Console Scope Bars
The console has a small vertical space, especially when the inspector is docked. Adding a scope bar reduces this space even more. Is there another way to add this functionality ? Maybe the horizontal space could be exploited. This goes along the bottom bar, next to the "Clear Console" button, so it doesn't take up any extra vertical space. I'm attaching a screen shot after I write this comment. Created attachment 39455 [details]
Screen Shot
With errors and warnings selected.
Of course, the bottom bar. Disregard my previous comment. Comment on attachment 39452 [details] Console Scope Bars > + var createFilterElement = function (category) { This is better written as: function createFilterElement(category) { > + }; Thers is a stray indent space before the "}" brace. No need for the semicolon. > + display: list-item; This should be display: block; Looks good, if you fix these three things. Committed in http://trac.webkit.org/changeset/48314 |