Attachment 18317 Details for Bug 10310 – Bug 10310: Prevent submitting form with enter does not work with IE

[patch] Bug 10310: Prevent submitting form with enter does not work with IE

Bug-10310-Prevent-submitting-form-with-enter-does-.patch (text/plain), 1.65 KB, created by Jonathan Druart on 2013-05-22 14:40:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2013-05-22 14:40:02 UTC

Size: 1.65 KB

patch

obsolete

>From 8f61a69de267685c2e386bbdc9fe1429da227944 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@biblibre.com>
>Date: Wed, 22 May 2013 15:49:26 +0200
>Subject: [PATCH] Bug 10310: Prevent submitting form with enter does not work
> with IE
>
>I am not able to test this patch with IE...
>I tested it with Chromium and FF and it works great.
>
>This patch can be tested on the neworderempty.pl page
>---
> .../intranet-tmpl/prog/en/js/prevent_submit.js     |   27 ++++++++------------
> 1 file changed, 10 insertions(+), 17 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/js/prevent_submit.js b/koha-tmpl/intranet-tmpl/prog/en/js/prevent_submit.js
>index bafd42a..2e6d385 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/js/prevent_submit.js
>+++ b/koha-tmpl/intranet-tmpl/prog/en/js/prevent_submit.js
>@@ -1,18 +1,11 @@
>-var prevent_nav = window.Event ? true : false;
>-if (prevent_nav) {
>-    window.captureEvents(Event.KEYDOWN);
>-    window.onkeydown = NetscapeEventHandler_KeyDown;
>-} else {
>-    document.onkeydown = IEEventHandler_KeyDown;
>-}
>-
>-function NetscapeEventHandler_KeyDown(e) {
>-    if (e.which == 13 && e.target.type != 'textarea' && e.target.type != 'submit') { return false; }
>-    return true;
>-}
>-
>-function IEEventHandler_KeyDown() {
>-    if (event.keyCode == 13 && event.srcElement.type != 'textarea' && event.srcElement.type != 'submit')
>+$(document).ready(function() {
>+  // We don't want to apply this for the search form
>+  $("#doc3 form").keypress(function (e) {
>+    if( e.which == 13
>+        && e.target.nodeName == "INPUT"
>+        && $(e.target).attr('type') != "submit"
>+    ) {
>         return false;
>-    return true;
>-}
>+    }
>+  });
>+});
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 10310: 18314 | 18316 | 18317 | 18324 | 18326 | 18329 | 18330