Author: thrantal
Date: Wed Jun 25 20:31:22 2008
New Revision: 671713

URL: http://svn.apache.org/viewvc?rev=671713&view=rev
Log:
WICKET-1713: Fixed the case where elements with stored tabIndexes are updated 
via ajax between opening and closing the ModalWindow. The DOM elements coming 
from the server don't have the custom property that was set to the instances of 
the client, so the (usually correct) tabIndex coming from the server must not 
be overwritten with null.

Modified:
    
wicket/branches/wicket-1.3.x/jdk-1.4/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/res/modal.js

Modified: 
wicket/branches/wicket-1.3.x/jdk-1.4/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/res/modal.js
URL: 
http://svn.apache.org/viewvc/wicket/branches/wicket-1.3.x/jdk-1.4/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/res/modal.js?rev=671713&r1=671712&r2=671713&view=diff
==============================================================================
--- 
wicket/branches/wicket-1.3.x/jdk-1.4/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/res/modal.js
 (original)
+++ 
wicket/branches/wicket-1.3.x/jdk-1.4/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/res/modal.js
 Wed Jun 25 20:31:22 2008
@@ -1447,8 +1447,10 @@
                                var tagElements = 
this.document.getElementsByTagName(this.tabbableTags[j]);
                                for (var k = 0 ; k < tagElements.length; k++) {
                                        var element = tagElements[k];
-                                       element.tabIndex = 
element.hiddenTabIndex;
-                                       element.hiddenTabIndex = null;
+                                       if (typeof(element.hiddenTabIndex) != 
'undefined') {
+                                               element.tabIndex = 
element.hiddenTabIndex;
+                                               element.hiddenTabIndex = null;
+                                       }
                                        element.tabEnabled = true;
                                }
                        }


Reply via email to