Non-Regression: Gallery button is seen enabled in New camera app |
|||
Issue descriptionChrome Version:67.0.3381.0/10525.0.0 dev channel Daisy,Kip,Reks OS: Chrome OS What steps will reproduce the problem? (1)Sign in to user>> Install New camera app version: 5.0.0( https://chrome.google.com/webstore/detail/msgname-canary/afoodkbadjkccjlgmeldfjefkfgpalan?hl=en-US&gl=US&authuser=0 ) (2)Now open camera app and observe>> the gallery icon is not clickable but is seen enabled Actual: When the gallery icon is not clickable it should be seen enabled Expected: Gallery icon should be seen disabled This is a Non-Regression issue able to reproduce across all milestones @Yuli: Please confirm the issue
,
Mar 29 2018
,
Mar 31 2018
The gallery button is disabled (working as expected); however, the dimmed opacity for disabled buttons might not be noticeable enough. (It applies to all buttons in camera app: 75% opacity for disabled buttons, and 100% opacity for enabled buttons.) I'll change the dimmed opacity to 70% for all disabled buttons and see if it works better.
,
Apr 3 2018
The following revision refers to this bug: https://chromium.googlesource.com/apps/camera/+/bdd331329457db383de48258b134d61c962610c6 commit bdd331329457db383de48258b134d61c962610c6 Author: Yuli Huang <yuli@chromium.org> Date: Tue Apr 03 01:23:30 2018 Fix disabled buttons' dimmed opacity. BUG= chromium:827122 TEST=Tested manually on elm. Change-Id: Ie70694a3b01ba6dee5a2d8c7a6910f3d6ddf049d Reviewed-on: https://chromium-review.googlesource.com/989357 Reviewed-by: Sheng-hao Tsao <shenghao@google.com> Tested-by: yuli <yuli@chromium.org> [modify] https://crrev.com/bdd331329457db383de48258b134d61c962610c6/src/css/main.css
,
Apr 3 2018
Please verify the fix after the canary is updated. Thanks.
,
Jan 15
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/bdd331329457db383de48258b134d61c962610c6 commit bdd331329457db383de48258b134d61c962610c6 Author: Yuli Huang <yuli@chromium.org> Date: Tue Apr 03 01:23:30 2018 Fix disabled buttons' dimmed opacity. BUG= chromium:827122 TEST=Tested manually on elm. Change-Id: Ie70694a3b01ba6dee5a2d8c7a6910f3d6ddf049d Reviewed-on: https://chromium-review.googlesource.com/989357 Reviewed-by: Sheng-hao Tsao <shenghao@google.com> Tested-by: yuli <yuli@chromium.org> [modify] https://crrev.com/bdd331329457db383de48258b134d61c962610c6/src/css/main.css |
|||
►
Sign in to add a comment |
|||
Comment 1 by kebalaji@chromium.org
, Mar 29 2018Components: Platform>Apps>Camera
Labels: -Pri-3 Inhouse-HYD-Reported M-67 Pri-2
Summary: Non-Regression: Gallery button is seen enabled in New camera app (was: Non-Regression: Gallery button is seen enabled in New czmera app)