New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 753429 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Smartlock dialog box should appear centered under user pod

Project Member Reported by zalcorn@chromium.org, Aug 8 2017

Issue description

On Sign-In screen for users with smartlock enabled and bluetooth disabled, a dialog appears below and to the left of the user pod. This dialog should appear underneath the user pod and be the same width as the "password incorrect" dialog.
 

Comment 1 by wzang@chromium.org, Aug 8 2017

Status: Started (was: Assigned)
Are we going to file a bug for 'smart lock always shown' even though it's not enabled in settings? Please also suggest a eng owner who's familiar with smart lock back end.
Filed crbug/753442
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 10 2017

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

commit 4b1b41347f0afe0abf358c1c1237ff563132b571
Author: Wenzhao Zang <wzang@chromium.org>
Date: Thu Aug 10 05:22:18 2017

Adjust smart lock bubble position and PIN keyboard ripple size

In response to the second round of UI review:

1) Refactor the codes to reuse the error bubble codes for smart lock
bubble.

2) Adjust PIN keyboard ripple size and color.

Bug:  753429 ,  753426 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I7f48b52b3cf66d150b227e6ff6d2f01f7ca80710
Reviewed-on: https://chromium-review.googlesource.com/607713
Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org>
Reviewed-by: Alexander Alekseev <alemate@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#493261}
[modify] https://crrev.com/4b1b41347f0afe0abf358c1c1237ff563132b571/chrome/browser/resources/chromeos/quick_unlock/md_pin_keyboard.html
[modify] https://crrev.com/4b1b41347f0afe0abf358c1c1237ff563132b571/ui/login/account_picker/md_screen_account_picker.js
[modify] https://crrev.com/4b1b41347f0afe0abf358c1c1237ff563132b571/ui/login/account_picker/md_user_pod_row.js

Comment 4 by wzang@chromium.org, Aug 10 2017

Labels: Merge-Request-61
Status: Fixed (was: Started)
Labels: -Merge-Request-61 Merge-Approved-61
Approving merge to M61 Chrome OS.
Project Member

Comment 6 by bugdroid1@chromium.org, Aug 10 2017

Labels: -merge-approved-61 merge-merged-3163
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3d8444b4839743cb1a510fb102e23b61200d8f73

commit 3d8444b4839743cb1a510fb102e23b61200d8f73
Author: Wenzhao Zang <wzang@chromium.org>
Date: Thu Aug 10 18:53:58 2017

Adjust smart lock bubble position and PIN keyboard ripple size

In response to the second round of UI review:

1) Refactor the codes to reuse the error bubble codes for smart lock
bubble.

2) Adjust PIN keyboard ripple size and color.

TBR=wzang@chromium.org

(cherry picked from commit 4b1b41347f0afe0abf358c1c1237ff563132b571)

Bug:  753429 ,  753426 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I7f48b52b3cf66d150b227e6ff6d2f01f7ca80710
Reviewed-on: https://chromium-review.googlesource.com/607713
Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org>
Reviewed-by: Alexander Alekseev <alemate@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#493261}
Reviewed-on: https://chromium-review.googlesource.com/610935
Reviewed-by: Wenzhao (Colin) Zang <wzang@chromium.org>
Cr-Commit-Position: refs/branch-heads/3163@{#449}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/3d8444b4839743cb1a510fb102e23b61200d8f73/chrome/browser/resources/chromeos/quick_unlock/md_pin_keyboard.html
[modify] https://crrev.com/3d8444b4839743cb1a510fb102e23b61200d8f73/ui/login/account_picker/md_screen_account_picker.js
[modify] https://crrev.com/3d8444b4839743cb1a510fb102e23b61200d8f73/ui/login/account_picker/md_user_pod_row.js

Comment 7 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment