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

geosolutions-it#6913: grid cards keyboard navigation #10150

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fkellner
Copy link
Contributor

@fkellner fkellner commented Apr 3, 2024

Description

Makes grid cards keyboard navigable and accessible again (with tab + enter,
you can reach a map now).

Does not change how anything looks for non-keyboard users.

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix

Issue

What is the current behavior?

#6913 Reaching a map only using the keyboard is impossible.
Power users and people with motoric impairments often prefer
to navigate a website using the keyboard rather than the mouse.

What is the new behavior?

The grid cards, which represent buttons to maps, now can be focused
via keyboard and pressed via Enter button.

Breaking change

  • No

Other useful information

Tests only include the "clicking with the Enter button" behaviour, since the testing framework does not seem to
support side effects like focusing elements with the tab button.

Makes grid cards keyboard navigable and accessible again

On Behalf of DB Systel
@tdipisa tdipisa linked an issue Apr 4, 2024 that may be closed by this pull request
@tdipisa tdipisa added this to the 2024.01.01 milestone Apr 4, 2024
@tdipisa tdipisa added enhancement BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch labels Apr 4, 2024
@tdipisa
Copy link
Member

tdipisa commented Apr 4, 2024

@fkellner thank you so much for your contribution. We will review as soon as possible.
@allyoucanmap, let's evaluate a possible inclusion in 2024.01.01 (waiting for the review).

@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Apr 4, 2024
@tdipisa tdipisa removed this from the 2024.01.01 milestone Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Illogical tab order
2 participants