You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mm...@apache.org on 2007/06/22 05:35:18 UTC

svn commit: r549695 - /myfaces/tomahawk/trunk/sandbox/core/src/main/resources/org/apache/myfaces/custom/ppr/resource/ppr.js

Author: mmarinschek
Date: Thu Jun 21 20:35:17 2007
New Revision: 549695

URL: http://svn.apache.org/viewvc?view=rev&rev=549695
Log:
TOMAHAWK-1028: enable inputs submitting a ppr-panelgroup - in a data-table as well; make sure inputs also work in IE, adding comments

Modified:
    myfaces/tomahawk/trunk/sandbox/core/src/main/resources/org/apache/myfaces/custom/ppr/resource/ppr.js

Modified: myfaces/tomahawk/trunk/sandbox/core/src/main/resources/org/apache/myfaces/custom/ppr/resource/ppr.js
URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/sandbox/core/src/main/resources/org/apache/myfaces/custom/ppr/resource/ppr.js?view=diff&rev=549695&r1=549694&r2=549695
==============================================================================
--- myfaces/tomahawk/trunk/sandbox/core/src/main/resources/org/apache/myfaces/custom/ppr/resource/ppr.js (original)
+++ myfaces/tomahawk/trunk/sandbox/core/src/main/resources/org/apache/myfaces/custom/ppr/resource/ppr.js Thu Jun 21 20:35:17 2007
@@ -372,9 +372,12 @@
 
 org.apache.myfaces.PPRCtrl.prototype._addEventHandler = function (formElement) {
     if (this._isButton(formElement) || this._isCheckbox(formElement) || this._isRadio(formElement)) {
+        //for these element-types, onclick is appropriate
         this._addOnClickHandler(formElement);
     }
     else if(this._isText(formElement) || this._isDropdown(formElement)) {
+        //for these element-types, onblur will be working.
+        //attention: onchange won't work properly in IE6 at least (field will never loose focus)
         this._addOnBlurHandler(formElement);
     }
 }