Attachment 145930 Details for Bug 32771 – Bug 32771: Standardize structure around action fieldsets in serials

[patch] Bug 32771: Standardize structure around action fieldsets in serials

Bug-32771-Standardize-structure-around-action-fiel.patch (text/plain), 3.96 KB, created by Owen Leonard on 2023-02-01 11:45:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2023-02-01 11:45:06 UTC

Size: 3.96 KB

patch

obsolete

>From e0a6cb4a4b6f74d1f3d799c356126657c7c3a71f Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Mon, 30 Jan 2023 16:06:01 +0000
>Subject: [PATCH] Bug 32771: Standardize structure around action fieldsets in
> serials
>
>This patch updates serials templates so that fieldsets with the
>"action" class are placed outside the form's main fieldset.
>
>To test, apply the patch and rebuild the staff interface CSS. Go to
>serials and check the following pages to confirm that changes to
>form structure look correct:
>
>- Navigate directly to /cgi-bin/koha/serials/serials-search.pl
>  - Check the form on this page.
>  - Perform a subscription search. On the search results page, check
>    the sidebar form.
>- Click "Manage numbering patterns" in the left-hand sidebar menu.
>  - Click "New numbering pattern" and check the forms on that page.
>---
> .../intranet-tmpl/prog/en/includes/subscriptions-search.inc | 6 +++---
> .../intranet-tmpl/prog/en/modules/serials/serials-search.tt | 6 +++---
> .../prog/en/modules/serials/subscription-numberpatterns.tt  | 6 +++---
> 3 files changed, 9 insertions(+), 9 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/subscriptions-search.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/subscriptions-search.inc
>index fd1222ea33..5e1275d249 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/subscriptions-search.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/subscriptions-search.inc
>@@ -43,10 +43,10 @@
>             [% IF (basketno) %]
>                 <input type="hidden" name="basketno" value="[% basketno | html %]" />
>             [% END %]
>-            <fieldset class="action">
>-              <input type="submit" class="btn btn-primary" value="Search" />
>-            </fieldset>
>             </div>
>         </fieldset>
>+        <fieldset class="action">
>+            <input type="submit" class="btn btn-primary" value="Search" />
>+        </fieldset>
>     </form>
> </div>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-search.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-search.tt
>index dc43ca6544..2fc1b48464 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-search.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-search.tt
>@@ -300,10 +300,10 @@
>                                     [% IF ( mana ) %]
>                                         <input type="hidden" name="mana" value="1" />
>                                     [% END %]
>-                                    <fieldset class="action">
>-                                        <input type="submit" class="btn btn-primary" value="Search" />
>-                                    </fieldset>
>                                 </fieldset> 
>+                                <fieldset class="action">
>+                                    <input type="submit" class="btn btn-primary" value="Search" />
>+                                </fieldset>
>                             </form>
>                         </div> 
>                     [% END # /UNLESS ( done_searched )%]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-numberpatterns.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-numberpatterns.tt
>index 3f94b03c3f..79f22ccc19 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-numberpatterns.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-numberpatterns.tt
>@@ -249,11 +249,11 @@
>                   </tr>
>                 </tbody>
>               </table>
>-              <fieldset class="action">
>-                  <button type="button" id="test_pattern" class="btn btn-default">Test pattern</button>
>-              </fieldset>
>               <div id="predictionpattern"></div>
>             </fieldset>
>+            <fieldset class="action">
>+                <button type="button" id="test_pattern" class="btn btn-default">Test pattern</button>
>+            </fieldset>
>           </form>
> 
>       [% ELSE %]
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 32771: 145930 | 146038 | 146449 | 147362 | 148514