When in Touchview password fields should show what was typed |
|||||
Issue descriptionWhen using a virtual keyboard it is easy to mistype a character and not know it. Password fields should briefly show the most recent pressed key before turning it to a *. Behavior should match Android on this. See internal 30613396 for more context.
,
Aug 5 2016
Sounds good. I assume we'll need some sort of broadcast mechanism to update the state of any active password fields as the device shifts in/out of touchview?
,
Aug 5 2016
Yes. We need some infra code change to enable/disable password echo dynamically. Currently, it is initialized during renderer and Textfield initialization.
,
Aug 25 2016
,
Aug 26 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a34e954cec188f2cc09270ef801dd35ecd1072ac commit a34e954cec188f2cc09270ef801dd35ecd1072ac Author: xiyuan <xiyuan@chromium.org> Date: Fri Aug 26 21:29:41 2016 views: Textfield gets password reveal duration from ViewsDelegate Remove password_reveal_duration_ member in Textfield and always gets it from ViewsDelegate. BUG=634472 TEST=TextfieldTest.SetPasswordRevealDuration Review-Url: https://codereview.chromium.org/2286433002 Cr-Commit-Position: refs/heads/master@{#414811} [modify] https://crrev.com/a34e954cec188f2cc09270ef801dd35ecd1072ac/ui/views/controls/textfield/textfield.cc [modify] https://crrev.com/a34e954cec188f2cc09270ef801dd35ecd1072ac/ui/views/controls/textfield/textfield.h [modify] https://crrev.com/a34e954cec188f2cc09270ef801dd35ecd1072ac/ui/views/views_delegate.cc [modify] https://crrev.com/a34e954cec188f2cc09270ef801dd35ecd1072ac/ui/views/views_delegate.h
,
Aug 26 2016
kuscher@, sky raised a point during the review that whether we want to enable the feature when user is using touch (e.g. the last pointer event is touch) and maybe even do it for desktop. UX inputs/thoughts on this are appreciated.
,
Feb 22 2017
I'd prefer to only do it for touch (Virtual Keyboard) as we might get into a situation where a teacher for example logs into a website while presenting only to have the whole class memorize his/her password. Kuscher, thoughts?
,
Jun 8 2017
The teacher/presentation/in-a-public-place story is pretty significant. I would strongly prefer a system that allowed the user to actively control the password-reveal. Many password ui's show an eyeball icon to show/hide the password. For system-controlled interfaces (like system login) we could use a similar mechanism. This wouldn't work for web content though. Perhaps we could move the show/hide control into the vk? This would provide a consistent and universal control for showing the password.
,
Jan 10 2018
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by xiy...@chromium.org
, Aug 5 2016