Fingerprint enrollment tweaks (from DF feedback) |
||||
Issue descriptionThree changes for enrollment flow (in OOBE and in-session) based on Dogfood feedback: 1. Touching the FP sensor on the first screen should begin enrollment (user shouldn't have to click next), applies to this screen: https://docs.google.com/presentation/d/1U_ObeMIzR5RzjbOaLXbnpBabL8nBlmMUbpdCFoU6-4E/edit#slide=id.g3b73ef5920_0_492 2. Remove "index" from first string, any finger will work. 3. Replace "on the top left" with "on the edge" as some users will set up in portrait mode. I'm going to mark this as need for M71 as we've gotten consistent feedback about these things and hopefully the changes aren't huge.
,
Sep 25
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3d08da2cb0798f30f71002db238b77a00c726bec commit 3d08da2cb0798f30f71002db238b77a00c726bec Author: Sarah Hu <xiaoyinh@chromium.org> Date: Tue Sep 25 18:59:15 2018 Update fingerprint enrollment strings Bug: 888704 Change-Id: I9167e623424c48de14dd1b5eeddf4ffd966d7d12 Reviewed-on: https://chromium-review.googlesource.com/1244077 Reviewed-by: Wenzhao (Colin) Zang <wzang@chromium.org> Commit-Queue: Xiaoyin Hu <xiaoyinh@chromium.org> Cr-Commit-Position: refs/heads/master@{#594034} [modify] https://crrev.com/3d08da2cb0798f30f71002db238b77a00c726bec/chrome/app/chromeos_strings.grdp [modify] https://crrev.com/3d08da2cb0798f30f71002db238b77a00c726bec/chrome/app/chromeos_strings_grdp/IDS_OOBE_FINGERPINT_SETUP_SCREEN_INSTRUCTION_MOVE_FINGER.png.sha1 [modify] https://crrev.com/3d08da2cb0798f30f71002db238b77a00c726bec/chrome/app/chromeos_strings_grdp/IDS_OOBE_FINGERPINT_SETUP_SCREEN_SENSOR_DESCRIPTION.png.sha1
,
Sep 25
,
Sep 25
Sarah, is (1) the existing behavior for OOBE enrollment too? Not working for me there today.
,
Sep 25
,
Sep 25
Discussed with zalcorn@ offline: For (1): If user touches the FP sensor on the first screen[1] we should just progress to enrollment screen[2] [1]:https://docs.google.com/presentation/d/1U_ObeMIzR5RzjbOaLXbnpBabL8nBlmMUbpdCFoU6-4E/edit#slide=id.g3b73ef5920_0_498 [2]: https://docs.google.com/presentation/d/1U_ObeMIzR5RzjbOaLXbnpBabL8nBlmMUbpdCFoU6-4E/edit#slide=id.g3b73ef5920_0_504
,
Sep 26
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3e746cda389ed52435c288b5c026aae560b7b07a commit 3e746cda389ed52435c288b5c026aae560b7b07a Author: Sarah Hu <xiaoyinh@chromium.org> Date: Wed Sep 26 21:32:17 2018 cros: Progress to fingerprint enrollment when user touches FP sensor Bug: 888704 Change-Id: I3d6ccd1d4d4d19377db2d0698ac403ff79cd40e6 Reviewed-on: https://chromium-review.googlesource.com/1244124 Commit-Queue: Xiaoyin Hu <xiaoyinh@chromium.org> Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#594480} [modify] https://crrev.com/3e746cda389ed52435c288b5c026aae560b7b07a/chrome/browser/resources/chromeos/login/fingerprint_setup.js
,
Sep 26
|
||||
►
Sign in to add a comment |
||||
Comment 1 by xiaoyinh@chromium.org
, Sep 25