Issue metadata
Sign in to add a comment
|
Regression: Unwanted highlight is seen on crop dimension in Gallery.app |
||||||||||||||||||||||
Issue descriptionVersion: 55.0.2883.11/8872.8.0 (Official Build) dev-channel Peppy, Gnawty, Daisy OS: Chrome What steps will reproduce the problem? (1) Sign in to user>> try to download 1, 2 images>> open it in gallery>>click on edit settings>> Click on Crop icon>> and observe Expected: Default focus should not be seen on 1st dimensions box (ex: 1x1) Actual: Instead default focus is seen 1st dimensions box (ex: 1x1) This is a Regression as issue is not seen in M54: 54.0.2840.51/8743.57.0 beta channel Peppy Issue is not applicable to Linux and Windows
,
Jun 7 2018
,
Jan 3
I've attempted a fix of this at https://chromium-review.googlesource.com/c/chromium/src/+/1394023
,
Jan 8
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6bf8f784e384ab9a8aeb092aa66e108623e6e6c9 commit 6bf8f784e384ab9a8aeb092aa66e108623e6e6c9 Author: Alexander Nohe <nohe@chromium.org> Date: Tue Jan 08 17:20:40 2019 Remove immediate focus from crop aspect ratio buttons. This ensures that the aspect ratio buttons are not selected when the toolbar is first shown in the photo editor. Removing this focus helps the user realize that the 1x1 aspect ratio is not immediately selected when loading the crop tool. Bug: 655943 Change-Id: I5dd7e11af7af5fefddb01c5587f2cf83078c0c05 Reviewed-on: https://chromium-review.googlesource.com/c/1394023 Commit-Queue: Alexander Nohe <nohe@chromium.org> Reviewed-by: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#620772} [modify] https://crrev.com/6bf8f784e384ab9a8aeb092aa66e108623e6e6c9/ui/file_manager/gallery/js/image_editor/image_editor_toolbar.js [modify] https://crrev.com/6bf8f784e384ab9a8aeb092aa66e108623e6e6c9/ui/file_manager/integration_tests/gallery/photo_editor.js
,
Jan 8
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by yawano@chromium.org
, Dec 20 2016Labels: Hotlist-GoodFirstBug