[Webkit-unassigned] [Bug 86844] Web Inspector: "Deactivate All Breapoints" should visually get reflected in Breakpoints pane.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri May 18 05:35:36 PDT 2012


https://bugs.webkit.org/show_bug.cgi?id=86844





--- Comment #4 from Alexander Pavlov (apavlov) <apavlov at chromium.org>  2012-05-18 05:34:40 PST ---
View in context: https://bugs.webkit.org/attachment.cgi?id=142691&action=review

> Source/WebCore/ChangeLog:9
> +        Breakpoints-pane as well

nit: we usually spell panes, like "the Breakpoints pane", without dashes. The entire description can also fit a single line (WebKit has no limitations on the line length, except the programmer's reason.)

> Source/WebCore/inspector/front-end/inspector.css:2142
> +.breakpoints-list-deactivated {

You could have used the selector of ".breakpoints-deactivated .breakpoint-list" and avoid having to modify ScriptsPanel.js

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list