Issue metadata
Sign in to add a comment
|
Regression : 'Gallery' icon is seen blur in Files App context Menu |
||||||||||||||||||||||
Issue descriptionChrome Version: 71.0.3572.0/11140.0.0 dev-channel Daisy,Reks and Kip OS: Chrome What steps will reproduce the problem? (1)Sign into User -> press 'Ctrl+F5(Overview Mode Button)' for screenshot -> go to Files App-> Right click for context menu (2)Now observe 'Gallery' icon is seen blur available at first option in context menu (Please refer Video and Screenshot) Expected: 'Gallery' icon should not be seen blur in context Menu Actual: Instead 'Gallery' icon is seen blur This is Regression Issue as same is working fine in M-70
,
Oct 11
I confirm that I can reproduce locally. From gallery icons history, I traced to this CL: https://chromium-review.googlesource.com/1243680 Assigning to CL owner.
,
Oct 12
,
Oct 12
Apparently removing a 32px icon was not as safe as I assumed. Preparing a CL that restores all icon sizes that were previously used. Will need new icon files for these sizes, requesting them from UX.
,
Nov 8
,
Nov 8
,
Nov 8
,
Nov 9
All assets for the gallery app attached.
,
Nov 9
These work great, thank! However, I need more sizes for the other 3 icon sets (files app, video player, audio player). I am sorry, I should have been clearer.
,
Nov 9
,
Nov 14
,
Nov 15
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c commit e9dca2f22e8863e5f8ec15ca228531f29cec5d5c Author: Vladislav Kaznacheev <kaznacheev@google.com> Date: Thu Nov 15 15:30:59 2018 Bring back full range of icon sizes for Files app Return to using icon sizes from before crrev.com/c/1243680. This should exclude a possibility of any regressions. Updating the icon look to the latest UI design. Bug: 893548 Test: steps in bug Change-Id: I596750416a5c87f0239119086b4db84554f3d3a0 Reviewed-on: https://chromium-review.googlesource.com/c/1336188 Reviewed-by: Naoki Fukino <fukino@chromium.org> Commit-Queue: Vladislav Kaznacheev <kaznacheev@chromium.org> Cr-Commit-Position: refs/heads/master@{#608373} [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/audio_player/icons/audio-player-256.png [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/audio_player/icons/audio-player-32.png [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/audio_player/icons/audio-player-64.png [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/audio_player/icons/audio-player-96.png [modify] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/audio_player/manifest.json [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/file_manager/common/images/icon256.png [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/file_manager/common/images/icon32.png [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/file_manager/common/images/icon64.png [modify] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/file_manager/manifest.json [modify] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/file_manager_resources.grd [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/gallery/images/icon256.png [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/gallery/images/icon32.png [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/gallery/images/icon64.png [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/gallery/images/icon96.png [modify] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/gallery/manifest.json [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/video_player/images/icon/video-player-256.png [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/video_player/images/icon/video-player-32.png [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/video_player/images/icon/video-player-64.png [add] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/video_player/images/icon/video-player-96.png [modify] https://crrev.com/e9dca2f22e8863e5f8ec15ca228531f29cec5d5c/ui/file_manager/video_player/manifest.json
,
Nov 15
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by weifangsun@chromium.org
, Oct 9Components: Platform>Apps>PhotoEditor