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

Issue 717322 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 3
Type: Bug

Blocking:
issue 714691



Sign in to add a comment

Allow focusing elements properly in Payment Request

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

Issue description

All the UI elements in the Payment Request dialog that can be interacted with should be focusable.

Moreover, pushing/popping a new sheet should focus the correct element in the topmost sheet.
 
Blocking: 714691
Project Member

Comment 2 by bugdroid1@chromium.org, May 2 2017

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

commit 818ada352c5ec2c3b9da4baf2c9da3472f016fcc
Author: anthonyvd <anthonyvd@chromium.org>
Date: Tue May 02 14:33:56 2017

[Web Payments] Implement proper focusing in Payment Request

BUG= 717322 

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

[modify] https://crrev.com/818ada352c5ec2c3b9da4baf2c9da3472f016fcc/chrome/browser/ui/views/payments/payment_request_dialog_view.cc
[modify] https://crrev.com/818ada352c5ec2c3b9da4baf2c9da3472f016fcc/chrome/browser/ui/views/payments/payment_request_dialog_view.h
[modify] https://crrev.com/818ada352c5ec2c3b9da4baf2c9da3472f016fcc/chrome/browser/ui/views/payments/payment_request_row_view.cc
[modify] https://crrev.com/818ada352c5ec2c3b9da4baf2c9da3472f016fcc/chrome/browser/ui/views/payments/payment_request_row_view.h
[modify] https://crrev.com/818ada352c5ec2c3b9da4baf2c9da3472f016fcc/chrome/browser/ui/views/payments/payment_request_sheet_controller.cc
[modify] https://crrev.com/818ada352c5ec2c3b9da4baf2c9da3472f016fcc/chrome/browser/ui/views/payments/payment_request_sheet_controller.h
[modify] https://crrev.com/818ada352c5ec2c3b9da4baf2c9da3472f016fcc/chrome/browser/ui/views/payments/payment_request_views_util.cc
[modify] https://crrev.com/818ada352c5ec2c3b9da4baf2c9da3472f016fcc/chrome/browser/ui/views/payments/view_stack.cc
[modify] https://crrev.com/818ada352c5ec2c3b9da4baf2c9da3472f016fcc/chrome/browser/ui/views/payments/view_stack.h

Remaining issues with focus:

- Vector icons (pencils, back arrow) need to visually change when focused
- Choose appropriate initial focus target for each sheet
- Scrollable views should move if an off-screen field is focused
- Enter button doesn't activate buttons

Comment 6 by ma...@chromium.org, Jun 2 2017

I think most of the issues you list above are Mac-only? I recognize some of the things in your list as being fixed in #c2 and #c3 on other platforms at least.

Also please Merge-Request-60 for #c4 and open bugs for remaining Mac issues (individual bugs makes merging easier)
Labels: Merge-Request-60
Status: Fixed (was: Assigned)
I haven't tested other platforms for those remaining issues.
Project Member

Comment 8 by sheriffbot@chromium.org, Jun 3 2017

Labels: -Merge-Request-60 Hotlist-Merge-Approved Merge-Approved-60
Your change meets the bar and is auto-approved for M60. Please go ahead and merge the CL to branch 3112 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 9 by bugdroid1@chromium.org, Jun 5 2017

Labels: -merge-approved-60 merge-merged-3112
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ef5ad984bfd147865511727d32f2ff0802e06ae4

commit ef5ad984bfd147865511727d32f2ff0802e06ae4
Author: tmartino <tmartino@chromium.org>
Date: Mon Jun 05 17:09:15 2017

[WebPayments] Making all fields and buttons focusable

BUG= 717322 
TBR=mathp@chromium.org
NOPRESUBMIT=true
NOTRY=true

Review-Url: https://codereview.chromium.org/2922483002
Cr-Original-Commit-Position: refs/heads/master@{#476656}
Review-Url: https://codereview.chromium.org/2926443002
Cr-Commit-Position: refs/branch-heads/3112@{#154}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}

[modify] https://crrev.com/ef5ad984bfd147865511727d32f2ff0802e06ae4/chrome/browser/ui/views/payments/credit_card_editor_view_controller.cc
[modify] https://crrev.com/ef5ad984bfd147865511727d32f2ff0802e06ae4/chrome/browser/ui/views/payments/payment_method_view_controller.cc
[modify] https://crrev.com/ef5ad984bfd147865511727d32f2ff0802e06ae4/chrome/browser/ui/views/payments/payment_request_sheet_controller.cc
[modify] https://crrev.com/ef5ad984bfd147865511727d32f2ff0802e06ae4/chrome/browser/ui/views/payments/payment_sheet_view_controller.cc
[modify] https://crrev.com/ef5ad984bfd147865511727d32f2ff0802e06ae4/chrome/browser/ui/views/payments/profile_list_view_controller.cc
[modify] https://crrev.com/ef5ad984bfd147865511727d32f2ff0802e06ae4/chrome/browser/ui/views/payments/validating_combobox.cc

Cc: kavvaru@chromium.org
Labels: Needs-Feedback
Could some one please provide us steps to verify if this can be verified manually.

Thanks,
On Linux, Mac, and Windows:

1) Open https://rsolomakhin.github.io/pr/expl/
2) Click "Buy" on the page.
3) Verify: pressing Tab on the keyboard moves focus through all clickable views.

4) Open the order summary page.
5) Verify: pressing Tab on the keyboard moves focus through all clickable views.

6) Open the shipping address selector.
7) Verify: pressing Tab on the keyboard moves focus through all clickable views.

8) Open the address editor.
9) Verify: pressing Tab on the keyboard moves focus through all clickable views.

9) Open the card selector.
10) Verify: pressing Tab on the keyboard moves focus through all clickable views.

11) Open the card editor.
12) Verify: pressing Tab on the keyboard moves focus through all clickable views.

13) Open https://rsolomakhin.github.io/pr/contact/
14) Click "Buy" on the page.
15) Verify: pressing Tab on the keyboard moves focus through all clickable views.

16) Open the contact info selector.
17) Verify: pressing Tab on the keyboard moves focus through all clickable views.

18) Open the contact info editor.
19) Verify: pressing Tab on the keyboard moves focus through all clickable views.

And on ChromeOS.
Components: -UI>Browser>Autofill>Payments UI>Browser>Payments

Sign in to add a comment