Bugzilla – Attachment 160571 Details for
Bug 35707
Item statuses in the holdings table on biblio details should appear one per line
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 35707: Add clearfix to item statuses
Bug-35707-Add-clearfix-to-item-statuses.patch (text/plain), 9.55 KB, created by
Martin Renvoize (ashimema)
on 2024-01-05 12:16:36 UTC
(
hide
)
Description:
Bug 35707: Add clearfix to item statuses
Filename:
MIME Type:
Creator:
Martin Renvoize (ashimema)
Created:
2024-01-05 12:16:36 UTC
Size:
9.55 KB
patch
obsolete
>From 27fc4d459598f071f0219346de81b6303f538eeb Mon Sep 17 00:00:00 2001 >From: Martin Renvoize <martin.renvoize@ptfs-europe.com> >Date: Fri, 5 Jan 2024 12:14:31 +0000 >Subject: [PATCH] Bug 35707: Add clearfix to item statuses > >This patch adds the clearfix class to each of item status/availabiliy >spans. This has the effect of pushing each status onto it's own line. > >Test plan >1) Add an item to stockrotation and trigger the first transfer >2) Place a hold on the item >3) Note that you will now see two status's, one above the other as > displayed in the screenshots attached to the bug. >--- > .../prog/en/modules/catalogue/detail.tt | 36 +++++++++---------- > 1 file changed, 18 insertions(+), 18 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >index 05da5acf56f..2d2e2142e6a 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >@@ -471,9 +471,9 @@ > > [% IF item.object.checkout %] > [% IF item.object.checkout.onsite_checkout %] >- <span>Currently in local use >+ <span class="localuse clearfix">Currently in local use > [% ELSE %] >- <span class="datedue">Checked out >+ <span class="datedue clearfix">Checked out > [% END %] > [% IF item.can_be_edited %] > [% IF item.object.checkout.onsite_checkout %] >@@ -487,9 +487,9 @@ > </span> > [% ELSIF ( transfer = item.object.get_transfer ) %] > [% IF (transfer.datesent) %] >- <span class="intransit">In transit from [% Branches.GetName( transfer.frombranch ) | html %] to [% Branches.GetName( transfer.tobranch ) | html %] since [% transfer.datesent | $KohaDates %]</span> >+ <span class="intransit clearfix">In transit from [% Branches.GetName( transfer.frombranch ) | html %] to [% Branches.GetName( transfer.tobranch ) | html %] since [% transfer.datesent | $KohaDates %]</span> > [% ELSE %] >- <span class="transitrequested">Transit pending from [% Branches.GetName( transfer.frombranch ) | html %] to [% Branches.GetName( transfer.tobranch ) | html %] since [% item.transfer.daterequested | $KohaDates %]</span> >+ <span class="transitrequested clearfix">Transit pending from [% Branches.GetName( transfer.frombranch ) | html %] to [% Branches.GetName( transfer.tobranch ) | html %] since [% item.transfer.daterequested | $KohaDates %]</span> > > [% END %] > [% END %] >@@ -497,32 +497,32 @@ > [% IF ( item.itemlost ) %] > [% SET itemlost_description = AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.itemlost', authorised_value => item.itemlost ) %] > [% IF itemlostloop %] >- <span class="lost">[% itemlost_description | html %]</span> >+ <span class="lost clearfix">[% itemlost_description | html %]</span> > [% ELSE %] >- <span class="lost">Unavailable (lost or missing)</span> >+ <span class="lost clearfix">Unavailable (lost or missing)</span> > [% END %] > [% END %] > > [% IF ( item.withdrawn ) %] > [% SET withdrawn_description = AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.withdrawn', authorised_value => item.withdrawn ) %] > [% IF withdrawn_description %] >- <span class="wdn">[% withdrawn_description | html %]</span> >+ <span class="wdn clearfix">[% withdrawn_description | html %]</span> > [% ELSE %] >- <span class="wdn">Withdrawn</span> >+ <span class="wdn clearfix">Withdrawn</span> > [% END %] > [% END %] > > [% IF ( item.damaged ) %] > [% SET damaged_description = AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.damaged', authorised_value => item.damaged ) %] > [% IF damaged_description %] >- <span class="dmg">[% damaged_description | html %]</span> >+ <span class="dmg clearfix">[% damaged_description | html %]</span> > [% ELSE %] >- <span class="dmg">Damaged</span> >+ <span class="dmg clearfix">Damaged</span> > [% END %] > [% END %] > > [% IF ( item.notforloan || item.itemtype.notforloan ) %] >- <span class="notforloan">Not for loan >+ <span class="notforloan clearfix">Not for loan > [% SET not_for_loan_description = AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.notforloan', authorised_value => item.notforloan ) %] > [% IF not_for_loan_description %] > <span class="reason">([% not_for_loan_description | html %])</span> >@@ -533,13 +533,13 @@ > [% SET hold = item.first_hold %] > [% IF hold %] > [% IF hold.waitingdate %] >- <span class="waitingat">Waiting at [% Branches.GetName( hold.branchcode ) | html %][% IF ( hold.desk_id ) %], [% hold.desk.desk_name | html %][% END %] since [% hold.waitingdate | $KohaDates %].</span> >+ <span class="waitingat clearfix">Waiting at [% Branches.GetName( hold.branchcode ) | html %][% IF ( hold.desk_id ) %], [% hold.desk.desk_name | html %][% END %] since [% hold.waitingdate | $KohaDates %].</span> > [% IF canreservefromotherbranches AND ( hold.waitingdate OR hold.priority == 1 ) %] > <span class="heldfor">Hold for:</span> > [% INCLUDE 'patron-title.inc' patron=hold.borrower hide_patron_infos_if_needed=1 %] > [% END %] > [% ELSE %] >- <span class="holdonitem">There is an item level hold on this item (priority = [% hold.priority | html %]).</span> >+ <span class="holdonitem clearfix">There is an item level hold on this item (priority = [% hold.priority | html %]).</span> > [% END %] > [% END %] > >@@ -547,24 +547,24 @@ > [% SET recall = item.object.recall %] > [% IF recall %] > [% IF recall.waiting_date %] >- <span>Waiting at [% Branches.GetName( recall.pickup_library_id ) | html %] since [% recall.waiting_date | $KohaDates %]</span> >+ <span class="recallat clearfix">Waiting at [% Branches.GetName( recall.pickup_library_id ) | html %] since [% recall.waiting_date | $KohaDates %]</span> > [% ELSE %] > [% patron_link = BLOCK %]<a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% recall.patron_id | uri %]">[% recall.patron.firstname | html %] [% recall.patron.surname | html %] ([% recall.patron.cardnumber | html %])</a>[% END %] >- <span>recalled by [% patron_link| $raw %] on [% recall.created_date | $KohaDates %]</span> >+ <span class="recallby clearfix">recalled by [% patron_link| $raw %] on [% recall.created_date | $KohaDates %]</span> > [% END %] > [% END %] > [% END %] > > [% UNLESS ( item.notforloan || item.itemtype.notforloan || item.onloan || item.itemlost || item.withdrawn || item.damaged || item.transfer || hold || ( Koha.Preference('UseRecalls') && recall ) ) %] >- <span>Available</span> >+ <span class="clearfix">Available</span> > [% END %] > > [% IF ( item.restricted ) %] >- <span class="restricted">([% AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.restricted', authorised_value => item.restricted ) | html %])</span> >+ <span class="restricted clearfix">([% AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.restricted', authorised_value => item.restricted ) | html %])</span> > [% END %] > > [% IF ( item.bundle_host ) %] >- <span class="bundled">In bundle: [% INCLUDE 'biblio-title.inc' biblio = item.bundle_host.biblio link = 1 %]</span> >+ <span class="bundled clearfix">In bundle: [% INCLUDE 'biblio-title.inc' biblio = item.bundle_host.biblio link = 1 %]</span> > [% END %] > > </td> >-- >2.43.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 35707
:
160569
|
160570
|
160571
|
160651
|
160663
|
160683