Bug 5571 - tags as bulleted list too long
Summary: tags as bulleted list too long
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: Main
Hardware: All All
: PATCH-Sent (DO NOT USE) minor (vote)
Assignee: Owen Leonard
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-03 14:38 UTC by Nicole C. Engard
Modified: 2012-10-25 23:04 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
long list of tags (16.99 KB, image/png)
2011-01-03 14:38 UTC, Nicole C. Engard
Details
wrapping tags (18.49 KB, image/png)
2011-01-03 14:40 UTC, Nicole C. Engard
Details
Proposed fix (2.60 KB, patch)
2011-01-04 21:09 UTC, Owen Leonard
Details | Diff | Splinter Review
Proposed fix for 3.2.x (2.24 KB, patch)
2011-01-06 18:43 UTC, Owen Leonard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nicole C. Engard 2011-01-03 14:38:07 UTC
Created attachment 2939 [details]
long list of tags

On the opac detail display (with and without XSLT on) the tags show as a bulleted list.  This isn't how it always was and if you have a title with a lot of tags (or show a lot of tags on the detail page) it can be way too long.  This should be a comma separated list to save space or a multi-column bulleted list if the bullets are necessary.
Comment 1 Nicole C. Engard 2011-01-03 14:40:48 UTC
Created attachment 2940 [details]
wrapping tags

Also, with the bullets if the tags are on with a cover image you have the chance that the tags will wrap under the cover image, making it a confusing for the viewer.
Comment 2 Owen Leonard 2011-01-04 21:09:24 UTC
Created attachment 2943 [details] [review]
Proposed fix
Comment 3 Chris Cormack 2011-01-05 02:48:37 UTC
Pushed please test
Comment 4 Owen Leonard 2011-01-06 18:43:10 UTC
Created attachment 2958 [details] [review]
Proposed fix for 3.2.x