The functionality of OpacAddMastheadLibraryPulldown would be helpful to have on the staff interface. Often staff need to limit their search a specific library, without having to go to the advanced search. This would be an excellent addition, and by default, should limit to the library you are logged in to, with the option via the dropdown to search all libraries.
Created attachment 130478 [details] [review] Bug 30077: Add system preference IntranetAddMastheadLibraryPulldown
I setting this to ASSIGNED and not NSO because I have a couple questions: 1. If by default it sets the limit to the logged in branch and you do a search at a different branch, should the limiter then be set to the branch you just searched or still your logged in branch? 2. Does these need to account for IndependentBranches?
> 1. If by default it sets the limit to the logged in branch and you do a > search at a different branch, should the limiter then be set to the branch > you just searched or still your logged in branch? My expectation would be to set my search back to the branch I am logged in at. If I need to do an extensive search at another location, then I would change My Library. > > 2. Does these need to account for IndependentBranches? Maybe? What I typically see with independent branches is that the libraries are not interested in seeing/looking for any materials other than their own. So I would imagine they would use this to keep the search at their own location and not change it. BUT, I don't have a lot of experience with independent branches, so if anyone else can add to this, that would be great!
Created attachment 130513 [details] [review] Bug 30077: Add system preference IntranetAddMastheadLibraryPulldown To test: 1. Apply patch, updatedatabase, restart_all 2. Search for the system pref 'IntranetAddMastheadLibraryPulldown' 3. Set it to 'Show'. 4. Go to any page that contains header search box and click on 'Search the catalog'. 5. To the left of the search bar you should see a dropdown for branches. 6. This should be defaulted to the branch you are logged in at. 7. Do a search and make sure it is correctly limiting to your logged in branch. 8. Try changing to a different branch and doing another search. It should correcrly limit to that branch BUT it should also default back to your logged in branch for the next search. 9. Try selecting 'All libraries' from the top of the dropdown and make sure it is correctly searching all branches. 10. Turn on the system pref 'IntranetCatalogSearchPulldown' and make sure it looks nice with both of those dropdowns and functions when you use limiters from both dropdowns.
*** Bug 29615 has been marked as a duplicate of this bug. ***
Sorry, I have a note of dissent here. I think the idea is great - having the drop down menu available, mirroring the OPAC functionality, would be very useful. But I disagree with this part: "and by default, should limit to the library you are logged in to, with the option via the dropdown to search all libraries." The default search option on the OPAC is customisable, using the 'SearchMyLibraryFirst' preference. I think that the configuration for this should be similarly customisable, because reading through the bug report, my stance would be opposite to Donna's. Obviously, both workflows are equally valid - and it will entirely depend on the library's set up. Is it possible to have a second system preference so the default search behaviour could be configured? For example: IntranetAddMastheadLibraryPulldown: Show SearchLoggedInLibraryFirst: Yes This meets Donna's criteria - defaulting the search to the logged in library. Or: IntranetAddMastheadLibraryPulldown: Show SearchLoggedInLibraryFirst: No This meets our criteria - having the drop down menu visible so it can be used, but defaulting to 'All Libraries'.
I had filed the same bug earlier as bug 29615 and I very much agree with Sally. We should start out by duplicating the behaviour of the OPAC preference to make things consistent. Then having a SearchMyLibraryFrist equivalent for the staff interface would make a lot of sense as a second step. For our installations not wanting to limit would currently be the more common setting.
Created attachment 131528 [details] [review] Bug 30077: Add system preference IntranetAddMastheadLibraryPulldown To test: 1. Apply patch, updatedatabase, restart_all 2. Search for the system pref 'IntranetAddMastheadLibraryPulldown' 3. Set it to 'Show'. 4. Go to any page that contains header search box and click on 'Search the catalog'. 5. To the left of the search bar you should see a dropdown for branches. 6. This should be defaulted to 'All libraries'. 7. In the dropdown choose any branch and do a search and make sure it is correctly limiting to that branch. 8. Try changing to a different branch and doing another search. It should correcrly limit to that branch BUT it should also default back to 'All libraries' for the next search. 9. Try selecting 'All libraries' from the top of the dropdown and make sure it is correctly searching all branches. 10. Turn on the system pref 'IntranetCatalogSearchPulldown' and make sure it looks nice with both of those dropdowns and functions when you use limiters from both dropdowns.
This patch should now default to 'All libraries', like the OPAC. I think we should file a separate bug to add the system preference "SearchLoggedInLibraryFirst".
Created attachment 131544 [details] [review] Bug 30077: Add system preference IntranetAddMastheadLibraryPulldown To test: 1. Apply patch, updatedatabase, restart_all 2. Search for the system pref 'IntranetAddMastheadLibraryPulldown' 3. Set it to 'Show'. 4. Go to any page that contains header search box and click on 'Search the catalog'. 5. To the left of the search bar you should see a dropdown for branches. 6. This should be defaulted to 'All libraries'. 7. In the dropdown choose any branch and do a search and make sure it is correctly limiting to that branch. 8. Try changing to a different branch and doing another search. It should correcrly limit to that branch BUT it should also default back to 'All libraries' for the next search. 9. Try selecting 'All libraries' from the top of the dropdown and make sure it is correctly searching all branches. 10. Turn on the system pref 'IntranetCatalogSearchPulldown' and make sure it looks nice with both of those dropdowns and functions when you use limiters from both dropdowns. Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>
Works perfectly, thanks Lucas - great feature!
I believe 8) is not right: 8. Try changing to a different branch and doing another search. It should correcrly limit to that branch BUT it should also default back to 'All libraries' for the next search. * In the OPAC the branch selection is maintained and I believe this is the correct behaviour. * We keep all the other search related options: search terms, search option (IntranetCatalogSearchPulldown). So It doesn't make sense that the branch selection would be the only search parameter to be reset. It creates a mismatch of what you see and what has been searched for.
Created attachment 132982 [details] [review] Bug 30077: (follow-up) retain branch selection To test: 1. Apply patch, updatedatabase, restart_all 2. Search for the system pref 'IntranetAddMastheadLibraryPulldown' 3. Set it to 'Show'. 4. Go to any page that contains header search box and click on 'Search the catalog'. 5. To the left of the search bar you should see a dropdown for branches. 6. This should be defaulted to 'All libraries'. 7. In the dropdown choose any branch and do a search and make sure it is correctly limiting to that branch. 8. Try changing to a different branch and doing another search. It should correcrly limit to that branch and retain that branch in the dropdown, just like the OPAC. 9. Try selecting 'All libraries' from the top of the dropdown and make sure it is correctly searching all branches. 10. Turn on the system pref 'IntranetCatalogSearchPulldown' and make sure it looks nice with both of those dropdowns and functions when you use limiters from both dropdowns.
Created attachment 133037 [details] [review] Bug 30077: (follow-up) Set max width for library drapdown If you have a library with a very long name the dropdown will expand to accommodate it. I think this is awkward-looking, so I propose to set a max width on <select> elements in the header search region. The patch requires a rebuild of the staff interface CSS: https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client
> If you have a library with a very long name the dropdown will expand to > accommodate it. This is why I have a library in my test dataset named "Gloucester Memorial Free Public Library Incorporated"
(In reply to Owen Leonard from comment #14) > Created attachment 133037 [details] [review] [review] > Bug 30077: (follow-up) Set max width for library drapdown > > If you have a library with a very long name the dropdown will expand to > accommodate it. I think this is awkward-looking, so I propose to set a > max width on <select> elements in the header search region. > > The patch requires a rebuild of the staff interface CSS: > https://wiki.koha-community.org/wiki/ > Working_with_SCSS_in_the_OPAC_and_staff_client Nice catch, thanks Owen!
Created attachment 135530 [details] [review] Bug 30077: Add system preference IntranetAddMastheadLibraryPulldown To test: 1. Apply patch, updatedatabase, restart_all 2. Search for the system pref 'IntranetAddMastheadLibraryPulldown' 3. Set it to 'Show'. 4. Go to any page that contains header search box and click on 'Search the catalog'. 5. To the left of the search bar you should see a dropdown for branches. 6. This should be defaulted to 'All libraries'. 7. In the dropdown choose any branch and do a search and make sure it is correctly limiting to that branch. 8. Try changing to a different branch and doing another search. It should correcrly limit to that branch BUT it should also default back to 'All libraries' for the next search. 9. Try selecting 'All libraries' from the top of the dropdown and make sure it is correctly searching all branches. 10. Turn on the system pref 'IntranetCatalogSearchPulldown' and make sure it looks nice with both of those dropdowns and functions when you use limiters from both dropdowns. Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk> Signed-off-by: David Nind <david@davidnind.com>
Created attachment 135531 [details] [review] Bug 30077: (follow-up) retain branch selection To test: 1. Apply patch, updatedatabase, restart_all 2. Search for the system pref 'IntranetAddMastheadLibraryPulldown' 3. Set it to 'Show'. 4. Go to any page that contains header search box and click on 'Search the catalog'. 5. To the left of the search bar you should see a dropdown for branches. 6. This should be defaulted to 'All libraries'. 7. In the dropdown choose any branch and do a search and make sure it is correctly limiting to that branch. 8. Try changing to a different branch and doing another search. It should correcrly limit to that branch and retain that branch in the dropdown, just like the OPAC. 9. Try selecting 'All libraries' from the top of the dropdown and make sure it is correctly searching all branches. 10. Turn on the system pref 'IntranetCatalogSearchPulldown' and make sure it looks nice with both of those dropdowns and functions when you use limiters from both dropdowns. Signed-off-by: David Nind <david@davidnind.com>
Created attachment 135532 [details] [review] Bug 30077: (follow-up) Set max width for library drapdown If you have a library with a very long name the dropdown will expand to accommodate it. I think this is awkward-looking, so I propose to set a max width on <select> elements in the header search region. The patch requires a rebuild of the staff interface CSS: https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client Signed-off-by: David Nind <david@davidnind.com>
Created attachment 138134 [details] [review] Bug 30077: Add system preference IntranetAddMastheadLibraryPulldown To test: 1. Apply patch, updatedatabase, restart_all 2. Search for the system pref 'IntranetAddMastheadLibraryPulldown' 3. Set it to 'Show'. 4. Go to any page that contains header search box and click on 'Search the catalog'. 5. To the left of the search bar you should see a dropdown for branches. 6. This should be defaulted to 'All libraries'. 7. In the dropdown choose any branch and do a search and make sure it is correctly limiting to that branch. 8. Try changing to a different branch and doing another search. It should correcrly limit to that branch BUT it should also default back to 'All libraries' for the next search. 9. Try selecting 'All libraries' from the top of the dropdown and make sure it is correctly searching all branches. 10. Turn on the system pref 'IntranetCatalogSearchPulldown' and make sure it looks nice with both of those dropdowns and functions when you use limiters from both dropdowns. Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk> Signed-off-by: David Nind <david@davidnind.com>
Created attachment 138135 [details] [review] Bug 30077: (follow-up) retain branch selection To test: 1. Apply patch, updatedatabase, restart_all 2. Search for the system pref 'IntranetAddMastheadLibraryPulldown' 3. Set it to 'Show'. 4. Go to any page that contains header search box and click on 'Search the catalog'. 5. To the left of the search bar you should see a dropdown for branches. 6. This should be defaulted to 'All libraries'. 7. In the dropdown choose any branch and do a search and make sure it is correctly limiting to that branch. 8. Try changing to a different branch and doing another search. It should correcrly limit to that branch and retain that branch in the dropdown, just like the OPAC. 9. Try selecting 'All libraries' from the top of the dropdown and make sure it is correctly searching all branches. 10. Turn on the system pref 'IntranetCatalogSearchPulldown' and make sure it looks nice with both of those dropdowns and functions when you use limiters from both dropdowns. Signed-off-by: David Nind <david@davidnind.com>
Created attachment 138136 [details] [review] Bug 30077: (follow-up) Set max width for library drapdown If you have a library with a very long name the dropdown will expand to accommodate it. I think this is awkward-looking, so I propose to set a max width on <select> elements in the header search region. The patch requires a rebuild of the staff interface CSS: https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client Signed-off-by: David Nind <david@davidnind.com>
Created attachment 138396 [details] [review] Bug 30077: Add system preference IntranetAddMastheadLibraryPulldown To test: 1. Apply patch, updatedatabase, restart_all 2. Search for the system pref 'IntranetAddMastheadLibraryPulldown' 3. Set it to 'Show'. 4. Go to any page that contains header search box and click on 'Search the catalog'. 5. To the left of the search bar you should see a dropdown for branches. 6. This should be defaulted to 'All libraries'. 7. In the dropdown choose any branch and do a search and make sure it is correctly limiting to that branch. 8. Try changing to a different branch and doing another search. It should correcrly limit to that branch BUT it should also default back to 'All libraries' for the next search. 9. Try selecting 'All libraries' from the top of the dropdown and make sure it is correctly searching all branches. 10. Turn on the system pref 'IntranetCatalogSearchPulldown' and make sure it looks nice with both of those dropdowns and functions when you use limiters from both dropdowns. Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk> Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 138397 [details] [review] Bug 30077: (follow-up) retain branch selection To test: 1. Apply patch, updatedatabase, restart_all 2. Search for the system pref 'IntranetAddMastheadLibraryPulldown' 3. Set it to 'Show'. 4. Go to any page that contains header search box and click on 'Search the catalog'. 5. To the left of the search bar you should see a dropdown for branches. 6. This should be defaulted to 'All libraries'. 7. In the dropdown choose any branch and do a search and make sure it is correctly limiting to that branch. 8. Try changing to a different branch and doing another search. It should correcrly limit to that branch and retain that branch in the dropdown, just like the OPAC. 9. Try selecting 'All libraries' from the top of the dropdown and make sure it is correctly searching all branches. 10. Turn on the system pref 'IntranetCatalogSearchPulldown' and make sure it looks nice with both of those dropdowns and functions when you use limiters from both dropdowns. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 138398 [details] [review] Bug 30077: (follow-up) Set max width for library drapdown If you have a library with a very long name the dropdown will expand to accommodate it. I think this is awkward-looking, so I propose to set a max width on <select> elements in the header search region. The patch requires a rebuild of the staff interface CSS: https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 22.11. Nice work everyone, thanks!
Created attachment 138484 [details] [review] Bug 30077: (QA follow-up) Avoid warning Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Enhancement will not be backported to 22.05.x series