New issue
Advanced search Search tips

Issue 879617 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 31
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Update bundled Play Store icon for MD compliance

Project Member Reported by kaznacheev@chromium.org, Aug 31

Issue description

Currently bundled Play Store icons have too much padding. Also, only 48px and 96px resolutions are available, which leads to upscaling in the new style launcher (where icons are shown at 64dip).

 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 31

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1a715f17837d89a7c42fa846b3176784a5ee2a29

commit 1a715f17837d89a7c42fa846b3176784a5ee2a29
Author: Vladislav Kaznacheev <kaznacheev@google.com>
Date: Fri Aug 31 18:29:26 2018

Update bundled Play Store icons

- New icons are MD compliant
- Adding 32px, 64px and 128px versions to avoid excessive scaling

Bug:  879617 
Test: manual
Change-Id: I4767a039aec651ac54e66c997e7855e0ff0f0197
Reviewed-on: https://chromium-review.googlesource.com/1199805
Reviewed-by: Yury Khmel <khmel@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Vladislav Kaznacheev <kaznacheev@chromium.org>
Cr-Commit-Position: refs/heads/master@{#588097}
[add] https://crrev.com/1a715f17837d89a7c42fa846b3176784a5ee2a29/chrome/browser/resources/chromeos/arc_support/icon/128.png
[add] https://crrev.com/1a715f17837d89a7c42fa846b3176784a5ee2a29/chrome/browser/resources/chromeos/arc_support/icon/32.png
[modify] https://crrev.com/1a715f17837d89a7c42fa846b3176784a5ee2a29/chrome/browser/resources/chromeos/arc_support/icon/48.png
[add] https://crrev.com/1a715f17837d89a7c42fa846b3176784a5ee2a29/chrome/browser/resources/chromeos/arc_support/icon/64.png
[modify] https://crrev.com/1a715f17837d89a7c42fa846b3176784a5ee2a29/chrome/browser/resources/chromeos/arc_support/icon/96.png
[modify] https://crrev.com/1a715f17837d89a7c42fa846b3176784a5ee2a29/chrome/browser/resources/component_extension_resources.grd
[modify] https://crrev.com/1a715f17837d89a7c42fa846b3176784a5ee2a29/chrome/browser/ui/app_list/arc/arc_app_icon.cc

Status: Fixed (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 11

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/245d5bd42732840ea7fd42b8ab68553c7ef21257

commit 245d5bd42732840ea7fd42b8ab68553c7ef21257
Author: Vladislav Kaznacheev <kaznacheev@google.com>
Date: Tue Sep 11 18:48:44 2018

Use fewer sizes for bundled Play Store icons

- Use only 32 and 192 sizes
- Apply png optimizer.
- Use correct icons in icon_decode_request.cc

Bug:  879617 
Test: manual
Change-Id: Icced1e403c75852b2143c695dd22032c3a9e4f8c
Reviewed-on: https://chromium-review.googlesource.com/1216084
Reviewed-by: Yury Khmel <khmel@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Vladislav Kaznacheev <kaznacheev@chromium.org>
Cr-Commit-Position: refs/heads/master@{#590420}
[modify] https://crrev.com/245d5bd42732840ea7fd42b8ab68553c7ef21257/chrome/browser/chromeos/arc/icon_decode_request.cc
[delete] https://crrev.com/2f9563e49f6c28dfe52940901417a4031b95a49b/chrome/browser/resources/chromeos/arc_support/icon/128.png
[add] https://crrev.com/245d5bd42732840ea7fd42b8ab68553c7ef21257/chrome/browser/resources/chromeos/arc_support/icon/192.png
[modify] https://crrev.com/245d5bd42732840ea7fd42b8ab68553c7ef21257/chrome/browser/resources/chromeos/arc_support/icon/32.png
[delete] https://crrev.com/2f9563e49f6c28dfe52940901417a4031b95a49b/chrome/browser/resources/chromeos/arc_support/icon/48.png
[delete] https://crrev.com/2f9563e49f6c28dfe52940901417a4031b95a49b/chrome/browser/resources/chromeos/arc_support/icon/64.png
[delete] https://crrev.com/2f9563e49f6c28dfe52940901417a4031b95a49b/chrome/browser/resources/chromeos/arc_support/icon/96.png
[modify] https://crrev.com/245d5bd42732840ea7fd42b8ab68553c7ef21257/chrome/browser/resources/component_extension_resources.grd
[modify] https://crrev.com/245d5bd42732840ea7fd42b8ab68553c7ef21257/chrome/browser/ui/app_list/arc/arc_app_icon.cc

Sign in to add a comment