Attachment 153499 Details for Bug 34196 – Bug 34196: Improve styling of sidebar form on funds admin page

[patch] Bug 34196: Improve styling of sidebar form on funds admin page

Bug-34196-Improve-styling-of-sidebar-form-on-funds.patch (text/plain), 3.06 KB, created by Owen Leonard on 2023-07-14 18:16:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2023-07-14 18:16:54 UTC

Size: 3.06 KB

patch

obsolete

>From f89ef5c448d83b5d87fdae0e2add7719fbbc192c Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 14 Jul 2023 18:11:04 +0000
>Subject: [PATCH] Bug 34196: Improve styling of sidebar form on funds admin
> page
>
>This patch makes a few markup changes to the sidebar filter form on the
>funds administration page in order to make it more usable and more
>consistent with other similar forms.
>
>The patch removes the link for expanding and collapsing the sidebar
>filter because it looked bad and was inconsistent with all similar
>forms.
>
>To test, apply the patch and go to Administration -> Funds. Confirm that
>the sidebar form looks correct. Compare to, for example, the sidebar
>form at Acquisitions -> Invoices.
>---
> .../prog/en/modules/admin/aqbudgets.tt        | 26 ++++++++-----------
> 1 file changed, 11 insertions(+), 15 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgets.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgets.tt
>index 23e84e043e9..65385579b6f 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgets.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgets.tt
>@@ -433,7 +433,6 @@
> 
> [% IF op == 'list' %]
>   <form action="/cgi-bin/koha/admin/aqbudgets.pl" method="get">
>-    <a href="#" id="filterbutton">Filters</a>
>     <fieldset class="brief" id="fundfilters">
>     <h4>Fund filters</h4>
>         <ol>
>@@ -444,15 +443,6 @@
>                 [% PROCESS options_for_libraries libraries => Branches.all( selected => selected_branchcode, unfiltered => 1 ) %]
>             </select>
>         </li>
>-        <li class="radio">
>-
>-        <label for="show_mine">Show my funds only:</label>
>-            [% IF ( show_mine ) %]
>-                <input type="checkbox" id="show_mine"  name="show_mine" value="1" checked="checked" />
>-            [% ELSE %]
>-                <input type="checkbox" id="show_mine"  name="show_mine" value="1" />
>-        [% END %]
>-        </li>
> 
>         [% IF periods %]
>           <li>
>@@ -469,8 +459,18 @@
>             </select>
>           </li>
>         [% END %]
>-        </ol>
>+        <li class="radio">
> 
>+        <label for="show_mine">Show my funds only:</label>
>+            [% IF ( show_mine ) %]
>+                <input type="checkbox" id="show_mine"  name="show_mine" value="1" checked="checked" />
>+            [% ELSE %]
>+                <input type="checkbox" id="show_mine"  name="show_mine" value="1" />
>+        [% END %]
>+        </li>
>+        </ol>
>+    </fieldset>
>+    <fieldset class="action">
>         <input type="hidden" name="op" value="list" />
>         <input type="submit" class="submit" name="filter" value="Go" />
>     </fieldset>
>@@ -717,10 +717,6 @@
>                     oTable.fnAddFilters("filter", 750);
>                 [% END %]
> 
>-                $("#filterbutton").click(function() {
>-                    $("#fundfilters").slideToggle(0);
>-                });
>-
>                 $(".deletefund-disabled").tooltip().on("click", function(e){
>                     e.preventDefault();
>                     alert(_("This fund has sub funds. It cannot be deleted."));
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 34196: 153021 | 153499 | 153511 | 155077