Bug 29195

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 Flags
Console Scope Bars
timothy: review+, bweinstein: commit-queue-
Screen Shot none

Description Brian Weinstein 2009-09-11 11:18:17 PDT
The Web Inspector Console should have scope bars.
Comment 1 Brian Weinstein 2009-09-11 11:19:02 PDT
Created attachment 39452 [details]
Console Scope Bars
Comment 2 Anthony Ricaud 2009-09-11 11:27:40 PDT
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.
Comment 3 Brian Weinstein 2009-09-11 11:34:54 PDT
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.
Comment 4 Brian Weinstein 2009-09-11 11:35:52 PDT
Created attachment 39455 [details]
Screen Shot

With errors and warnings selected.
Comment 5 Anthony Ricaud 2009-09-11 11:49:16 PDT
Of course, the bottom bar. Disregard my previous comment.
Comment 6 Timothy Hatcher 2009-09-11 12:02:26 PDT
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.
Comment 7 Brian Weinstein 2009-09-11 13:46:47 PDT
Committed in http://trac.webkit.org/changeset/48314