Attachment 170828 Details for Bug 37761 – Bug 37761: Fixed styling of Tabs on curbside_pickups.tt page

[patch] Bug 37761: Fixed styling of Tabs on curbside_pickups.tt page

Bug-37761-Fixed-styling-of-Tabs-on-curbsidepickups.patch (text/plain), 4.08 KB, created by Eric Garcia on 2024-08-28 16:18:13 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Eric Garcia

Created: 2024-08-28 16:18:13 UTC

Size: 4.08 KB

patch

obsolete

>From 75090fce50042780d8990ba24c3de3e6693ff31b Mon Sep 17 00:00:00 2001
>From: Eric Garcia <cubingguy714@gmail.com>
>Date: Wed, 28 Aug 2024 16:14:44 +0000
>Subject: [PATCH] Bug 37761: Fixed styling of Tabs on curbside_pickups.tt page
>
>To test:
>- Enable the CurbsidePickup system preference if necessary.
>- Properly testing requires testing data. You can run this command to
>  REPLACE your curbside pickup data with sample data:
>
>  bash <(curl -s https://gitlab.com/-/snippets/2572579/raw/main/test_curbside_pickups.sh)
>
>- Go to Circulation -> Curbside pickups.
>- Look at the tabs above each table, no style. (To be staged, Staged & ready, etc )
>
>Apply patch and notice each tab has the correct style.
>---
> .../prog/en/modules/circ/curbside_pickups.tt           | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/curbside_pickups.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/curbside_pickups.tt
>index 011b5b6a22..0da5dd787b 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/curbside_pickups.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/curbside_pickups.tt
>@@ -155,35 +155,35 @@
>                         [% ELSE %]
>                         <li role="presentation">
>                         [% END %]
>-                            <a id="to-be-staged-tab"      href="#to-be-staged" role="tab" data-bs-toggle="tab">To be staged ([% to_be_staged.count | html %])</a>
>+                            <a id="to-be-staged-tab" class="nav-link" href="#to-be-staged" role="tab" data-bs-toggle="tab">To be staged ([% to_be_staged.count | html %])</a>
>                         </li>
>                         [% IF tab == 'staged-and-ready' %]
>                         <li role="presentation" class="active">
>                         [% ELSE %]
>                         <li role="presentation">
>                         [% END %]
>-                            <a id="staged-and-ready-tab"  href="#staged-and-ready" role="tab" data-bs-toggle="tab">Staged & ready ([% staged_and_ready.count | html %])</a>
>+                            <a id="staged-and-ready-tab" class="nav-link" href="#staged-and-ready" role="tab" data-bs-toggle="tab">Staged & ready ([% staged_and_ready.count | html %])</a>
>                         </li>
>                         [% IF tab == 'patron-is-outside' %]
>                         <li role="presentation" class="active">
>                         [% ELSE %]
>                         <li role="presentation">
>                         [% END %]
>-                            <a id="patron-is-outside-tab" href="#patron-is-outside" role="tab" data-bs-toggle="tab">Patron is outside ([% patron_outside.count | html %])</a>
>+                            <a id="patron-is-outside-tab" class="nav-link" href="#patron-is-outside" role="tab" data-bs-toggle="tab">Patron is outside ([% patron_outside.count | html %])</a>
>                         </li>
>                         [% IF tab == 'delivered-today' %]
>                         <li role="presentation" class="active">
>                         [% ELSE %]
>                         <li role="presentation">
>                         [% END %]
>-                            <a id="delivered-today-tab"   href="#delivered-today" role="tab" data-bs-toggle="tab">Delivered today ([% delivered_today.count | html %])</a>
>+                            <a id="delivered-today-tab" class="nav-link"  href="#delivered-today" role="tab" data-bs-toggle="tab">Delivered today ([% delivered_today.count | html %])</a>
>                         </li>
>                         [% IF tab == 'schedule-pickup' %]
>                         <li role="presentation" class="active">
>                         [% ELSE %]
>                         <li role="presentation">
>                         [% END %]
>-                            <a id="schedule-pickup-tab"   href="#schedule-pickup" role="tab" data-bs-toggle="tab">Schedule pickup</a>
>+                            <a id="schedule-pickup-tab" class="nav-link" href="#schedule-pickup" role="tab" data-bs-toggle="tab">Schedule pickup</a>
>                         </li>
>                     </ul>
> 
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 37761: 170828 | 170833 | 171023 | 171029 | 171098 | 171415 | 174643 | 174644 | 174645