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

Feature/pagination focus #11565

Merged
merged 4 commits into from
Aug 11, 2022
Merged

Feature/pagination focus #11565

merged 4 commits into from
Aug 11, 2022

Conversation

gcamacho079
Copy link
Contributor

Description

When a user changes the element index page, focus is moved to the first focusable element (Resolves DEV-381)

@gcamacho079 gcamacho079 added the accessibility 👤 features related to accessibility label Jul 6, 2022
@linear
Copy link

linear bot commented Jul 6, 2022

DEV-381 Focus does not move to first element after activating a pagination button

Focus is not managed after activating page control, meaning screen reader users are not alerted to a shift in page context

Keyboard focus should move to first focusable item in element index table when pagination is activated

Resolves CMS-070

@brandonkelly brandonkelly changed the base branch from 4.2 to 4.3 July 27, 2022 22:49
# Conflicts:
#	src/web/assets/admintable/dist/css/app.css
#	src/web/assets/admintable/dist/js/app.js
#	src/web/assets/admintable/dist/js/app.js.map
#	src/web/assets/admintable/dist/manifest.json
#	src/web/assets/cp/dist/cp.js
#	src/web/assets/cp/dist/cp.js.map
#	src/web/assets/garnish/dist/garnish.js
#	src/web/assets/garnish/dist/garnish.js.map
#	src/web/assets/pluginstore/dist/css/app.css
#	src/web/assets/pluginstore/dist/css/app.css.map
[ci skip]
@brandonkelly brandonkelly merged commit 066e688 into 4.3 Aug 11, 2022
@brandonkelly brandonkelly deleted the feature/pagination-focus branch August 11, 2022 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility 👤 features related to accessibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants