New issue
Advanced search Search tips

Issue 717581 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Screen lock configuration requires password, but password field not in focus

Project Member Reported by norvez@chromium.org, May 2 2017

Issue description

Chrome Version: 30.0.3086.0 (ToT)
Chrome OS Version: 9514.0.0
Chrome OS Platform: eve

Steps To Reproduce:
(1) Start settings
(2) Click on People->Screen lock

Expected Result:
Pop up window requesting password has focus on the password field

Actual Result:
It doesn't have focus the first time, and never has focus when using the touchscreen

How frequently does this problem reproduce?
Always

What is the impact to the user, and is there a workaround? If so, what is
it?
Suboptimal UX, no workaround.


Note that it only seems to happen the first time, i.e. after a fresh launch of the Settings window. If it's not the first time, then the password is in focus. Unless you're using the touchscreen to browse the Settings faster, in that case it's never in focus.


 
Labels: -Proj-MaterialDesign-CrOS Proj-MaterialDesign-NativeUI
Labels: -Proj-MaterialDesign-NativeUI Proj-MaterialDesign-WebUI
Cc: tbuck...@chromium.org steve...@chromium.org
Labels: Hotlist-MD-Settings-PeopleCrOS M-60
Owner: sammiequon@chromium.org
Status: Assigned (was: Unconfirmed)
This should be handled in https://codereview.chromium.org/2841313002/. Will test the touch screen part.
Project Member

Comment 5 by bugdroid1@chromium.org, May 17 2017

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

commit a54497f5b8d234513196f991d3748afc6755a2e7
Author: sammiequon <sammiequon@chromium.org>
Date: Wed May 17 20:19:05 2017

md settings: Give focus to password prompt on tap.

Previously the password prompt would open on navigation to lock screen page but not focus, if the user used the touch screen and not the mouse. Also fixed a bug with easy unlock turn off dialog only being able to be used once.

TEST=manual
BUG= 717581 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2889803002
Cr-Commit-Position: refs/heads/master@{#472546}

[modify] https://crrev.com/a54497f5b8d234513196f991d3748afc6755a2e7/chrome/browser/resources/settings/people_page/lock_screen.html
[modify] https://crrev.com/a54497f5b8d234513196f991d3748afc6755a2e7/chrome/browser/resources/settings/people_page/people_page.js

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)
9574.0.0, 60.0.3105.0

Sign in to add a comment