You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Oct 30, 2024. It is now read-only.
This is more of a feature request than a bug. Have you thought about adding some accessibility elements to the pagination plugin?
I've started doing some myself adding aria-label="Go to Page ${pageNumber}" it would be nice to add aria-label="Current Page & aria-current="true" on the active element. Also aria-disabled="true" on the disabled elements (next / prev). Also adding aria-disabled="true" to the .mixitup-page-list when there are no results. If you can help at all with the above, I'd be more than happy to paste in my final code here for anyone else looking to add accessibility to the pagination.
Thanks
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Hi Patrick,
This is more of a feature request than a bug. Have you thought about adding some accessibility elements to the pagination plugin?
I've started doing some myself adding
aria-label="Go to Page ${pageNumber}"
it would be nice to addaria-label="Current Page
&aria-current="true"
on the active element. Alsoaria-disabled="true"
on the disabled elements (next / prev). Also addingaria-disabled="true"
to the .mixitup-page-list when there are no results. If you can help at all with the above, I'd be more than happy to paste in my final code here for anyone else looking to add accessibility to the pagination.Thanks
The text was updated successfully, but these errors were encountered: