Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tipping Banner needs better keyboard accessibility - Follow up to 6147 #6986

Closed
LaurenWags opened this issue Nov 18, 2019 · 3 comments
Closed
Assignees
Labels
accessibility bug closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. feature/rewards priority/P4 Planned work. We expect to get to it "soon". QA/Test-Plan-Specified QA/Yes release-notes/include

Comments

@LaurenWags
Copy link
Member

Description

Found while testing #6147

When you bring up the tip banner (and have enough BAT) the Send Tip button is in focus as indicated by the outline on the button. However, if you attempt to tab to choose another amount other than the pre-selected one, you exit the tip banner and there is no way to tab back into the tip banner.

Steps to Reproduce

  1. Go to a site and open the tip banner.
  2. If you have enough BAT for the pre-selected amount, the Send Tip button is in focus. If not, tab once so that the add funds link is in focus.
  3. Tab again.
  4. URL bar is now in focus. No matter how many times you select tab, you will never get back into the tip banner.

Actual result:

Selecting tab from the last tab-selectable item on the tip banner (either Send Tip or add funds) causes you to exit the tip banner and go to URL bar. Now, you are unable to tab back into the tip banner. You must use a mouse to get back into the tip banner.

Expected result:

I would expect tab to take me to the first tab selectable item in the banner, I believe this is the X in the circle in the upper right corner.

Reproduces how often:

easily

Brave version (brave://version info)

Brave 1.1.4 Chromium: 78.0.3904.97 (Official Build) beta (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS macOS Version 10.13.6 (Build 17G5019)

Version/Channel Information:

  • Can you reproduce this issue with the current release? n/a
  • Can you reproduce this issue with the beta channel? yes
  • Can you reproduce this issue with the dev channel? yes
  • Can you reproduce this issue with the nightly channel? yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

cc @brave/legacy_qa
cc @mandar-brave @alexwykoff as this relates to tipping efficiency changes

@LaurenWags
Copy link
Member Author

Still occurs as of the below version:


Brave | 1.21.74 Chromium: 89.0.4389.72 (Official Build) (x86_64)
-- | --
Revision | 3f345f156bfd157bd1bea06310e55f3fb2490359-refs/branch-heads/4389@{#1393}
OS | macOS Version 10.15.7 (Build 19H524)

@Miyayes Miyayes removed the priority/P5 Not scheduled. Don't anticipate work on this any time soon. label Jun 29, 2022
@Miyayes
Copy link
Collaborator

Miyayes commented Jul 6, 2022

Note: Tipping banner has probably changed since this issue was originally filed. So, check if this still applies.

@Miyayes Miyayes added the priority/P4 Planned work. We expect to get to it "soon". label Jul 6, 2022
@emerick
Copy link
Contributor

emerick commented Jul 7, 2022

Unable to repro. I think the most recent redesign of the tipping banner ended up addressing this, but please reopen if I'm incorrect.

@emerick emerick closed this as completed Jul 7, 2022
@emerick emerick added this to the 1.43.x - Nightly milestone Jul 7, 2022
@Miyayes Miyayes closed this as not planned Won't fix, can't repro, duplicate, stale Jul 7, 2022
@Miyayes Miyayes added the closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. label Jul 7, 2022
@LaurenWags LaurenWags removed this from the 1.43.x - Beta milestone Aug 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility bug closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. feature/rewards priority/P4 Planned work. We expect to get to it "soon". QA/Test-Plan-Specified QA/Yes release-notes/include
Projects
None yet
Development

No branches or pull requests

5 participants