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

Facet suggest: filtering facets, choosing a value, then attempting to filter again does not work #4731

Closed
sandbergja opened this issue Feb 5, 2025 · 3 comments

Comments

@sandbergja
Copy link
Member

Thanks to @caroldh for identifying and reporting this issue!

Expected behavior

You can filter facets, choose a value, then filter again.

Actual behavior

You can only do it once, and then the UI breaks :-(

Steps to replicate

  1. Search catalog.princeton.edu for economic policy
  2. In the Language facet, click "more"
  3. Filter the languages to "russ"
  4. Select Russian
  5. Note that your URL contains only_values=true and query_fragment=russ
  6. In the Language facet, click "more" again
  7. Note that you don't get a filter, or really much content in the modal at all!
    The modal contains only the word Russian, an X, and the number 10,935

Impact of this bug

Makes it hard to use facets to limit your search

Implementation notes, if any

@maxkadel
Copy link
Contributor

maxkadel commented Feb 6, 2025

Open PR against Blacklight main to fix this bug. Once that is approved and merged, we will then need to port the fix back to the release-8.x branch, do a Blacklight 8 release, and update Orangelight to use the newest version of Blacklight 8

@maxkadel
Copy link
Contributor

maxkadel commented Feb 6, 2025

PR against Blacklight main has been merged, now have and open PR against release-8.x branch

@maxkadel
Copy link
Contributor

maxkadel commented Feb 6, 2025

Closed by #4736

@maxkadel maxkadel closed this as completed Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants