(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo Reporter
	Description • 9 years ago

Build 10122 of Windows 10 has introduced a new style for context menus.

Besides the pure style changes, one major difference is that the menu has more spacing when it is triggered through a long-press on a touch screen as opposed to a right click with the mouse (see attachment).

We should mirror both the style and that behavior.

	Dão Gottwald [:dao]
	Comment 2 • 9 years ago

Can you provide a more spelled out spec / list of things we should change?

	(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo Reporter
	Comment 3 • 9 years ago

Here's the WIP spec: http://phlsa.github.io/fx-win10-context-menu/
Everything that is in there should be good to go, but there are some elements still missing (like the arrow for submenus).

Does a spec like that in CSS/HTML help you, or would you prefer a list of things that need to change?

	Dão Gottwald [:dao]
	Comment 4 • 9 years ago

We don't implement menus entirely in CSS, a lot of it is native widget rendering, so a list of points where our current rendering is off and should change would really be helpful.

	(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo Reporter
	Comment 5 • 9 years ago

Oh, OK - no problem!

- Brighter background color (#fff)
- Light gray as the hover color (#e5e5e5)
- No border on hover effect
- Higher items (more top and bottom padding on the items - 3px paddint top and bottom)
- No vertical rule at the left side of the items
- No colored box around checkmarks and radio items
- Different arrow graphic (tbd)
- Add padding at the top and bottom of the menu (5px)

I think that's most of the differences.

	Dão Gottwald [:dao]
	Comment 6 • 9 years ago

(In reply to Philipp Sackl [:phlsa] please use needinfo from comment #5)
> Oh, OK - no problem!
> 
> - Brighter background color (#fff)
> - Light gray as the hover color (#e5e5e5)
> - No border on hover effect
> - Higher items (more top and bottom padding on the items - 3px paddint top
> and bottom)
> - No vertical rule at the left side of the items
> - No colored box around checkmarks and radio items
> - Different arrow graphic (tbd)
> - Add padding at the top and bottom of the menu (5px)
> 
> I think that's most of the differences.

Jim, most of the above stuff seems like it should be handled by native widget rendering. Do you think this can be fixed there?

	Jim Mathies [:jimm]
	Comment 7 • 9 years ago

(In reply to Dão Gottwald [:dao] from comment #6)
> (In reply to Philipp Sackl [:phlsa] please use needinfo from comment #5)
> > Oh, OK - no problem!
> > 
> > - Brighter background color (#fff)
> > - Light gray as the hover color (#e5e5e5)
> > - No border on hover effect
> > - Higher items (more top and bottom padding on the items - 3px paddint top
> > and bottom)
> > - No vertical rule at the left side of the items
> > - No colored box around checkmarks and radio items
> > - Different arrow graphic (tbd)
> > - Add padding at the top and bottom of the menu (5px)
> > 
> > I think that's most of the differences.
> 
> Jim, most of the above stuff seems like it should be handled by native
> widget rendering. Do you think this can be fixed there?

We might want to wait and check the rtm, the theme library tends to get updated last.

	(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo Reporter
	Comment 8 • 9 years ago

The changes I described are already implemented and being used in Windows Explorer, but not other classic Windows apps.
Could we just ask someone at Microsoft about whether or not they will update their theme library accordingly for the launch?

	Jim Mathies [:jimm]
	Comment 9 • 9 years ago

(In reply to Philipp Sackl [:phlsa] please use needinfo from comment #8)
> The changes I described are already implemented and being used in Windows
> Explorer, but not other classic Windows apps.
> Could we just ask someone at Microsoft about whether or not they will update
> their theme library accordingly for the launch?

They always update the theme library, but they usually do it near the end of their dev cycle. We had a similar problem with scrollbars in win8. Note most apps do not use the theme lib, we are 'special' in that our entire UI is rendered internally vs. using common control libraries.

I personally do not have a contact at ms for something like this.

	Tim Nguyen :ntim
	Comment 10 • 9 years ago

Windows 10 has a lot of different context menu styles, we should probably wait until MS makes up it's mind in a further update for a consistent style (if that ever happens ;) ).

	:Gijs (he/him)
	Comment 11 • 9 years ago

They've changed the context menu styles again (supposed to be more consistent now, see http://blogs.windows.com/bloggingwindows/2015/08/27/windows-10-insider-preview-build-10532-for-pc/ ), but it seems they still haven't updated what style is exposed to Firefox. :-(

	Sylvestre Ledru [:Sylvestre]
	Comment 12 • 6 years ago

Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information