Added css hook to work around bug in WebKit computed margin-right. Fixes #3333 -...

[jquery.git] / src / support.js

diff --git a/src/support.js b/src/support.js
index 7be28fd..939ad4f 100644 (file)
--- a/src/support.js
+++ b/src/support.js
@@ -12,7 +12,8 @@
        var all = div.getElementsByTagName("*"),
                a = div.getElementsByTagName("a")[0],
                select = document.createElement("select"),
-               opt = select.appendChild( document.createElement("option") );
+               opt = select.appendChild( document.createElement("option") ),
+               input = div.getElementsByTagName("input")[0];
 
        // Can't get basic test support
        if ( !all || !all.length || !a ) {
@@ -51,7 +52,7 @@
                // Make sure that if no value is specified for a checkbox
                // that it defaults to "on".
                // (WebKit defaults to "" instead)
-               checkOn: div.getElementsByTagName("input")[0].value === "on",
+               checkOn: input.value === "on",
 
                // Make sure that a selected-by-default option has a working selected property.
                // (WebKit defaults to false instead of true, IE too, if it's in an optgroup)
@@ -61,26 +62,30 @@
                deleteExpando: true,
                optDisabled: false,
                checkClone: false,
-               _scriptEval: null,
                noCloneEvent: true,
+               noCloneChecked: true,
                boxModel: null,
                inlineBlockNeedsLayout: false,
                shrinkWrapBlocks: false,
-               reliableHiddenOffsets: true
+               reliableHiddenOffsets: true,
+               reliableMarginRight: true
        };
 
+       input.checked = true;
+       jQuery.support.noCloneChecked = input.cloneNode( true ).checked;
+
        // Make sure that the options inside disabled selects aren't marked as disabled
        // (WebKit marks them as diabled)
        select.disabled = true;
        jQuery.support.optDisabled = !opt.disabled;
 
+       var _scriptEval = null;
        jQuery.support.scriptEval = function() {
-               if ( jQuery.support._scriptEval === null ) {
+               if ( _scriptEval === null ) {
                        var root = document.documentElement,
                                script = document.createElement("script"),
                                id = "script" + jQuery.now();
 
-                       script.type = "text/javascript";
                        try {
                                script.appendChild( document.createTextNode( "window." + id + "=1;" ) );
                        } catch(e) {}
@@ -91,10 +96,10 @@
                        // tag with appendChild/createTextNode
                        // (IE doesn't support this, fails, and uses .text instead)
                        if ( window[ id ] ) {
-                               jQuery.support._scriptEval = true;
+                               _scriptEval = true;
                                delete window[ id ];
                        } else {
-                               jQuery.support._scriptEval = false;
+                               _scriptEval = false;
                        }
 
                        root.removeChild( script );
@@ -102,7 +107,7 @@
                        root = script = id  = null;
                }
 
-               return jQuery.support._scriptEval;
+               return _scriptEval;
        };
 
        // Test to see if it's possible to delete an expando from an element
@@ -114,7 +119,7 @@
                jQuery.support.deleteExpando = false;
        }
 
-       if ( div.attachEvent && div.fireEvent ) {
+       if ( !div.addEventListener && div.attachEvent && div.fireEvent ) {
                div.attachEvent("onclick", function click() {
                        // Cloning a node shouldn't copy over any
                        // bound event handlers (IE does this)
@@ -136,10 +141,16 @@
        // Figure out if the W3C box model works as expected
        // document.body must exist before we can do this
        jQuery(function() {
-               var div = document.createElement("div");
-               div.style.width = div.style.paddingLeft = "1px";
+               var div = document.createElement("div"),
+                       body = document.getElementsByTagName("body")[0];
+
+               // Frameset documents with no body should not run this code
+               if ( !body ) {
+                       return;
+               }
 
-               document.body.appendChild( div );
+               div.style.width = div.style.paddingLeft = "1px";
+               body.appendChild( div );
                jQuery.boxModel = jQuery.support.boxModel = div.offsetWidth === 2;
 
                if ( "zoom" in div.style ) {
@@ -178,7 +189,18 @@
                jQuery.support.reliableHiddenOffsets = jQuery.support.reliableHiddenOffsets && tds[0].offsetHeight === 0;
                div.innerHTML = "";
 
-               document.body.removeChild( div ).style.display = "none";
+               // Check if div with explicit width and no margin-right incorrectly
+               // gets computed margin-right based on width of container. For more
+               // info see bug #3333
+               // Fails in WebKit before Feb 2011 nightlies
+               // WebKit Bug 13343 - getComputedStyle returns wrong value for margin-right
+               if ( document.defaultView && document.defaultView.getComputedStyle ) {
+                       div.style.width = "1px";
+                       div.style.marginRight = "0";
+                       jQuery.support.reliableMarginRight = ( parseInt(document.defaultView.getComputedStyle(div).marginRight, 10) || 0 ) === 0;
+               }
+
+               body.removeChild( div ).style.display = "none";
                div = tds = null;
        });