Attachment 37399 Details for Bug 9120 – Bug 9120: Prevent form submission on adding tags - OPAC

Bug 9120: Prevent form submission on adding tags - OPAC

Bug-9120-Prevent-form-submission-on-adding-tags---.patch (text/plain), 2.74 KB, created by Aleisha Amohia on 2015-03-31 03:48:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Aleisha Amohia

Created: 2015-03-31 03:48:04 UTC

Size: 2.74 KB

patch

obsolete

>From 7fa67b2f83f418df0923caaeec444b6a8f728b21 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@biblibre.com>
>Date: Wed, 18 Mar 2015 17:21:42 +0100
>Subject: [PATCH] Bug 9120: Prevent form submission on adding tags - OPAC
>
>At the OPAC, if the user adds tags pressing 'Enter', the page is
>reloaded and the tags are not added.
>To prevent that, this patch suggests to block the "press enter" event on
>the inputs.
>
>Test plan:
>0/ Confirm the bad behavior, without this patch.
>1/ Apply this patch and confirm nothing happen on pressing enter when
>the focus is on the add tag input (At the opac, on the result view).
>
>Patch works as expected.
>Signed-off-by: Marc Veron <veron@veron.ch>
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
>---
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt |    6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>index 2622b59..d01ddeb 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>@@ -482,7 +482,7 @@
>                                                                 <span class="actions"><a class="tag_add" id="tag_add[% SEARCH_RESULT.biblionumber %]" href="#">Add tag</a></span>
>                                                                 <div id="tagform[% SEARCH_RESULT.biblionumber %]" class="tag_results_input" style="display:none;">
>                                                                         <label for="newtag[% SEARCH_RESULT.biblionumber %]">New tag(s), separated by a comma:</label>
>-                                                                        <input type="text" name="newtag[% SEARCH_RESULT.biblionumber %]" id="newtag[% SEARCH_RESULT.biblionumber %]" maxlength="100" />
>+                                                                        <input type="text" name="newtag[% SEARCH_RESULT.biblionumber %]" id="newtag[% SEARCH_RESULT.biblionumber %]" class="newtag" maxlength="100" />
>                                                                         <input name="tagbutton" class="btn btn-small tagbutton" title="[% SEARCH_RESULT.biblionumber %]" type="submit" value="Add" />
>                                                                         <a class="cancel_tag_add" id="cancel[% SEARCH_RESULT.biblionumber %]" href="#">(done)</a>
>                                                                 </div>
>@@ -915,6 +915,10 @@ $(document).ready(function(){
>         });
> [% END %]
> 
>+$("input.newtag").on('keydown', function(e){
>+    if (e.keyCode == 13) { e.preventDefault(); }
>+});
>+
> });
> //]]>
> </script>
>-- 
>1.7.10.4

Actions: View

Attachments on bug 9120: 36991 | 37043 | 37071 | 37399