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

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

Bug-9120-Prevent-form-submission-on-adding-tags---.patch (text/plain), 2.56 KB, created by Jonathan Druart on 2015-03-18 16:26:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2015-03-18 16:26:56 UTC

Size: 2.56 KB

patch

obsolete

>From cc13b9a5e81cb7456d8a76e0cc823d8fbbf0dce2 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).
>---
> 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 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>
>-- 
>2.1.0

Actions: View | Diff | Splinter Review

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