Summary: | Web Inspector: brush up breakpoints UI and UX. | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Pavel Feldman <pfeldman> | ||||||||
Component: | Web Inspector (Deprecated) | Assignee: | Pavel Feldman <pfeldman> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, podivilov, rik, timothy, yurys | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Attachments: |
|
Description
Pavel Feldman
2010-11-03 02:11:52 PDT
Created attachment 72797 [details]
[IMAGE] Screenshot while running with patch (1).
Created attachment 72798 [details]
[IMAGE] Screenshot while running with patch (2).
Created attachment 72799 [details]
[PATCH] Proposed fix.
Looks good. Comment on attachment 72799 [details]
[PATCH] Proposed fix.
rubber stamp = me
Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/ChangeLog M WebCore/inspector/front-end/BreakpointManager.js M WebCore/inspector/front-end/BreakpointsSidebarPane.js M WebCore/inspector/front-end/ElementsTreeOutline.js M WebCore/inspector/front-end/inspector.css Committed r71234 Comment on attachment 72799 [details] [PATCH] Proposed fix. View in context: https://bugs.webkit.org/attachment.cgi?id=72799&action=review > WebCore/inspector/front-end/inspector.css:4102 > - margin: 4px 0; > - color: rgb(33%, 33%, 33%); > - cursor: pointer; > + padding: 2px 0; > + color: black; What cursor is used for the breakpoint-list? As long as it isn't the ibar! > WebCore/inspector/front-end/inspector.css:4396 > \ No newline at end of file > It's announcing it =). |