Bug 48901 – Web Inspector: brush up breakpoints UI and UX.

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Pavel Feldman]

- Numerous style improvements
- Start editing XHR on double click
- Do not convert entire DOM breakpoint into a link
- Render DOM breakpoints on two rows
- etc.

[Pavel Feldman]

Created attachment 72797 [details]
[IMAGE] Screenshot while running with patch (1).

[Pavel Feldman]

Created attachment 72798 [details]
[IMAGE] Screenshot while running with patch (2).

[Pavel Feldman]

Created attachment 72799 [details]
[PATCH] Proposed fix.

[Pavel Podivilov]

Looks good.

[Jeremy Orlow]

Comment on attachment 72799 [details]
[PATCH] Proposed fix.

rubber stamp = me

[Pavel Feldman]

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

[Joseph Pecoraro]

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 =).