Bug 142714 – Web Inspector: Breakpoint icons should not get pushed off of debugger sidebar due to long resource names

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Joseph Pecoraro]

Created attachment 248692 [details]
[IMAGE] Issue - Breakpoints pushed off sidebar

* SUMMARY
Breakpoint icons should not get pushed off of debugger sidebar due to long resource names. See attached screenshot.

[Radar WebKit Bug Importer]

<rdar://problem/20168053>

[Timothy Hatcher]

It looks like there is some min-width forcing the TreeOutline to stop shirnking as the sidebar gets too small. You can see this if you start at a large width and resize smaller.

[Jonathan Wells]

Created attachment 250901 [details]
[PATCH] style fix.

[Jonathan Wells]

Created attachment 250903 [details]
[IMAGE] Fixed.

[Brian Burg]

Comment on attachment 250901 [details]
[PATCH] style fix.

The debugger sidebar has 3 details section. Have you tested the other two (pause reason and call stack)?

[Timothy Hatcher]

Comment on attachment 250901 [details]
[PATCH] style fix.

View in context: https://bugs.webkit.org/attachment.cgi?id=250901&action=review

> Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.css:76
> +.sidebar > .panel.navigation.debugger .details-section.scripts.collapsed > .content {
> +    display: block;
> +}

Why is this one here? If the details section is collapsed, content should be display: none. Like it is for .sidebar > .panel.navigation.debugger .details-section.collapsed > .content.

[Jonathan Wells]

(In reply to comment #5)
> Comment on attachment 250901 [details]
> [PATCH] style fix.
> 
> The debugger sidebar has 3 details section. Have you tested the other two
> (pause reason and call stack)?

I have, and made sure collapsing and expanding each section worked properly. Everything looks as it should.

[Jonathan Wells]

(In reply to comment #6)
> Comment on attachment 250901 [details]
> [PATCH] style fix.
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=250901&action=review
> 
> > Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.css:76
> > +.sidebar > .panel.navigation.debugger .details-section.scripts.collapsed > .content {
> > +    display: block;
> > +}
> 
> Why is this one here? If the details section is collapsed, content should be
> display: none. Like it is for .sidebar > .panel.navigation.debugger
> .details-section.collapsed > .content.

When the debugger is not paused, we now only show the scripts .content div and we hide its header. So we never want it to be collapsed unless the debugger is paused. Otherwise a user might collapse the scripts panel, then resume the debugger, and suddenly there is nothing to see, no scripts, and no way to expand the scripts.

[WebKit Commit Bot]

Comment on attachment 250901 [details]
[PATCH] style fix.

Clearing flags on attachment: 250901

Committed r182939: <http://trac.webkit.org/changeset/182939>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.