Bug 6260 - Missing space in availability information of OPAC XSLT results
Summary: Missing space in availability information of OPAC XSLT results
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: 3.4
Hardware: All All
: P5 - low trivial (vote)
Assignee: Owen Leonard
QA Contact: Bugs List
URL: /cgi-bin/koha/opac-search.pl?q=bodensee
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-25 11:09 UTC by Katrin Fischer
Modified: 2013-12-05 20:01 UTC (History)
3 users (show)

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


Attachments
screenshot (12.27 KB, image/png)
2011-04-25 11:09 UTC, Katrin Fischer
Details
Proposed fix (1.18 KB, patch)
2011-08-08 17:03 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 Katrin Fischer 2011-04-25 11:09:27 UTC
Created attachment 3997 [details]
screenshot

There should be a space between 'No copies available' and 'On Hold (1)'.

Or perhaps don't show 'No copies available' when there are items attached to the record in different status?
Comment 1 Nicole C. Engard 2011-04-25 11:48:21 UTC
I like your second option, only show no-items available when there are no items attached. I'd go a step further (probably another bug) and say if there is a URL and no items attached you also don't show 'no items available'
Comment 2 Katrin Fischer 2011-04-25 11:54:55 UTC
Hi Nicole, I like your idea! 

Perhaps for things having a URL but no items we shouldn't show the whole line 'Availability: '
Comment 3 Nicole C. Engard 2011-04-25 12:30:21 UTC
That is even better!!
Comment 4 Owen Leonard 2011-08-08 17:03:38 UTC
Created attachment 4894 [details] [review]
Proposed fix

This patch corrects the spacing issue. It doesn't address any of the other ideas presented here.
Comment 5 Ian Walls 2011-08-10 11:03:47 UTC
Trivial, single-character fix to a .xsl, marking as Passed QA
Comment 6 Chris Cormack 2011-08-11 00:44:09 UTC
Pushed, and reset patch status, as there are more issues on this bug to address
Comment 7 Owen Leonard 2011-08-11 19:58:39 UTC
Since there are some competing suggestions here I would like to ask that separate bugs be created. Please test the spacing issue and mark this fixed if appropriate.