Comment 13 for bug 1970476

Revision history for this message
Galen Charlton (gmc) wrote :

I've reviewed this and note that when library groups were introduced that the separate scope selector on the advanced search page was intentional:

https://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=9ddce92adc9

However, the commit that added Bootstrap OPAC support did not set the name of the scope selector correctly, so it currently does nothing.

Here's an example of how to restore some the functionality of the scope selector:

https://gist.github.com/gmcharlt/d70805a6283294d013c3569f5205a15d

This works in the sense that choosing a depth, a library group, or a location group from that selector applies the selected limit, but there remain rough edges. For example, if you navigate back to the advanced search page, your chosen search scope does not remain selected.

I'm neutral on the question of whether to fully resurrect the split location + scope selectors for the advanced search for or just have both basic and advanced search use the unified selector, but I want to explicitly raise the question for final discussion before pushing anything.