Bug 26263 - Style error on OPAC search results when item-level_itypes is set to bibliographic
Summary: Style error on OPAC search results when item-level_itypes is set to bibliogra...
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: master
Hardware: All All
: P5 - low normal (vote)
Assignee: Martin Renvoize
QA Contact: Testopia
URL:
Keywords:
Depends on: 25242
Blocks:
  Show dependency treegraph
 
Reported: 2020-08-20 22:54 UTC by Katrin Fischer
Modified: 2021-06-14 21:29 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Screenshot of result list with bibliographic level item type showing (135.26 KB, image/png)
2020-08-20 22:56 UTC, Katrin Fischer
Details
Bug 26263: Add column width to itypecol (941 bytes, patch)
2020-08-21 15:35 UTC, Martin Renvoize
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Katrin Fischer 2020-08-20 22:54:33 UTC
When item-level_itypes is set to bibliographic, the OPAC results list splits 50 / 50 with the item type showing in the first half.

To test:
- Set tem-level_itypes to bibliogrphic
- Make sure you have the record level itemtype set in some records
- Search for them in the OPAC
- Verify the display issue in the result list
Comment 1 Katrin Fischer 2020-08-20 22:56:10 UTC
Created attachment 108771 [details]
Screenshot of result list with bibliographic level item type showing
Comment 2 Jonathan Druart 2020-08-21 13:47:39 UTC
Caused by bug 25242.
Comment 3 Martin Renvoize 2020-08-21 15:35:33 UTC
Created attachment 108833 [details] [review]
Bug 26263: Add column width to itypecol

We add a static width to the itypecol and set a matching max-width for
the contained image to improve the layout of the results table for
accessability and reactivity.
Comment 4 Martin Renvoize 2020-08-21 15:37:17 UTC
This patch has the side effect of enforcing a max-width on the item type icon images people are using..

I feel this is a reasonable caveat, but I'm not entirely sure how much those icons are customised or whether those that are customising them are using much larger sizes that may not scale down well?
Comment 5 Owen Leonard 2020-08-21 17:59:36 UTC
I think what this bug shows is that Bug 25242 was too wide-reaching. I think it should be reconsidered.
Comment 6 Katrin Fischer 2020-08-21 18:47:55 UTC
(In reply to Owen Leonard from comment #5)
> I think what this bug shows is that Bug 25242 was too wide-reaching. I think
> it should be reconsidered.

I agree, we should reconsider - can we limit the effect to details?
Comment 7 Martin Renvoize 2020-09-02 15:37:59 UTC
Marking as resolved as we reverted.
Comment 8 Martin Renvoize 2020-09-02 15:38:32 UTC
Note.. I still think the colwidth is better with this patch even with prior patch reverted..