Attachment 82334 Details for Bug 20809 – Bug 20809: (follow-up) Corrections to CSS and JS

[patch] Bug 20809: (follow-up) Corrections to CSS and JS

Bug-20809-follow-up-Corrections-to-CSS-and-JS.patch (text/plain), 5.55 KB, created by Owen Leonard on 2018-11-14 13:48:47 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2018-11-14 13:48:47 UTC

Size: 5.55 KB

patch

obsolete

>From a45d946afd5e8bfc5c54a6c2b70e1fcc9b1d02bd Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Wed, 12 Sep 2018 16:33:48 +0000
>Subject: [PATCH] Bug 20809: (follow-up) Corrections to CSS and JS
>
>This follow-up modifies JavaScript so that the confirmation dialog is
>triggered when deleting a patron image from the modal window.
>
>The CSS is modified to improve the alignment of patron image and edit
>button.
>---
> koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss   |  5 +++--
> .../intranet-tmpl/prog/en/includes/str/members-menu.inc  | 15 ++++++++++++++-
> .../intranet-tmpl/prog/en/modules/members/moremember.tt  | 16 ++--------------
> 3 files changed, 19 insertions(+), 17 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss b/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss
>index 26e4884..30585cb 100644
>--- a/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss
>+++ b/koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss
>@@ -1698,15 +1698,16 @@ dd {
> }
> 
> .patronimage {
>+    border: 1px solid #EEE;
>     display: block;
>     max-width: 160px;
>+    margin: auto;
>     opacity: 1;
>     transition: .2s ease;
> 
>-    .empty {
>+    &.empty {
>         background: transparent url( "../img/patron-blank.min.svg" ) center 5px no-repeat;
>         height: 125px;
>-        margin: .3em 0 .3em .3em;
>         padding: 0;
>         width: 80%;
>     }
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/str/members-menu.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/str/members-menu.inc
>index 703930c..2d5ebfe 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/str/members-menu.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/str/members-menu.inc
>@@ -2,7 +2,7 @@
> [% USE Koha %]
> [% USE Categories %]
> 
>-<script type="text/javascript">
>+<script>
>     var advsearch = "[% advsearch | html %]";
>     var destination = "[% destination | html %]";
> 
>@@ -20,4 +20,17 @@
>             number_of_adult_categories = [% number_of_adult_categories | html %]
>         [% END %]
>     [% END %]
>+
>+    [% IF ( patron.image ) %]
>+        $("body").on("click", "#delpicture", function(){
>+             return confirm(_("Are you sure you want to delete this patron image? This cannot be undone."));
>+        });
>+        $('#manage-patron-image').find("input[value*=Upload]").click(function(){
>+            if($("#uploadfile").val() == ""){
>+                alert(_("Please choose a file to upload"));
>+                return false;
>+            }
>+            return confirm(_("Are you sure you want to replace the current patron image? This cannot be undone."));
>+        });
>+    [% END %]
> </script>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
>index 6e15e3c..f56341f 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
>@@ -269,9 +269,9 @@
>                                         [% END %]
>                                         <form method="post" id="picture-upload" style="display:none;" action="/cgi-bin/koha/tools/picture-upload.pl" enctype="multipart/form-data">
>                                             [% IF ( patron.image ) %]
>-                                                <div class="hint">To update the image for [% patron.title | html %] [% patron.surname | html %], select a new image file and click 'Upload.' <br />Click the 'Delete' button to remove the current image.</div>
>+                                                <div class="hint">To update the image for [% patron.title %] [% patron.firstname | html %] [% patron.surname | html %], select a new image file and click 'Upload.' <br />Click the 'Delete' button to remove the current image.</div>
>                                             [% ELSE %]
>-                                                <div class="hint">[% patron.title | html %] [% patron.firstname | html %] [% patron.surname | html %] does not currently have an image available. To import an image for [% patron.title | html %] [% patron.surname | html %], enter the name of an image file to upload.</div>
>+                                                <div class="hint">[% patron.title | html %] [% patron.firstname | html %] [% patron.surname | html %] does not currently have an image available. To import an image for [% patron.title | html %] [% patron.firstname | html %] [% patron.surname | html %], enter the name of an image file to upload.</div>
>                                             [% END %]
>                                             <p>Only PNG, GIF, JPEG, XPM formats are supported.</p>
>                                             <label for="uploadfile">Select the file to upload: </label>
>@@ -857,18 +857,6 @@
>                 ],
>                 "bPaginate": false
>             }));
>-            [% IF ( patron.image ) %]
>-                $('#delpicture').click(function(){
>-                     return confirm(_("Are you sure you want to delete this patron image? This cannot be undone."));
>-                });
>-                $('#manage-patron-image').find("input[value*=Upload]").click(function(){
>-                    if($("#uploadfile").val() == ""){
>-                        alert(_("Please choose a file to upload"));
>-                        return false;
>-                    }
>-                    return confirm(_("Are you sure you want to replace the current patron image? This cannot be undone."));
>-                });
>-            [% END %]
> 
>             $("#suspend_until").datepicker({
>                 onClose: function(dateText, inst) {
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 20809: 75514 | 75515 | 75637 | 75963 | 77462 | 77465 | 77789 | 77790 | 78409 | 78410 | 78595 | 78670 | 78711 | 82333 | 82334 | 82335 | 83711 | 83712 | 83713 | 83844 | 83845 | 83846 | 83847 | 84477 | 84478 | 84479 | 84480 | 84481 | 84482 | 84483 | 84484