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

b-pagination-nav ARIA default value #4160

Closed
kieran-github opened this issue Sep 27, 2019 · 3 comments · Fixed by #4810
Closed

b-pagination-nav ARIA default value #4160

kieran-github opened this issue Sep 27, 2019 · 3 comments · Fixed by #4810

Comments

@kieran-github
Copy link

kieran-github commented Sep 27, 2019

Hello,

when using the b-component with ARIA attribute label-page it always appends the page number to the string in that attribute as per https://bootstrap-vue.js.org/docs/components/pagination-nav. For example if I wanted screenreader to read the software as Page {pagenumber} selected. It would then generate an aria attribute that the screenreader software would then read as Page {pagenumber} selected pagenumber. Is there anyway to remove the default values so I dynamically use the ARIA labels the v-model?

Thanks.

@kieran-github
Copy link
Author

kieran-github commented Sep 27, 2019

This may be a seperate issue, but is there an attribute that allows me to add if the screenreader software has navigated to the pagination button has not selected it? For example if the page 6 button is focused but not yet selected and I can get the screenreader software to say "Page 6, not selected" but when it is selected I can get it to say "Page 6 selected". Any help is greatly appreciated, thanks.

@tmorehouse
Copy link
Member

tmorehouse commented Sep 27, 2019

Currently no, but the current page button does have the aria-checked="true" attribute set on it for screen readers.

@tmorehouse
Copy link
Member

tmorehouse commented Sep 27, 2019

In a future release we will be re-visiting ARIA for pagination-nav (currently it uses the same ARIA markup as pagination... but needs to be tweaked a bit)

@tmorehouse tmorehouse added this to To do in 2.1.0 via automation Sep 27, 2019
@tmorehouse tmorehouse removed this from To do in 2.1.0 Oct 30, 2019
@tmorehouse tmorehouse added this to Features in Roadmap Oct 30, 2019
@tmorehouse tmorehouse changed the title b-component ARIA default value b-pagination-nav ARIA default value Oct 30, 2019
@jacobmllr95 jacobmllr95 removed this from Features in Roadmap Nov 13, 2019
@jacobmllr95 jacobmllr95 added this to To do in 2.2.0 via automation Nov 13, 2019
@jacobmllr95 jacobmllr95 removed this from To do in 2.2.0 Nov 20, 2019
@jacobmllr95 jacobmllr95 added this to To do in 2.3.0 via automation Nov 20, 2019
@tmorehouse tmorehouse removed this from To do in 2.3.0 Jan 24, 2020
tmorehouse added a commit that referenced this issue Feb 23, 2020
: #4811, #4160) (#4810)

Co-authored-by: Jacob Müller <jacob.mueller.elz@gmail.com>
@tmorehouse tmorehouse added this to Completed in Roadmap Mar 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Roadmap
Completed
3 participants