New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 620817 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug



Sign in to add a comment

Update Chrome for Windows app icon

Project Member Reported by cl...@chromium.org, Jun 16 2016

Issue description

Current app icon shown in Start Menu, etc. is too large.

Assets here: https://drive.google.com/corp/drive/folders/0B9Rk2knI0XDlN0g3VEd3SzB0UkE

Icon is transparent and should be given a pure white "backplate" as per https://msdn.microsoft.com/en-us/windows/uwp/controls-and-patterns/tiles-and-notifications-app-assets
 
current.png
108 KB View Download
desired.png
118 KB View Download
I can do this. Just to clarify, the assets provided are transparent background, but we want solid (255, 255, 255), right?

xref https://bugs.chromium.org/p/chromium/issues/detail?id=571230.
Status: Started (was: Untriaged)
(As I mentioned in the other bug, I think we should do full-bleed with the correct chrome font too eventually, but we can't easily do that due to app shortcut icons needing to have custom titles.)

Comment 3 by cl...@chromium.org, Jun 17 2016

Re comment 2: Correct.
Man, VisualElements is such a disaster. ForegroundText isn't respected on Win10. So, we can't have dark text on Win10 without baking it in, it seems.
It's sort of documented as such here: https://msdn.microsoft.com/en-us/library/windows/apps/dn423310.aspx .

"This value is ignored. All tiles use the light text color."

That note sort of looks like it's supposed to apply to only Phone, but it's clearly not taking effect on Win10 TH2 either.

We also (!) can't specify "transparent" as the BG color (doing so causes the , so we can't match UWP apps).

I looked a couple other apps that use VisualElementsManifest.xml. Dropbox hard codes it to #007EE5 which matches other things nicely... unless the user changes their accent colour which is new after TH2. Firefox picked a neutral grey. Edge also hardcodes to the fixed blue.
"(doing so causes the ," should have been "(doing so causes the manifest to be ignored entirely, "

Comment 7 by cl...@chromium.org, Jun 17 2016

Well, the background color is supposed to be white. The asset is just transparent because it seemed from the documentation (https://msdn.microsoft.com/en-us/windows/uwp/controls-and-patterns/tiles-and-notifications-app-assets) like that was what was necessary to trigger the non-full-bleed tile UI.

But if we can't set dark foreground text, we might need to use the full-bleed tile UI anyway. That runs into a problem with Chrome Apps using the same icon, you said, right?
> But if we can't set dark foreground text, we might need to use the full-bleed tile UI anyway. That runs into a problem with Chrome Apps using the same icon, you said, right?

That's right. We'd have to do #3 here https://bugs.chromium.org/p/chromium/issues/detail?id=571230#c25 which is a large-ish undertaking (or at least a lot bigger than subbing in some new art).

Comment 9 by cl...@chromium.org, Jun 17 2016

Re your comment here: https://bugs.chromium.org/p/chromium/issues/detail?id=571230#c26

Does the issue with Chrome Apps inheriting Chrome's icon actually make a difference for what Chrome's icon should be? Either way, all Chrome Apps have a "bunch of indistinguishable icons," no?

Comment 10 by cl...@chromium.org, Jun 17 2016

Never mind, I realize what you mean - these full-bleed tiles would also lack app name text, so they'd really be indistinguishable.

Given all this, let's use the same non-full-bleed icon but on a dark background. I'll get back to you with a value.

Comment 11 by cl...@chromium.org, Jun 17 2016

All right, how about setting #212121 as the background color? I'd expect something like this.
CL here with new assets and #212121: https://codereview.chromium.org/2073123003
Status: Fixed (was: Started)
I'm going to call this fixed for now. We should revisit once we have a solution for apps icons to make this prettier (i.e. white background).

Sign in to add a comment