In OPAC search results, titles are not aligned below each other if the cover display differs

Bug #1920268 reported by Eva Cerninakova
24
This bug affects 5 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned

Bug Description

In the search result screen in Bootstrap OPAC, the titles are not aligned below each other. Left alignment depends on availability of the cover image for the title or on the size of the cover image, see the attached image title_alingnment.png (example from test sever https://festivus.evergreencatalog.com/)

I'm not sure if that's the intention.
However, considering the accessibility, this might be a problem. For example, it makes it difficult for visually impaired people to navigate the list of results.

Unless we missed some furher context of the problem, to align the list of titles regardless of the availability of the title cover image or its size could be quite simple:

In the templates-bootstrap/opac/parts/result/table.tt2 (https://git.evergreen-ils.org/?p=Evergreen.git;a=blob;f=Open-ILS/src/templates-bootstrap/opac/parts/result/table.tt2;h=6b0726ac562b0624b11d2a6b06c09ae9d0f0da64;hb=HEAD)
 - to change col-sm-auto to col-sm-2 on line 91,
 - to change  class='my-2' to class='my-2 img-fluid' on line 93.

We applied the above template changes to our test catalog (Evergreen version 3.6.2) and it works, see the attached image aligned_titles.png. The display is seamless even on mobile devices.

Revision history for this message
Eva Cerninakova (ece) wrote :
Revision history for this message
Eva Cerninakova (ece) wrote :
tags: added: opac usability
Changed in evergreen:
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
Eva Cerninakova (ece) wrote :
Gina Monti (gmonti90)
tags: added: bootstrap design
Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

Working branch user/jeffdavis/lp1920268-bootstrap-align-titles implements Eva's proposed changes:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jeffdavis/lp1920268-bootstrap-align-titles

(Possibly the commit should be modified to make Eva the author.)

tags: added: pullrequest
Changed in evergreen:
milestone: none → 3.9.1
Revision history for this message
Garry Collum (gcollum) wrote :
tags: added: signedoff
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Thanks, Eva, Jeff, and Garry! Pushed to rel_3_9 and above.

Changed in evergreen:
status: Confirmed → Fix Released
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Fixing my mistake; I should have marked this "Fix Committed", but accidentally chose "Fix Released"

Changed in evergreen:
status: Fix Released → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.