Summary: | Web Inspector: Sources: gear icons moves to 2nd line when sidebar is narrow | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Nikita Vasilyev <nvasilyev> | ||||||||
Component: | Web Inspector | Assignee: | Devin Rousso <hi> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | commit-queue, hi, inspector-bugzilla-changes, jamaln, joepeck, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Attachments: |
|
Removing the flex-wrap property from .navigation-bar and .navigation-bar .item in NavigationBar.css fixes this, but I'm guessing those are in there serving another purpose and removing them may not be the best idea... https://i.imgur.com/8bWGoEB.png Created attachment 376347 [details]
Patch
Comment on attachment 376347 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=376347&action=review r=me > Source/WebInspectorUI/ChangeLog:54 > + > + Nit: too many lines > Source/WebInspectorUI/UserInterface/Views/TextNavigationItem.js:40 > // Public > > + get totalMargin() Shouldn't this be in some Protected region? Comment on attachment 376347 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=376347&action=review >> Source/WebInspectorUI/ChangeLog:54 >> + > > Nit: too many lines Oops :P >> Source/WebInspectorUI/UserInterface/Views/TextNavigationItem.js:40 >> + get totalMargin() > > Shouldn't this be in some Protected region? Yes, it should be. I'd originally intended this to be `static`, but due to `WI.ButtonNavigationItem.Style` it ended up being a member variable. I'll move it. Created attachment 376597 [details]
Patch
Comment on attachment 376597 [details] Patch Clearing flags on attachment: 376597 Committed r248818: <https://trac.webkit.org/changeset/248818> All reviewed patches have been landed. Closing bug. |
Created attachment 370200 [details] [Image] Bug The gear icon should stay on the same line — it shouldn't wrap.