Bugzilla – Attachment 182929 Details for
Bug 39939
Cancel selected holds button on the holds awaiting pickup page is the same color as the background
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 39939: Cancel selected holds button on waitingreserves.pl is the same color as the background
Bug-39939-Cancel-selected-holds-button-on-waitingr.patch (text/plain), 5.76 KB, created by
Owen Leonard
on 2025-06-03 18:59:05 UTC
(
hide
)
Description:
Bug 39939: Cancel selected holds button on waitingreserves.pl is the same color as the background
Filename:
MIME Type:
Creator:
Owen Leonard
Created:
2025-06-03 18:59:05 UTC
Size:
5.76 KB
patch
obsolete
>From ee9a3b9a77befe7956696c0809e56d5d71b3d5c3 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Tue, 3 Jun 2025 16:32:58 +0000 >Subject: [PATCH] Bug 39939: Cancel selected holds button on waitingreserves.pl > is the same color as the background > >This patch makes some minor changes to the "Holds awaiting pickup" page >so that the cancel buttons are clear and usable. > >The patch also wraps the reference to the >TransferWhenCancelAllWaitingHolds system preference in a permission >check so that it can be linked to the preference when appropriate. > >To test you should have some waiting holds in three categories: > >1. Holds which have not been waiting longer than the number of days > specified in the ReservesMaxPickUpDelay system preference. > >2. Holds which have been waiting longer. I belive the only way to test > this is to manually update the expiration date of an already waiting > hold so that it is in the past. > >3. Waiting holds which a patron has requested be cancelled. (Requires > that the patron and hold are allowed according to the "Cancellation > allowed" setting under Administration -> Circulation and fine rules > -> Default waiting hold cancellation policy. > - Log in to the OPAC as a patron who has a waiting hold. On the "Your > summary" page, under the "Holds" tab, click the "Request to cancel" > button for at least one of the waiting holds. > >- In the staff interface, go to Circulation -> Holds awaiting pickup. > - Under each tab, check the appearance of the "Cancel selected" > buttons (and the "Cancel and transfer all" button under "Holds > awaiting past..." tab). >- When testing as a user with system preference privileges, and the > TransferWhenCancelAllWaitingHolds preference set to "Don't transfer," > the hint shown under the "Holds awaiting past..." tab should be linked > to system preferences. A user without system preference privileges > should not see a link. > >Sponsored-by: Athens County Public Libraries >--- > .../prog/en/modules/circ/waitingreserves.tt | 30 ++++++++++--------- > 1 file changed, 16 insertions(+), 14 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt >index 3f771e140b6..5b44502d5c4 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt >@@ -15,13 +15,15 @@ > [% END %]</title > > > [% INCLUDE 'doc-head-close.inc' %] >-<style> >- .tab-toolbar { >- padding: 0.5rem 0; >- margin-bottom: 0.5rem; >- border-bottom: 1px solid #eee; >- } >-</style> >+[% FILTER collapse %] >+ <style> >+ .tab-toolbar { >+ padding: 0.5rem 0; >+ margin-bottom: 0.5rem; >+ border-bottom: 1px solid #eee; >+ } >+ </style> >+[% END %] > </head> > > <body id="circ_waitingreserves" class="circ"> >@@ -104,8 +106,8 @@ > [% WRAPPER tab_panels %] > [% WRAPPER tab_panel tabname="holdswaiting" bt_active= 1 %] > [% IF ( reserveloop ) %] >- <div id="toolbar" class="btn-toolbar"> >- <button class="btn cancel_selected_holds" data-bulk="true"></button> >+ <div class="tab-toolbar"> >+ <button class="btn btn-default cancel_selected_holds" data-bulk="true"></button> > </div> > [% INCLUDE waiting_holds.inc table_name='holdst' reserveloop=reserveloop tab='holdwaiting' %] > [% ELSE %] >@@ -124,9 +126,9 @@ > <input type="hidden" name="allbranches" value="[% allbranches | html %]" /> > <input type="hidden" name="tab" value="holdsover" /> > [% IF TransferWhenCancelAllWaitingHolds %] >- <input type="submit" class="btn btn-primary" value="Cancel and transfer all" /> >+ <button type="submit" class="btn btn-primary">Cancel and transfer all</button> > [% ELSE %] >- <input type="submit" class="btn btn-primary" value="Cancel all" /> >+ <button type="submit" class="btn btn-primary">Cancel all</button> > [% END %] > </form> > </div> >@@ -134,7 +136,7 @@ > [% UNLESS TransferWhenCancelAllWaitingHolds %] > <div class="hint" > >Only items that need not be transferred will be cancelled >- [% IF ( CAN_user_parameters ) %] >+ [% IF ( CAN_user_parameters_manage_sysprefs ) %] > (<a href="/cgi-bin/koha/admin/preferences.pl?op=search&searchfield=TransferWhenCancelAllWaitingHolds">TransferWhenCancelAllWaitingHolds syspref</a>) > [% ELSE %] > (TransferWhenCancelAllWaitingHolds syspref) >@@ -149,8 +151,8 @@ > > [% WRAPPER tab_panel tabname="holdscancelled" %] > [% IF cancel_reqs_count %] >- <div id="toolbar" class="btn-toolbar"> >- <button class="btn cancel_selected_holds" data-bulk="true"></button> >+ <div class="tab-toolbar"> >+ <button class="btn btn-default cancel_selected_holds" data-bulk="true"></button> > </div> > [% INCLUDE waiting_holds.inc select_column='1' table_name='holdscr' reserveloop=cancel_reqs tab='holdscr' %] > [% ELSE %] >-- >2.39.5
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 39939
:
182929
|
182956
|
182961
|
182962
|
183495
|
183496