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

Bottom of search results paging #1380

Closed
dbranchini opened this issue Sep 5, 2024 · 2 comments
Closed

Bottom of search results paging #1380

dbranchini opened this issue Sep 5, 2024 · 2 comments
Assignees

Comments

@dbranchini
Copy link

dbranchini commented Sep 5, 2024

Paging at the bottom of the search results isn't appearing as links. See screenshot.

Currently:

As designed:

Also note, we're getting an accessibility error on the use of aria-current="page" due to double quotes. I'm also concerned that we might be mixing and matching two different accessibility approaches creating duplication of information. Note the use of both aria-current="page" and aria-label="Current Page, Page 4". See screenshot.

The original references to paging and accessibility are in this ticket - #1170.

@dnoneill
Copy link
Contributor

dnoneill commented Sep 5, 2024

closed by
#1388
projectblacklight/blacklight#3316

@dnoneill
Copy link
Contributor

dnoneill commented Sep 6, 2024

Fixed with #1388 and projectblacklight/blacklight#3316

@dnoneill dnoneill closed this as completed Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

2 participants