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

Allow bookmarks page to be sorted by library #4306

Closed
1 of 3 tasks
sandbergja opened this issue Sep 11, 2024 · 2 comments · Fixed by #4439
Closed
1 of 3 tasks

Allow bookmarks page to be sorted by library #4306

sandbergja opened this issue Sep 11, 2024 · 2 comments · Fixed by #4439
Assignees
Labels
feature Updates on existing feature or a new feature request PO (high value) PO reviewed and high value

Comments

@sandbergja
Copy link
Member

sandbergja commented Sep 11, 2024

User story

From a libanswers user:

[It would be amazing] to be able to sort bookmarks by library

Acceptance criteria

  • The bookmarks page includes an option to sort by library (as well as the existing sort options such as year, author, and title)
  • I can successfully use this feature with Voice Over in Safari.
  • I have checked this feature using one of the following browser extensions and confirmed that it does not add any level A or level AA issues:
    • axe dev tools
    • Lighthouse
    • Wave

Notes

Related ticket #4305

We were thinking about removing the Relevance option as part of this ticket. However, it is difficult to remove this given current Blacklight configuration options, and it wasn't necessary to address this user's need. So we removed it as a success criteria. It may be useful to do a Blacklight feature request in the future to allow the default option to accept a lambda as if or unless condition (as other config options do).

@sandbergja sandbergja added feature Updates on existing feature or a new feature request PO (review) labels Sep 11, 2024
@sandbergja
Copy link
Member Author

@ellen-aa @caroldh what do you think?

@ellen-aa
Copy link

High priority to me - would be great to do at the same time as 4305. @caroldh, what do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Updates on existing feature or a new feature request PO (high value) PO reviewed and high value
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants