Bug 147774

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:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Patch	timothy: review-
[Proposed] After Patch is applied	none
Applied patch with 200ms Timeout	none

Devin Rousso

Reported 2015-08-07 01:34:06 PDT

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.

Attachments
Patch (11.57 KB, patch) 2015-08-07 01:40 PDT, Devin Rousso	timothy: review-	Details Formatted Diff Diff
[Proposed] After Patch is applied (3.53 MB, video/quicktime) 2015-08-07 01:41 PDT, Devin Rousso	no flags	Details
Applied patch with 200ms Timeout (3.87 MB, video/quicktime) 2015-08-07 11:20 PDT, Devin Rousso	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Devin Rousso

Comment 1 2015-08-07 01:40:57 PDT

Created attachment 258468 [details] Patch

Devin Rousso

Comment 2 2015-08-07 01:41:50 PDT

Created attachment 258469 [details] [Proposed] After Patch is applied

Timothy Hatcher

Comment 3 2015-08-07 10:24:14 PDT

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.

Devin Rousso

Comment 4 2015-08-07 11:20:25 PDT

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.

Timothy Hatcher

Comment 5 2015-08-07 11:24:26 PDT

That looks better to me.

Nikita Vasilyev

Comment 6 2015-08-07 11:37:53 PDT

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

Blaze Burg

Comment 7 2015-08-07 18:35:38 PDT

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

Timothy Hatcher

Comment 8 2015-08-07 19:55:53 PDT

Yeah, I have concerns about it matching anything else on the system.

Timothy Hatcher

Comment 9 2015-08-11 22:39:34 PDT

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.

David Kilzer (:ddkilzer)

Comment 10 2016-09-09 11:19:49 PDT

<rdar://problem/28228731>

Blaze Burg

Comment 11 2016-09-10 01:06:38 PDT

Should this be marked WONTFIX? I haven't heard anyone complain about it. Also it would be hard to make this accessible.

Note You need to log in before you can comment on or make changes to this bug.