Summary: | Web Inspector: Modify Styles sidebar navigation item to show select popover on hover | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Devin Rousso <hi> | ||||||||
Component: | Web Inspector | Assignee: | Devin Rousso <hi> | ||||||||
Status: | RESOLVED WONTFIX | ||||||||||
Severity: | Normal | CC: | bburg, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Attachments: |
|
Description
Devin Rousso
2015-08-07 01:34:06 PDT
Created attachment 258468 [details]
Patch
Created attachment 258469 [details]
[Proposed] After Patch is applied
I worry that if you try to mouse from Node to Layers in a straight line over Styles you won't be able to click on Layers. You will likely hit the Styles popup menu item which covers Layers. Maybe a slight delay for the hover? But at that point, it still slows you down or makes you click on Styles to get the popup faster. Created attachment 258514 [details]
Applied patch with 200ms Timeout
This actually isn't too bad. It could use some fine tuning, but it works much better than I was expecting.
That looks better to me. I don't like showing any navigation menus on hover. I don't recall it being used on OS X either. (In reply to comment #0) > It's pretty annoying to have to click twice to change style sidebar panels, > so instead of requiring a click to open the select element, have it > automatically open on hover. You can select a different item by clicking and holding, which is technically only one click. https://cloudup.com/ciadDyu73CY Comment on attachment 258468 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=258468&action=review > Source/WebInspectorUI/UserInterface/Views/ScopeRadioButtonNavigationItem.js:87 > + var from = event.fromElement || event.relatedTarget; let Yeah, I have concerns about it matching anything else on the system. Comment on attachment 258468 [details]
Patch
r- for now. We should live with wha the have for a bit longer. We might need something different altogether.
Should this be marked WONTFIX? I haven't heard anyone complain about it. Also it would be hard to make this accessible. |