Bugzilla – Attachment 180795 Details for
Bug 38994
Class up the circulation homepage
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 38994: Add classes to circulation module homepage
Bug-38994-Add-classes-to-circulation-module-homepa.patch (text/plain), 16.80 KB, created by
Jason Robb
on 2025-04-10 17:44:10 UTC
(
hide
)
Description:
Bug 38994: Add classes to circulation module homepage
Filename:
MIME Type:
Creator:
Jason Robb
Created:
2025-04-10 17:44:10 UTC
Size:
16.80 KB
patch
obsolete
>From 398f4b53f3f0f11afb0ffd99c6ad0e47a3e7253a Mon Sep 17 00:00:00 2001 >From: Andrew Fuerste Henry <andrew@bywatersolutions.com> >Date: Thu, 10 Apr 2025 14:09:55 +0000 >Subject: [PATCH] Bug 38994: Add classes to circulation module homepage > >To test: >1: inspect /koha/circ/circulation-home.pl, see most elements lack distinct classes >2: apply patch, reload page >3: inspect page again, confirm nothing has changed in display >4: confirm page elements have distinct classes now and can be accurately targetted with CSS > >Signed-off-by: Jason Robb <jrobb@sekls.org> >--- > .../prog/en/modules/circ/circulation-home.tt | 178 ++++++++++-------- > 1 file changed, 96 insertions(+), 82 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation-home.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation-home.tt >index 487cc5efbf..ee5dbc7c76 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation-home.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation-home.tt >@@ -28,140 +28,154 @@ > <h1>Circulation</h1> > > <div class="row"> >- <div class="col-12 col-sm-4"> >+ <div class="col-12 col-sm-4 circulation-actions"> > <h3>Circulation</h3> > > <ul class="buttons-list"> > <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/circulation.pl"><i class="fa fa-upload"></i> Check out</a> >+ <a class="circ-button checkout" href="/cgi-bin/koha/circ/circulation.pl"><i class="fa fa-upload"></i> Check out</a> > </li> > <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/returns.pl"><i class="fa fa-download"></i> Check in</a> >+ <a class="circ-button checkin" href="/cgi-bin/koha/circ/returns.pl"><i class="fa fa-download"></i> Check in</a> > </li> > <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/renew.pl"><i class="fa fa-retweet"></i> Renew</a> >+ <a class="circ-button renew" href="/cgi-bin/koha/circ/renew.pl"><i class="fa fa-retweet"></i> Renew</a> > </li> > [% IF ( CAN_user_superlibrarian || CAN_user_loggedinlibrary ) %] > <li> > [% IF Koha.Preference('UseCirculationDesks') %] >- <a class="circ-button" href="/cgi-bin/koha/circ/set-library.pl"><i class="fa fa-home"></i> Set library and desk</a> >+ <a class="circ-button set-library-desk" href="/cgi-bin/koha/circ/set-library.pl"><i class="fa fa-home"></i> Set library and desk</a> > [% ELSE %] >- <a class="circ-button" href="/cgi-bin/koha/circ/set-library.pl"><i class="fa fa-home"></i> Set library</a> >+ <a class="circ-button set-library" href="/cgi-bin/koha/circ/set-library.pl"><i class="fa fa-home"></i> Set library</a> > [% END %] > </li> > [% ELSIF Koha.Preference('UseCirculationDesks') %] > <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/set-library.pl"><i class="fa fa-home"></i> Set desk</a> >+ <a class="circ-button set-desk" href="/cgi-bin/koha/circ/set-library.pl"><i class="fa fa-home"></i> Set desk</a> > </li> > [% END %] > [% IF ( fast_cataloging ) %] > [% IF ( CAN_user_editcatalogue_fast_cataloging ) %] > <li> >- <a class="circ-button" href="/cgi-bin/koha/cataloguing/addbiblio.pl?frameworkcode=FA"><i class="fa fa-plus"></i> Fast cataloging</a> >+ <a class="circ-butto fastadd" href="/cgi-bin/koha/cataloguing/addbiblio.pl?frameworkcode=FA"><i class="fa fa-plus"></i> Fast cataloging</a> > </li> > [% END %] > [% END %] > [% IF ( Koha.Preference('AllowCheckoutNotes') && CAN_user_circulate_manage_checkout_notes ) %] > <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/checkout-notes.pl"><i class="fa fa-sticky-note"></i> Checkout notes</a> >+ <a class="circ-button checkout-notes" href="/cgi-bin/koha/circ/checkout-notes.pl"><i class="fa fa-sticky-note"></i> Checkout notes</a> > </li> > [% END %] > [% IF Koha.Preference('OnSiteCheckouts') %] > <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/on-site_checkouts.pl"><i class="fa-solid fa-location-dot"></i> Pending on-site checkouts</a> >+ <a class="circ-button pending-onsite" href="/cgi-bin/koha/circ/on-site_checkouts.pl"><i class="fa-solid fa-location-dot"></i> Pending on-site checkouts</a> > </li> > [% END %] > </ul> > </div> > > <div class="col-12 col-sm-4"> >- <h3>Holds and bookings</h3> >+ <div class="holds-bookings"> >+ <h3>Holds and bookings</h3> > >- <ul class="buttons-list"> >- <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/view_holdsqueue.pl"><i class="fa fa-tasks"></i> Holds queue</a> >- </li> >- <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/pendingreserves.pl"><i class="fa-solid fa-hand-back-fist"></i> Holds to pull</a> >- </li> >- <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/waitingreserves.pl"><i class="fa-solid fa-calendar-days"></i> Holds awaiting pickup</a> >- </li> >- [% IF ( Koha.Preference('CurbsidePickup') && CAN_user_circulate_manage_curbside_pickups ) %] >+ <ul class="buttons-list"> > <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/curbside_pickups.pl"><i class="fa-solid fa-rotate"></i> Curbside pickups</a> >+ <a class="circ-button holds-queue" href="/cgi-bin/koha/circ/view_holdsqueue.pl"><i class="fa fa-tasks"></i> Holds queue</a> > </li> >- [% END %] >- <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/reserveratios.pl"><i class="fa-solid fa-chart-line"></i> Hold ratios</a> >- </li> >- <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/pendingbookings.pl"><i class="fa-solid fa-calendar-days"></i> Bookings to collect</a> >- </li> >- </ul> >+ <li> >+ <a class="circ-button holds-to-pull" href="/cgi-bin/koha/circ/pendingreserves.pl"><i class="fa-solid fa-hand-back-fist"></i> Holds to pull</a> >+ </li> >+ <li> >+ <a class="circ-button holds-awaiting" href="/cgi-bin/koha/circ/waitingreserves.pl"><i class="fa-solid fa-calendar-days"></i> Holds awaiting pickup</a> >+ </li> >+ [% IF ( Koha.Preference('CurbsidePickup') && CAN_user_circulate_manage_curbside_pickups ) %] >+ <li> >+ <a class="circ-button curbside" href="/cgi-bin/koha/circ/curbside_pickups.pl"><i class="fa-solid fa-rotate"></i> Curbside pickups</a> >+ </li> >+ [% END %] >+ <li> >+ <a class="circ-button holds-ratio" href="/cgi-bin/koha/circ/reserveratios.pl"><i class="fa-solid fa-chart-line"></i> Hold ratios</a> >+ </li> >+ <li> >+ <a class="circ-button bookings-to-collect" href="/cgi-bin/koha/circ/pendingbookings.pl"><i class="fa-solid fa-calendar-days"></i> Bookings to collect</a> >+ </li> >+ </ul> >+ </div> > > [% IF Koha.Preference('UseRecalls') and CAN_user_recalls %] >- <h3>Recalls</h3> >+ <div class="recalls"> >+ <h3>Recalls</h3> > >- <ul class="buttons-list"> >- <li >- ><a class="circ-button" href="/cgi-bin/koha/recalls/recalls_queue.pl" title="All active recalls"><i class="fa fa-tasks"></i> Recalls queue</a></li >- > >- <li >- ><a class="circ-button" href="/cgi-bin/koha/recalls/recalls_to_pull.pl" title="Recalls that could be filled but have not been set waiting"><i class="fa-solid fa-hand-back-fist"></i> Recalls to pull</a></li >- > >- <li >- ><a class="circ-button" href="/cgi-bin/koha/recalls/recalls_overdue.pl" title="Recalled items that are overdue to be returned"><i class="fa-solid fa-clock"></i> Overdue recalls</a></li >- > >- <li >- ><a class="circ-button" href="/cgi-bin/koha/recalls/recalls_waiting.pl" title="Recalled items awaiting pickup"><i class="fa-solid fa-calendar-days"></i> Recalls awaiting pickup</a></li >- > >- <li >- ><a class="circ-button" href="/cgi-bin/koha/recalls/recalls_old_queue.pl" title="Inactive recalls"><i class="fa fa-tasks"></i> Old recalls</a></li >- > >- </ul> >+ <ul class="buttons-list"> >+ <li >+ ><a class="circ-button recalls-queue" href="/cgi-bin/koha/recalls/recalls_queue.pl" title="All active recalls"><i class="fa fa-tasks"></i> Recalls queue</a></li >+ > >+ <li >+ ><a class="circ-button recalls-to-pull" href="/cgi-bin/koha/recalls/recalls_to_pull.pl" title="Recalls that could be filled but have not been set waiting" >+ ><i class="fa-solid fa-hand-back-fist"></i> Recalls to pull</a >+ ></li >+ > >+ <li >+ ><a class="circ-button overdue-recalls" href="/cgi-bin/koha/recalls/recalls_overdue.pl" title="Recalled items that are overdue to be returned"><i class="fa-solid fa-clock"></i> Overdue recalls</a></li >+ > >+ <li >+ ><a class="circ-button recalls-awaiting" href="/cgi-bin/koha/recalls/recalls_waiting.pl" title="Recalled items awaiting pickup"><i class="fa-solid fa-calendar-days"></i> Recalls awaiting pickup</a></li >+ > >+ <li >+ ><a class="circ-button old-recalls" href="/cgi-bin/koha/recalls/recalls_old_queue.pl" title="Inactive recalls"><i class="fa fa-tasks"></i> Old recalls</a></li >+ > >+ </ul> >+ </div> > [% END %] > </div> > > <div class="col-12 col-sm-4"> > [% IF Koha.Preference('ArticleRequests') %] >- <h3>Patron request</h3> >- <ul class="buttons-list"> >- <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/article-requests.pl" title="Article requests"><i class="fa-solid fa-newspaper"></i> Article requests</a> >- </li> >- </ul> >+ <div class="patron-request"> >+ <h3>Patron request</h3> >+ <ul class="buttons-list"> >+ <li> >+ <a class="circ-button article-request" href="/cgi-bin/koha/circ/article-requests.pl" title="Article requests"><i class="fa-solid fa-newspaper"></i> Article requests</a> >+ </li> >+ </ul> >+ </div> > [% END %] > >- <h3>Transfers</h3> >+ <div class="transfers"> >+ <h3>Transfers</h3> > >- <ul class="buttons-list"> >- [% IF !Koha.Preference('IndependentBranchesTransfers') || CAN_user_superlibrarian %] >- <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/branchtransfers.pl"><i class="fa-solid fa-right-left"></i> Transfer</a> >- </li> >- [% END %] >- [% IF Koha.Preference('StockRotation') %] >+ <ul class="buttons-list"> >+ [% IF !Koha.Preference('IndependentBranchesTransfers') || CAN_user_superlibrarian %] >+ <li> >+ <a class="circ-button transfer" href="/cgi-bin/koha/circ/branchtransfers.pl"><i class="fa-solid fa-right-left"></i> Transfer</a> >+ </li> >+ [% END %] >+ [% IF Koha.Preference('StockRotation') %] >+ <li> >+ <a class="circ-button transfers-to-send" href="/cgi-bin/koha/circ/transfers_to_send.pl"><i class="fa-solid fa-right-from-bracket"></i> Transfers to send</a> >+ </li> >+ [% END %] > <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/transfers_to_send.pl"><i class="fa-solid fa-right-from-bracket"></i> Transfers to send</a> >+ <a class="circ-button transfers-to-receive" href="/cgi-bin/koha/circ/transferstoreceive.pl"><i class="fa-solid fa-right-to-bracket"></i> Transfers to receive</a> > </li> >- [% END %] >- <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/transferstoreceive.pl"><i class="fa-solid fa-right-to-bracket"></i> Transfers to receive</a> >- </li> >- </ul> >+ </ul> >+ </div> > > [% IF ( CAN_user_circulate_overdues_report ) %] >- <h3>Overdues</h3> >+ <div class="overdues"> >+ <h3>Overdues</h3> > >- <ul class="buttons-list"> >- <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/overdue.pl" title="Warning: This report is very resource intensive on systems with large numbers of overdue items."><i class="fa-solid fa-clock"></i> Overdues</a> >- </li> >- <li> >- <a class="circ-button" href="/cgi-bin/koha/circ/branchoverdues.pl" title="Limited to your library. See report help for other details."><i class="fa-solid fa-clock"></i> Overdues with fines</a> >- </li> >- </ul> >+ <ul class="buttons-list"> >+ <li> >+ <a class="circ-button overdues-report" href="/cgi-bin/koha/circ/overdue.pl" title="Warning: This report is very resource intensive on systems with large numbers of overdue items." >+ ><i class="fa-solid fa-clock"></i> Overdues</a >+ > >+ </li> >+ <li> >+ <a class="circ-button overdues-fines" href="/cgi-bin/koha/circ/branchoverdues.pl" title="Limited to your library. See report help for other details."><i class="fa-solid fa-clock"></i> Overdues with fines</a> >+ </li> >+ </ul> >+ </div> > [% END %] > </div> > </div> >@@ -172,15 +186,15 @@ > <div class="col-md-12"> > <h4>Offline circulation</h4> > <div class="col-sm-5 col-md-3"> >- <p><a href="/cgi-bin/koha/offline_circ/process_koc.pl">Upload offline circulation file (.koc)</a></p> >- <p><a href="/cgi-bin/koha/offline_circ/list.pl">Pending offline circulation actions</a></p> >+ <p><a href="/cgi-bin/koha/offline_circ/process_koc.pl" class="offline-circ-upload">Upload offline circulation file (.koc)</a></p> >+ <p><a href="/cgi-bin/koha/offline_circ/list.pl" class="offline-circ-pending">Pending offline circulation actions</a></p> > </div> > <div class="col-sm-5 col-md-3"> > <p >- ><a href="https://github.com/bywatersolutions/koha-offline-circulation/releases"><i class="fa-solid fa-window-maximize"></i> Get desktop application</a></p >+ ><a href="https://github.com/bywatersolutions/koha-offline-circulation/releases" class="offline-desktop-app"><i class="fa-solid fa-window-maximize"></i> Get desktop application</a></p > > > <p >- ><a href="https://addons.mozilla.org/[% lang | uri %]/firefox/addon/koct/"><i class="fa-brands fa-firefox"></i> Get Firefox add-on</a></p >+ ><a href="https://addons.mozilla.org/[% lang | uri %]/firefox/addon/koct/" class="offline-firefox-addon"><i class="fa-brands fa-firefox"></i> Get Firefox add-on</a></p > > > </div> > </div> >-- >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 38994
:
177319
|
180782
|
180795
|
180894