[Regression] The gradient mask in the user scroll list needs UI polish |
|||||
Issue descriptionNot urgent, since it's a pure UI improvement and only applies to >6 users. There should be a gradient mask at the top and the bottom of the user scroll list. Currently the mask does exist in the views-based login, but: 1) its color should change together with wallpaper changes, 2) its height should be higher according to spec https://gallery.googleplex.com/projects/MCHbtQVoQ2HCZdwfnbqy2eSi/files/MCEk7MHTx6J1uiGu0WFxf6l3wMSJbxAVNpw)
,
May 11 2018
,
May 31 2018
,
May 31 2018
,
Jun 6 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d0bf4841162789baf0de248a3997ec072e527ee6 commit d0bf4841162789baf0de248a3997ec072e527ee6 Author: Quan Nguyen <qnnguyen@chromium.org> Date: Wed Jun 06 17:48:00 2018 Upgrade ScrollableUsersListView GradientParams on wallpaper change Bug: 839610 Change-Id: I4a4ee5e8c80a57a32f8368fe3716803c82148183 Reviewed-on: https://chromium-review.googlesource.com/1086068 Reviewed-by: Jacob Dufault <jdufault@chromium.org> Commit-Queue: Quan Nguyen <qnnguyen@chromium.org> Cr-Commit-Position: refs/heads/master@{#564957} [modify] https://crrev.com/d0bf4841162789baf0de248a3997ec072e527ee6/ash/login/ui/scrollable_users_list_view.cc [modify] https://crrev.com/d0bf4841162789baf0de248a3997ec072e527ee6/ash/login/ui/scrollable_users_list_view.h
,
Jul 10
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by wzang@chromium.org
, May 4 2018Labels: -Pri-3 ReleaseBlock-Stable Pri-1
Owner: jdufault@chromium.org
Status: Assigned (was: Available)