[Jifty-commit] r7695 - jifty/trunk/share/web/static/js

Jifty commits jifty-commit at lists.jifty.org
Wed Dec 2 13:43:52 EST 2009


Author: c9s
Date: Wed Dec  2 13:43:51 2009
New Revision: 7695

Modified:
   jifty/trunk/share/web/static/js/jifty.js

Log:
add a note to jifty.js

Modified: jifty/trunk/share/web/static/js/jifty.js
==============================================================================
--- jifty/trunk/share/web/static/js/jifty.js	(original)
+++ jifty/trunk/share/web/static/js/jifty.js	Wed Dec  2 13:43:51 2009
@@ -1,6 +1,14 @@
 /* An empty class so we can create things inside it */
 var Jifty = {};
 
+// NOTICE: line break should be in a proper place.
+//   something like 
+//          return 
+//          {
+//              a : 123
+//          };
+//   would cause IE javascript error.
+
 // to add a response_hook
 // the handler will get two arguments:
 //
@@ -1791,11 +1799,9 @@
 
      var self = this;
 
-     jQuery( this.element )
-     .bind("focus", function(event) {
+     jQuery( this.element ).bind("focus", function(event) {
          self.onFocus();
-     })
-     .bind("blur", function(event) {
+     }).bind("blur", function(event) {
          self.onBlur();
      });
 
@@ -1907,12 +1913,11 @@
 
     if (options == null) options = {};
     // jQuery built-ins
-    var effect =
-        name == 'Fade' ? 'fadeOut' :
-        name == 'Appear' ? 'fadeIn' :
-        name == 'SlideDown' ? 'slideDown' :
-        name == 'SlideUp' ? 'slideUp' :
-        name;
+    var effect = name == 'Fade' ? 'fadeOut' :
+                    name == 'Appear' ? 'fadeIn' :
+                    name == 'SlideDown' ? 'slideDown' :
+                    name == 'SlideUp' ? 'slideUp' :
+                    name;
 
     if ( jQuery.isFunction( jQuery(el)[ effect ] ) ) {
         if ( jQuery.isFunction(options["before"]) ) 


More information about the Jifty-commit mailing list