Bug 213492

Summary: Web Inspector: Elements: Node: use a `WI.ContextMenu` instead of a `<select>` for listener filtering
Product: WebKit Reporter: Devin Rousso <hi>
Component: Web InspectorAssignee: Devin Rousso <hi>
Status: RESOLVED FIXED    
Severity: Normal CC: hi, inspector-bugzilla-changes, nvasilyev, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Description Devin Rousso 2020-06-22 15:56:21 PDT
Using a `<select>` means that the context menu will appear in the same place no matter where the icon is clicked, but that is not needed, so just use a `WI.ContextMenu` instead (it's also simpler).
Comment 1 Devin Rousso 2020-06-22 15:57:42 PDT
Created attachment 402518 [details]
Patch
Comment 2 EWS 2020-06-22 16:43:51 PDT
Committed r263376: <https://trac.webkit.org/changeset/263376>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 402518 [details].
Comment 3 Radar WebKit Bug Importer 2020-06-22 16:44:16 PDT
<rdar://problem/64614336>
Comment 4 Nikita Vasilyev 2020-06-22 17:34:55 PDT
From the accessibility point, I think we should go the exact opposite way - use <select> elements instead of WI.ContextMenu on <div> elements whenever possible.

In VoiceOver, when simulating a click by pressing Cmd-Shift-Ctrl-Option-Space, the context menu is shown as expected, but it isn't immediately focused.

Also, with this patch, the filter icon is no longer tab-focusable.