-
Notifications
You must be signed in to change notification settings - Fork 255
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
Add facet prefix filtering when paginating within a facet #1262
Conversation
|
||
.pagination-xs { | ||
li { | ||
a, span { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Selector should have depth of applicability no greater than 2, but was 3
Each selector in a comma sequence should be on its own line
0822295
to
372d17d
Compare
# set :index_pagination to true if you want the pop-up window for large facets to have alphabetical index navigation | ||
# (useful when user clicks "more" on a large facet and wants to navigate alphabetically across a large set of results) | ||
# if you set :index_pagination to true, you MUST set :index_range to an array of characters that will be used to create the navigation (note: It is case sensitive when searching values) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment says to set :index_pagination => true
but in the example :index_pagination
is not set.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could case sensitivity be an option here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for catching that. I've tried to revise the comment to reflect the new configuration.
Solr's facet.prefix
option is case-sensitive. It looks like, in Solr 5.1, they landed a new facet.contains
parameter that may do case-insensitive filtering, so maybe there's hope for this in the future..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
…x navigation as a bootstrap pagination widget
372d17d
to
4bd3f56
Compare
…r-index Add facet prefix filtering when paginating within a facet
Fixes #1251. I took @peetucket's commit and tried to push logic down into the models and use some more bootstrap-y styling:
There's significant work left to make it a general purpose tool (say, disabling the prefixes that have 0 hits), but this may be good enough for some targeted use or in a very large index.