| Summary: | Web Inspector: Breakpoint icons should not get pushed off of debugger sidebar due to long resource names | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Joseph Pecoraro <joepeck> | ||||||||
| Component: | Web Inspector | Assignee: | Jonathan Wells <jonowells> | ||||||||
| Status: | RESOLVED FIXED | ||||||||||
| Severity: | Normal | CC: | burg, commit-queue, graouts, joepeck, jonowells, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||||||
| Priority: | P2 | Keywords: | InRadar | ||||||||
| Version: | 528+ (Nightly build) | ||||||||||
| Hardware: | All | ||||||||||
| OS: | All | ||||||||||
| Attachments: |
|
||||||||||
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. Created attachment 250901 [details]
[PATCH] style fix.
Created attachment 250903 [details]
[IMAGE] Fixed.
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)?
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. (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. (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. Comment on attachment 250901 [details] [PATCH] style fix. Clearing flags on attachment: 250901 Committed r182939: <http://trac.webkit.org/changeset/182939> All reviewed patches have been landed. Closing bug. |
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.