Changeset 2821

Timestamp:

08/21/07 08:25:11 (1 year ago)

Author:

jeresig

Message:

Using some of the ideas presented by rformato, I've significantly sped up $("#id") selection. It's now just 10% slower than doing: $(document.getElementById("test")), which seems quite acceptable. (Bug #1316)

Files:

• trunk/jquery/src/jquery/jquery.js (modified) (2 diffs)

trunk/jquery/src/jquery/jquery.js

@@ -35 +35 @@
 window.$ = jQuery;
 
+var quickExpr = /^[^<]*(<(.|\s)+>)[^>]*$|^#(\w+)$/;
+
 /**
  * This function accepts a string containing a CSS or
@@ -151 +153 @@
         a = a || document;
 
+        // Handle HTML strings
+        if ( typeof a  == "string" ) {
+            var m = quickExpr.exec(a);
+            if ( m && (m[1] || !c) ) {
+                // HANDLE: $(html) -> $(array)
+                if ( m[1] )
+                    a = jQuery.clean( [ m[1] ] );
+
+                // HANDLE: $("#id")
+                else {
+                    var tmp = document.getElementById( m[3] );
+                    if ( tmp )
+                        // Handle the case where IE and Opera return items
+                        // by name instead of ID
+                        if ( tmp.id != m[3] )
+                            return jQuery().find( a );
+                        else {
+                            this[0] = tmp;
+                            this.length = 1;
+                            return this;
+                        }
+                    else
+                        a = [];
+                }
+
+            // HANDLE: $(expr)
+            } else
+                return new jQuery( c ).find( a );
+
         // HANDLE: $(function)
         // Shortcut for document ready
-        if ( jQuery.isFunction(a) )
+        } else if ( jQuery.isFunction(a) )
             return new jQuery(document)[ jQuery.fn.ready ? "ready" : "load" ]( a );
-
-        // Handle HTML strings
-        if ( typeof a  == "string" ) {
-            // HANDLE: $(html) -> $(array)
-            var m = /^[^<]*(<(.|\s)+>)[^>]*$/.exec(a);
-            if ( m )
-                a = jQuery.clean( [ m[1] ] );
-
-            // HANDLE: $(expr)
-            else
-                return new jQuery( c ).find( a );
-        }
 
         return this.setArray(