Summary: | Image obscured by the search results toolbar when previewing cover images from OPAC search results | ||
---|---|---|---|
Product: | Koha | Reporter: | David Nind <david> |
Component: | OPAC | Assignee: | Owen Leonard <oleonard> |
Status: | Pushed to main --- | QA Contact: | Martin Renvoize (ashimema) <martin.renvoize> |
Severity: | normal | ||
Priority: | P5 - low | CC: | alexbuckley, martin.renvoize |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: |
This fixes the display of cover images in the OPAC - the image viewer was appearing behind the search results toolbar, partially obscuring the cover image.
|
Version(s) released in: |
25.05.00
|
Circulation function: | |||
Bug Depends on: | 26933 | ||
Bug Blocks: | |||
Attachments: |
Screenshot: cover image preview from OPAC search results partially obscured by the menu bar
Bug 38657: Prevent cover image from being from being obscured by the search results toolbar Bug 38657: Prevent cover image from being from being obscured by the search results toolbar Bug 38657: Prevent cover image from being from being obscured by the search results toolbar |
Description
David Nind
2024-12-09 18:19:08 UTC
Created attachment 175328 [details]
Screenshot: cover image preview from OPAC search results partially obscured by the menu bar
Screenshot showing the image preview being partially obscured by the search results menu bar.
Created attachment 175329 [details] [review] Bug 38657: Prevent cover image from being from being obscured by the search results toolbar On the OPAC search results page we're including the Chocolat image viewer CSS after our global CSS file. This means that rules which are intended to override the default Chocolat CSS aren't working. The fix is to move the line which includes the Chocolat CSS. To test, apply the patch and enable the LocalCoverImages and OPACLocalCoverImages system preferences. - If necessary, add a local cover image to a bibliographic record. - Go to the OPAC and perform a search which will return multiple record including the record with the cover image. - Click the cover image to trigger the image viewer. - The image viewer should not appear to be "behind" the search results toolbar. Sponsored-by: Athens County Public Libraries Created attachment 175330 [details] [review] Bug 38657: Prevent cover image from being from being obscured by the search results toolbar On the OPAC search results page we're including the Chocolat image viewer CSS after our global CSS file. This means that rules which are intended to override the default Chocolat CSS aren't working. The fix is to move the line which includes the Chocolat CSS. To test, apply the patch and enable the LocalCoverImages and OPACLocalCoverImages system preferences. - If necessary, add a local cover image to a bibliographic record. - Go to the OPAC and perform a search which will return multiple record including the record with the cover image. - Click the cover image to trigger the image viewer. - The image viewer should not appear to be "behind" the search results toolbar. Sponsored-by: Athens County Public Libraries Signed-off-by: David Nind <david@davidnind.com> Thanks Owen! Created attachment 175590 [details] [review] Bug 38657: Prevent cover image from being from being obscured by the search results toolbar On the OPAC search results page we're including the Chocolat image viewer CSS after our global CSS file. This means that rules which are intended to override the default Chocolat CSS aren't working. The fix is to move the line which includes the Chocolat CSS. To test, apply the patch and enable the LocalCoverImages and OPACLocalCoverImages system preferences. - If necessary, add a local cover image to a bibliographic record. - Go to the OPAC and perform a search which will return multiple record including the record with the cover image. - Click the cover image to trigger the image viewer. - The image viewer should not appear to be "behind" the search results toolbar. Sponsored-by: Athens County Public Libraries Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Pushed for 25.05! Well done everyone, thank you! not backporting to 24.05.x unless requested - dependency not in 24.05 |