Attachment 37071 Details for Bug 9120 – [PASSED QA] Bug 9120: Prevent form submission on adding tags - OPAC

[patch] [PASSED QA] Bug 9120: Prevent form submission on adding tags - OPAC

PASSED-QA-Bug-9120-Prevent-form-submission-on-addi.patch (text/plain), 2.70 KB, created by Kyle M Hall (khall) on 2015-03-20 18:13:37 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2015-03-20 18:13:37 UTC

Size: 2.70 KB

patch

obsolete

>From 786cd9d029ef470e5c49e60007ebffa3a2e7feac 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] [PASSED QA] 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>
>---
> .../opac-tmpl/bootstrap/en/modules/opac-results.tt |    6 +++++-
> 1 files changed, 5 insertions(+), 1 deletions(-)
>
>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 2d7ccf6..ec40189 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>@@ -486,7 +486,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>
>@@ -919,6 +919,10 @@ $(document).ready(function(){
>         });
> [% END %]
> 
>+$("input.newtag").on('keydown', function(e){
>+    if (e.keyCode == 13) { e.preventDefault(); }
>+});
>+
> });
> //]]>
> </script>
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

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