1174141 - [UX] Update Jump List icons

Status: VERIFIED FIXED

(Firefox :: Theme, defect, P3)

Description

[(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo]

Attachment: Current task bar context menu

In the context menu of the Firefox task bar item, we have icons for new tab, new window and new private window. Those icons are still from the Windows 7 era.

What needs to happen:
- Update the style of the icons (and make sure that they look great on the dark background that Windows uses here)
- Make them hidpi-ready (unclear whether we can specify hidpi graphics here. If not we can at least make sure that the icons we are using scale well)

Comment 1

[Stephen Horlander [:shorlander]]

(In reply to Philipp Sackl [:phlsa] please use needinfo from comment #0)
> - Make them hidpi-ready (unclear whether we can specify hidpi graphics here.
> If not we can at least make sure that the icons we are using scale well)

None of the built in apps have good scaling icons there. Current plan is to add a 32 x 32 icon to the .ico files and see what happens.

Comment 2

[(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo]

Stephen, I think Benjamin created icons for this, right?
Could you attach them to the bug? Thanks!

Comment 3

[Stephen Horlander [:shorlander]]

Attachment: update-taskbar-menu-icons.patch

Updates the styling of the Jump List icons. Windows doesn't seem to make use of the higher resolution icons in the menu.

Comment 4

[Stephen Horlander [:shorlander]]

Attachment: Screenshot

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/273a97c8cfc8

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/273a97c8cfc8

Comment 7

[Dão Gottwald [:dao]]

Comment on attachment 8655508 [details] [diff] [review]
update-taskbar-menu-icons.patch

Approval Request Comment
[Feature/regressing bug #]: Windows 10 release
[User impact if declined]: icons that feel dated
[Describe test coverage new/current, TreeHerder]:
[Risks and why]: replacing ico files, low risk
[String/UUID change made/needed]:

Comment 8

[Dão Gottwald [:dao]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/e1f65e6a5ea0

Comment 9

[Cornel Ionce [:noni] [Hubs QA]]

Verified this as fixed on Windows 10 64bit using:
*Firefox 42, buildID 20151026170526
*Firefox 43 beta 1, build ID:20151103023037.