From: Michał Gołębiowski Date: Thu, 5 Feb 2015 00:41:10 +0000 (+0100) Subject: CSS: Fix the pixelMarginRight support test in Android 2.3 X-Git-Tag: 3.0.0-alpha1~123 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=cdfc2d092afad5a3e5b3978b04b402a1ee9dce79;p=jquery.git CSS: Fix the pixelMarginRight support test in Android 2.3 --- diff --git a/src/css/curCSS.js b/src/css/curCSS.js index f5c85ecc0..aba57ad37 100644 --- a/src/css/curCSS.js +++ b/src/css/curCSS.js @@ -25,12 +25,6 @@ function curCSS( elem, name, computed ) { ret = jQuery.style( elem, name ); } - // Support: Android 4.0-4.3 - // A tribute to the "awesome hack by Dean Edwards" - // Android Browser returns percentage for some values, - // but width seems to be reliably pixels. - // This is against the CSSOM draft spec: - // http://dev.w3.org/csswg/cssom/#resolved-values if ( !support.pixelMarginRight() && rnumnonpx.test( ret ) && rmargin.test( name ) ) { // Remember the original values diff --git a/src/css/support.js b/src/css/support.js index 3165af87d..b26cc568e 100644 --- a/src/css/support.js +++ b/src/css/support.js @@ -62,8 +62,15 @@ define([ return boxSizingReliableVal; }, pixelMarginRight: function() { + // Support: Android 4.0-4.3 + // A tribute to the "awesome hack by Dean Edwards" + // Android Browser returns percentage for some values, + // but width seems to be reliably pixels. + // This is against the CSSOM draft spec: + // http://dev.w3.org/csswg/cssom/#resolved-values if ( pixelMarginRightVal == null ) { - div.style.cssText = "display:block;width:50%;margin-right:50%"; + div.style.cssText = "display:block;width:50%;border:0;margin:0;padding:0;" + + "margin-right:50%"; documentElement.appendChild( container ); pixelMarginRightVal = window.getComputedStyle( div, null ).marginRight === "4px";