Bug 27200

Summary: "Browse search" is broken
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: OPACAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: andrewfh, caroline.cyr-la-rose, cbutosi, fridolin.somers, victor
Version: unspecifiedKeywords: Manual
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29439
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
21.05.00,20.11.01
Bug Depends on: 20168    
Bug Blocks:    
Attachments: Bug 27200: (bug 20168 follow-up) Fix 'Browse search'
Bug 27200: (follow-up) Browse search interface update
Bug 27200: (bug 20168 follow-up) Fix 'Browse search'
Bug 27200: (follow-up) Browse search interface update
Bug 27200: (bug 20168 follow-up) Fix 'Browse search'
Bug 27200: (follow-up) Browse search interface update

Description Jonathan Druart 2020-12-11 09:21:28 UTC
The error is displayed (when it must be hidden): "An error occurred, please try again." and clicking the "Search" button does not do anything.

Caused by 
  commit 9cb89b46397efc5e288723829613c33e7eea0d69
  Bug 20168: Update of the OPAC bootstrap template to bootstrap v4
Comment 1 Jonathan Druart 2020-12-11 10:20:31 UTC
Created attachment 114324 [details] [review]
Bug 27200: (bug 20168 follow-up) Fix 'Browse search'

The 'hidden' class has been replaced by 'd-none' on bug 20168
Comment 2 Jonathan Druart 2020-12-11 10:21:26 UTC
This patch fixes the original issue, but screen still does not look very nice.
Comment 3 Craig 2020-12-11 15:06:40 UTC
Confirming that Jonathan's patch has restored the ability to search. However, the error statement is still visible and there are some css adjustments necessary as indicated.

To address these, I've simply added the following to OpacUserCSS until they can be patched in a subsequent Koha version:

#browse-search .error {display: none;}
#browse-search #browse-selection {padding-top: 15px;}
#browse-search #browse-searchresults, #browse-search #browse-selectionsearch {border: none;}

Thank you to all!
Comment 4 Owen Leonard 2020-12-16 19:00:23 UTC
Created attachment 114462 [details] [review]
Bug 27200: (follow-up) Browse search interface update

This patch updates the browse search interface to confirm to new
interface patterns in the Bootstrap 4 framework, especially in the form
markup.

The results list has been changed to a Bootstrap "Collapse" component
configured as an accordion
(https://getbootstrap.com/docs/4.5/components/collapse/#accordion-example).

Instead of loading the bibliographic record results below the list of
terms returned, the bibliographic results are now displayed in the
"panel" expanded below the selected term. Subtitle has been added to the
information displayed about the bibliographic record.

To test you must be using ElasticSearch and the OpacBrowseSearch
preference must be enabled. Apply the patch and rebuild the OPAC CSS
(https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client).

- Open the "Browse search" page in the OPAC.
- Test various searches: Author, Subject, and Title.
- When results are found, the should be displayed as a Bootstrap-styled
  accordion widget. Clicking any individual result should expand the
  panel containing the corresponding records.
- Clicking the record link should open the bibliographic detail page in
  a new window.
- When no results are found, a Bootstrap-style "alert" box should
  appear.
Comment 5 Victor Grousset/tuxayo 2020-12-17 21:20:34 UTC
Created attachment 114500 [details] [review]
Bug 27200: (bug 20168 follow-up) Fix 'Browse search'

The 'hidden' class has been replaced by 'd-none' on bug 20168

Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Comment 6 Victor Grousset/tuxayo 2020-12-17 21:20:37 UTC
Created attachment 114501 [details] [review]
Bug 27200: (follow-up) Browse search interface update

This patch updates the browse search interface to confirm to new
interface patterns in the Bootstrap 4 framework, especially in the form
markup.

The results list has been changed to a Bootstrap "Collapse" component
configured as an accordion
(https://getbootstrap.com/docs/4.5/components/collapse/#accordion-example).

Instead of loading the bibliographic record results below the list of
terms returned, the bibliographic results are now displayed in the
"panel" expanded below the selected term. Subtitle has been added to the
information displayed about the bibliographic record.

To test you must be using ElasticSearch and the OpacBrowseSearch
preference must be enabled. Apply the patch and rebuild the OPAC CSS
(https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client).

- Open the "Browse search" page in the OPAC.
- Test various searches: Author, Subject, and Title.
- When results are found, the should be displayed as a Bootstrap-styled
  accordion widget. Clicking any individual result should expand the
  panel containing the corresponding records.
- Clicking the record link should open the bibliographic detail page in
  a new window.
- When no results are found, a Bootstrap-style "alert" box should
  appear.

Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Comment 7 Nick Clemens 2020-12-21 16:20:24 UTC
Created attachment 114566 [details] [review]
Bug 27200: (bug 20168 follow-up) Fix 'Browse search'

The 'hidden' class has been replaced by 'd-none' on bug 20168

Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Comment 8 Nick Clemens 2020-12-21 16:20:27 UTC
Created attachment 114567 [details] [review]
Bug 27200: (follow-up) Browse search interface update

This patch updates the browse search interface to confirm to new
interface patterns in the Bootstrap 4 framework, especially in the form
markup.

The results list has been changed to a Bootstrap "Collapse" component
configured as an accordion
(https://getbootstrap.com/docs/4.5/components/collapse/#accordion-example).

Instead of loading the bibliographic record results below the list of
terms returned, the bibliographic results are now displayed in the
"panel" expanded below the selected term. Subtitle has been added to the
information displayed about the bibliographic record.

To test you must be using ElasticSearch and the OpacBrowseSearch
preference must be enabled. Apply the patch and rebuild the OPAC CSS
(https://wiki.koha-community.org/wiki/Working_with_SCSS_in_the_OPAC_and_staff_client).

- Open the "Browse search" page in the OPAC.
- Test various searches: Author, Subject, and Title.
- When results are found, the should be displayed as a Bootstrap-styled
  accordion widget. Clicking any individual result should expand the
  panel containing the corresponding records.
- Clicking the record link should open the bibliographic detail page in
  a new window.
- When no results are found, a Bootstrap-style "alert" box should
  appear.

Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Comment 9 Jonathan Druart 2020-12-26 17:15:30 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 10 Fridolin Somers 2020-12-30 14:20:26 UTC
Pushed to 20.11.x for 20.11.01
Comment 11 Andrew Fuerste-Henry 2021-01-04 15:34:54 UTC
Missing dependencies, not backported to 20.05