summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFabio Bas <ctrlaltca@gmail.com>2013-09-17 23:47:50 +0200
committerFabio Bas <ctrlaltca@gmail.com>2013-09-17 23:47:50 +0200
commitd9fc469530dd70be8dfdf337a525437774629ede (patch)
tree940992b653b47853df29df6997f55e84797b4288
parent577e690f1ce437150443fe89635b6a6ef7aab0f1 (diff)
Avoid triggering the default button of the form when pressing enter over a valid <a href='..'>; fixes #4803.2.2.03
-rw-r--r--HISTORY2
-rw-r--r--framework/Web/Javascripts/source/prado/controls/controls.js3
2 files changed, 4 insertions, 1 deletions
diff --git a/HISTORY b/HISTORY
index 66526085..286a7224 100644
--- a/HISTORY
+++ b/HISTORY
@@ -8,6 +8,8 @@ BUG: Issue #465 - Textmate editor plugin template error (ctrlaltca)
BUG: Issue #472 - No https support from wsdl generator (Marcin Piotrowski)
ENH: Issue #473 - wsdl support for additional attributes of a custom type's property (Marcin Piotrowski)
BUG: Issue #476 - Demo's don't work out of box (ctrlaltca)
+BUG: Issue #479 - TUrlMapping instanciates patterns twice (ctrlaltca)
+BUG: Issue #480 - THyperLink is not usable using keyboard navigation (david otto)
BUG: Issue #481 - Typo in composer: ext-eaccellerator (ciromattia)
BUG: Issue #482 - composer: add include path for prado.php (ciromattia)
BUG: Issue #484 - Wrong DateTimePatterns for italian culture (ciromattia)
diff --git a/framework/Web/Javascripts/source/prado/controls/controls.js b/framework/Web/Javascripts/source/prado/controls/controls.js
index fd8a4c91..8ea6afe3 100644
--- a/framework/Web/Javascripts/source/prado/controls/controls.js
+++ b/framework/Web/Javascripts/source/prado/controls/controls.js
@@ -419,9 +419,10 @@ Prado.WebUI.DefaultButton = Class.create(Prado.WebUI.Control,
{
var enterPressed = Event.keyCode(ev) == Event.KEY_RETURN;
var isTextArea = Event.element(ev).tagName.toLowerCase() == "textarea";
+ var isHyperLink = Event.element(ev).tagName.toLowerCase() == "a" && Event.element(ev).hasAttribute("href");
var isValidButton = Event.element(ev).tagName.toLowerCase() == "input" && Event.element(ev).type.toLowerCase() == "submit";
- if(enterPressed && !isTextArea && !isValidButton)
+ if(enterPressed && !isTextArea && !isValidButton && !isHyperLink)
{
var defaultButton = $(this.options['Target']);
if(defaultButton)