1396340 - Context Menu Does Not Follow Touch Density

Status: RESOLVED WONTFIX

(Firefox :: Menus, defect)

Description

[Amr]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0
Build ID: 20170902100317
Firefox for Android

Steps to reproduce:

- In "Customize Nightly", at the bottom, choose "Touch" from the "Density" menu.
- In any tab, right click anywhere, notice the spacing between items.
- Select text, right click, notice the spacing between items.
- Back to "Customize Nightly", choose "Normal" from the "Density" menu.
- In any tab, right click anywhere to open context menu, notice the spacing between items. 



Actual results:

- Spacing doesn't change regardless of the density selected.
- This is especially difficult in high DPI monitors where items are very small and close together
- Even more difficult for actual touch screen like Surface devices


Expected results:

- Spacing should respect selected density.
- For example, in Windows Explorer, when Windows is in touch mode, context menu is much better spaced and easier to use.

Comment 1

[Amr]

Attachment: Spacing of Context menu items while in Touch density

Comment 2

[Johann Hofmann [:johannh]]

You can get larger menu items by actually accessing them through touch, not with your mouse. I don't think it's intended to always resize in touch mode (that would make the menus quite large), though I'll defer to shorlander to WONTFIX this.

Comment 3

[Amr]

I figured out what's different between our tests.
All the results below are while Windows is in tablet mode, and Firefox is in Touch density. 

I found that if I bring up the context menu using my finger (touch event), the context menu actually respects Touch density and shows up with good spacing.
However, if I use a pen (on a Surface device), the regular menu shows up without following touch density.
I think in the second case, it's responding to a Click event. 

I believe it should respect Touch density because that's the selected option regardless of how the menu was opened.
If a user selects Touch density for accessibility reasons (and not to simply use Touch mode), they should have a well spaced out menu.

Comment 4

[Jared Wein [:jaws] (please needinfo? me)]

Closing as WONTFIX per comment #2. I'll leave the needinfo open for shorlander so he can reopen the bug if he thinks we should make this change.