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

Issue 670986 link

Starred by 3 users

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Feature

Blocking:
issue 714618
issue 770046



Sign in to add a comment

Material design password manager usability issues

Reported by nie.mam....@gmail.com, Dec 4 2016

Issue description

UserAgent: Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.14 Safari/537.36

Steps to reproduce the problem:
Navigate to the password manager and try to use it right away.

What is the expected behavior?

What went wrong?
The newly introduced material design password manager (replacing the old chrome://settings/passwords dialog) has some annoying usability issues and/or design changes.

On its load, the focus is set to the top search field (for settings panel search field). It used to be set to the password manager's search field, now it takes 3 additional TABs or 1 additional click to search through the password database.

The "show password" functionality has been moved to the menu for each entry in the manager. This introduces addition 1-2 clicks and 1 additional dialog to navigate through to display saved password. The design also prevents the user from displaying more than one password at a time.

Both this changes affect the efficiency of using password manager and interfere with user's habits.

Did this work before? Yes Before the introduction of the material design password manager

Chrome version: 56.0.2924.14  Channel: dev
OS Version: 6.3
Flash Version:
 

Comment 1 by ajha@chromium.org, Dec 5 2016

Components: -UI UI>Browser>Passwords
Labels: M-56 Proj-MaterialDesign-WebUI
Cc: rbasuvula@chromium.org
Labels: -Type-Bug-Regression Type-Feature
Status: Untriaged (was: Unconfirmed)
Considering this as a feature request and making the status to Untriaged so that the issue would get addressed.

Some one from Material Design team, please look in to it and update.

Thank you.

Comment 3 by vabr@chromium.org, Dec 8 2016

Cc: dbeam@chromium.org
Status: Unconfirmed (was: Untriaged)
I'll let the MD team comment on whether having the focus in the main search field as opposed to the passwords search team is intended or not.

The effects of moving "show password" into the separate dialogue can differ based on use-case. Some users might appreciate the increased space for viewing the password, for example. Also, viewing the password is a fallback which should not be needed too often (and the team is working on improving the success rate at filling into forms). As long as the controls are keyboard-accessible and have proper accessibility labelling, the increased number of typing needed to reach "show passwords" is not a concern.

Comment 4 by vabr@chromium.org, Dec 8 2016

Labels: Hotlist-Polish
I get that the new settings design is pretty and spacious and all (although I guess I won't understand why it needs to be confined to a 640px width even at 1920x1080 display resolution, but that's just a digression), but please don't trade actual functionality for that.

What do you think are the common use cases for a password manager, if not, well, managing passwords?

For viewing and filtering saved passwords, the material design is worse, because of the search field focus issue.
For viewing specific passwords, the material design is worse, because the "show" option is hidden in a dialog, in a menu.
For deleting specific passwords, the material design is worse, because the "delete" option is hidden in a menu (although that one's not an issue, as it's a destructive operation, and it's irrelevant to the "show details" dialog).

What other use cases can you see, that are handled much better with the introduction of a separate menu item AND a separate dialog?

The "view details" dialog does not show any additional information to the information that's already been shown on the password list itself, yet introduces additional screen to navigate through for one specific operation - of viewing a password. Functionally, it adds nothing in return for the convenience it's taken away, no matter how rarely it's needed.
The dialog would make some sense if it allowed, for example, to edit existing entries, but as yet, it does not.

Comment 6 by dbeam@chromium.org, Dec 8 2016

Cc: ainslie@chromium.org tbuck...@chromium.org
Owner: bettes@chromium.org
Status: Assigned (was: Unconfirmed)

Comment 7 by kolos@chromium.org, Apr 24 2017

Blocking: 714618

Comment 8 by battre@chromium.org, Jun 30 2017

 Issue 738356  has been merged into this issue.

Comment 9 by dbeam@chromium.org, Jun 30 2017

Cc: hcarmona@chromium.org
Cc: bettes@chromium.org
Labels: OS-Chrome OS-Linux OS-Mac
Owner: jdoerrie@chromium.org
Status: Started (was: Assigned)
Screenshot for Code Review
Code_Review.png
17.7 KB View Download
Equal_Width_Columns.png
34.9 KB View Download
System_Font_Password.png
22.8 KB View Download
Inherit_Font_Password.png
22.8 KB View Download
Project Member

Comment 13 by bugdroid1@chromium.org, Aug 2 2017

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

commit 394e764abad5181aeba22300061840f8344a45f4
Author: jdoerrie <jdoerrie@chromium.org>
Date: Wed Aug 02 18:52:04 2017

Add Show Password Button To Settings Page

This change adds a show password button directly in chrome://settings/passwords
to reduce the number of clicks necessary to view a given password.

Bug: 670986
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: If54c92d2ee6c2e64f90a5280e59a4c59af182740
Reviewed-on: https://chromium-review.googlesource.com/575143
Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Cr-Commit-Position: refs/heads/master@{#491454}
[modify] https://crrev.com/394e764abad5181aeba22300061840f8344a45f4/chrome/browser/resources/settings/passwords_and_forms_page/compiled_resources2.gyp
[modify] https://crrev.com/394e764abad5181aeba22300061840f8344a45f4/chrome/browser/resources/settings/passwords_and_forms_page/password_edit_dialog.html
[modify] https://crrev.com/394e764abad5181aeba22300061840f8344a45f4/chrome/browser/resources/settings/passwords_and_forms_page/password_edit_dialog.js
[modify] https://crrev.com/394e764abad5181aeba22300061840f8344a45f4/chrome/browser/resources/settings/passwords_and_forms_page/password_list_item.html
[modify] https://crrev.com/394e764abad5181aeba22300061840f8344a45f4/chrome/browser/resources/settings/passwords_and_forms_page/password_list_item.js
[modify] https://crrev.com/394e764abad5181aeba22300061840f8344a45f4/chrome/browser/resources/settings/passwords_and_forms_page/passwords_section.html
[modify] https://crrev.com/394e764abad5181aeba22300061840f8344a45f4/chrome/browser/resources/settings/passwords_and_forms_page/passwords_section.js
[modify] https://crrev.com/394e764abad5181aeba22300061840f8344a45f4/chrome/browser/resources/settings/passwords_and_forms_page/passwords_shared_css.html
[add] https://crrev.com/394e764abad5181aeba22300061840f8344a45f4/chrome/browser/resources/settings/passwords_and_forms_page/show_password_behavior.html
[add] https://crrev.com/394e764abad5181aeba22300061840f8344a45f4/chrome/browser/resources/settings/passwords_and_forms_page/show_password_behavior.js
[modify] https://crrev.com/394e764abad5181aeba22300061840f8344a45f4/chrome/browser/resources/settings/settings_resources.grd
[modify] https://crrev.com/394e764abad5181aeba22300061840f8344a45f4/chrome/test/data/webui/settings/settings_passwords_section_browsertest.js

Implementation LGTM. 

Comment 15 by kolos@chromium.org, Sep 29 2017

Blocking: 770046

Sign in to add a comment