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

Update account menu for keyboard accessibility #2587

Merged
merged 3 commits into from
Dec 4, 2023

Conversation

ystros
Copy link
Contributor

@ystros ystros commented Nov 3, 2023

Previously, the dropdown account menu in the top right could only be used by clicking on it using a mouse. People who use screen readers, such as those who are blind or have low-vision, require the ability to navigate and interact using keyboard-only. These users cannot use a mouse for user interaction.

Now, the account menu can be interacted with using keyboard. It uses the aria-expanded and aria-controls to provide better feedback to screen readers about whether the menu is opened or closed.

Previously, the dropdown account menu in the top right could only be
used by clicking on it using a mouse. People who use screen readers,
such as those who are blind or have low-vision, require the ability to
navigate and interact using keyboard-only. These users cannot use a
mouse for user interaction.

Now, the account menu can be interacted with using keyboard. It uses the
`aria-expanded` and `aria-controls` to provide better feedback to screen
readers about whether the menu is opened or closed.
@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/186402792

The labels on this github issue will be updated when the story is started.

@strehle
Copy link
Member

strehle commented Nov 6, 2023

@Tallicia can you retrigger concourse ?

@Tallicia
Copy link
Contributor

Tallicia commented Nov 9, 2023

I can't find the test. I am investigating and hope I don't need to add a commit as a way to re-trigger this. hmmm.

@Tallicia
Copy link
Contributor

Tallicia commented Nov 9, 2023

It looks like CI does not play nicely with "external" repos - https://hush-house.pivotal.io/teams/cf-uaa/pipelines/pull-requests/jobs/uaa-standalone-tests/builds/2133.2

@strehle strehle added this to the 76.26.0 milestone Nov 13, 2023
@strehle
Copy link
Member

strehle commented Nov 13, 2023

new UI
image

Copy link
Member

@strehle strehle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hsinn0 from my side UI change is OK, please check from vmware side

@strehle strehle requested a review from a team November 16, 2023 17:21
@strehle strehle added the in_review The PR is currently in review label Nov 16, 2023
Copy link
Contributor

@hsinn0 hsinn0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ystros, the change looks good. However, I have one question. In Chrome, I can expand the button with "enter" key or "space" key then select an option via "tab" key. However, in Firefox, I cannot do that. In Firefox, I still have to use mouse to select an option. Are you aware of that? And would it be possible to make the selection also work in Firefox?

bruce-ricard

This comment was marked as off-topic.

@cloudfoundry cloudfoundry deleted a comment from bruce-ricard Nov 17, 2023
@strehle strehle modified the milestones: 76.26.0, icebox Nov 22, 2023
@strehle strehle added the clarification needed The issue is not accepted but we need clarification label Nov 23, 2023
@strehle
Copy link
Member

strehle commented Nov 23, 2023

FYI @ystros we will include your change in uaa release as soon as open questions answered #2587 (review)

For now only the other PR from you is included in the recent release

@strehle strehle removed this from the icebox milestone Nov 23, 2023
@strehle strehle added this to the 76.27.0 milestone Nov 23, 2023
@ystros
Copy link
Contributor Author

ystros commented Dec 1, 2023

Hi @strehle @hsinn0! Apologies for the late reply.

I'm not seeing the problem you mention with my copy of latest Firefox (120.0.1) on Mac. I can tab to the account menu, and pressing either space or enter is expanding the menu as expected and seen in other browsers. Once expanded, I can press tab again to select the "Account Settings" or "Sign Out" options, and pressing enter on those opens the appropriate link.

I know certain browser have different keyboard navigation settings you need to enable (e.g. Safari does not allow tab navigation to buttons and links by default). Maybe I have some additional setting in Firefox enabled? This is what I see when I search Firefox settings for "keyboard"
Screenshot 2023-12-01 at 11 42 26 AM

@hsinn0
Copy link
Contributor

hsinn0 commented Dec 4, 2023

I know certain browser have different keyboard navigation settings you need to enable (e.g. Safari does not allow tab navigation to buttons and links by default). Maybe I have some additional setting in Firefox enabled? This is what I see when I search Firefox settings for "keyboard"

"Use the tab key ..." option was unchecked in my Mac FF settings. I do not know whether this is checked by default or not. In any case, the change is good.

@hsinn0 hsinn0 merged commit e1e13a6 into cloudfoundry:develop Dec 4, 2023
20 checks passed
@ystros ystros deleted the accessibility-dropdown-nav branch December 4, 2023 19:33
@strehle strehle removed in_review The PR is currently in review clarification needed The issue is not accepted but we need clarification labels Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

7 participants