updated the ruby test

Modernizr · Apr 16, 2012 · cb7f8a7 · cb7f8a7
1 parent d1f72d2
commit cb7f8a7
Showing 1 changed file with 51 additions and 54 deletions.
diff --git a/feature-detects/elem-ruby.js b/feature-detects/elem-ruby.js
@@ -1,56 +1,53 @@
-/*
- * Browser support test for the HTML5 <ruby>, <rt> and <rp> elements
- *
- * by @alrra
- *
- */
-
-Modernizr.addTest('ruby', function() {
-
-	var ruby = document.createElement('ruby'),
-		rt = document.createElement('rt'),
-		rp = document.createElement('rp'),
-		docElement = document.documentElement,
-		displayStyleProperty = 'display',
-		fontSizeStyleProperty = 'fontSize'; // 'fontSize' - because it`s only used for IE6 and IE7
-
-	ruby.appendChild(rp);
-	ruby.appendChild(rt);
-	docElement.appendChild(ruby);
-
-	// browsers that support <ruby> hide the <rp> via "display:none"
-	if((getStyle(rp,displayStyleProperty) == 'none')                                                             // for non-IE browsers
-	// but in IE browsers <rp> has "display:inline" so, the test needs other conditions:
-		|| (getStyle(ruby,displayStyleProperty) == 'ruby' && getStyle(rt,displayStyleProperty) == 'ruby-text')   // for IE8 & IE9
-		|| (getStyle(rp,fontSizeStyleProperty) == '6pt' && getStyle(rt,fontSizeStyleProperty) == '6pt')) {       // for IE6 & IE7
-
-		cleanUp();
-		return true;
-
-	} else {
-
-		cleanUp();
-		return false;
-
-	}
-
-	function getStyle(element, styleProperty) {
-		var result;
-
-		if (window.getComputedStyle) {          // for non-IE browsers
-			result = document.defaultView.getComputedStyle(element,null).getPropertyValue(styleProperty);
-		} else if (element.currentStyle) {      // for IE
-			result = element.currentStyle[styleProperty];
-		}
-
-		return result;
-	}
-
-	function cleanUp() {
-		docElement.removeChild(ruby); // the removed child node still exists in memory so ...
-		ruby = null;
-		rt = null;
-		rp = null;
-	}
+// Browser support test for the HTML5 <ruby>, <rt> and <rp> elements
+// http://www.whatwg.org/specs/web-apps/current-work/multipage/text-level-semantics.html#the-ruby-element
+//
+// by @alrra
+
+Modernizr.addTest('ruby', function () {
+
+    var ruby = document.createElement('ruby'),
+        rt = document.createElement('rt'),
+        rp = document.createElement('rp'),
+        docElement = document.documentElement,
+        displayStyleProperty = 'display',
+        fontSizeStyleProperty = 'fontSize'; // 'fontSize' - because it`s only used for IE6 and IE7
+
+    ruby.appendChild(rp);
+    ruby.appendChild(rt);
+    docElement.appendChild(ruby);
+
+    // browsers that support <ruby> hide the <rp> via "display:none"
+    if ( getStyle(rp, displayStyleProperty) == 'none'                                                          // for non-IE browsers
+    // but in IE browsers <rp> has "display:inline" so, the test needs other conditions:
+        || getStyle(ruby, displayStyleProperty) == 'ruby' && getStyle(rt, displayStyleProperty) == 'ruby-text' // for IE8 & IE9
+        || getStyle(rp, fontSizeStyleProperty) == '6pt' && getStyle(rt, fontSizeStyleProperty) == '6pt' ) {    // for IE6 & IE7
+
+        cleanUp();
+        return true;
+
+    } else {
+        cleanUp();
+        return false;
+    }
+
+    function getStyle( element, styleProperty ) {
+        var result;
+
+        if ( window.getComputedStyle ) {     // for non-IE browsers
+            result = document.defaultView.getComputedStyle(element,null).getPropertyValue(styleProperty);
+        } else if ( element.currentStyle ) { // for IE
+            result = element.currentStyle[styleProperty];
+        }
+
+        return result;
+    }
+
+    function cleanUp() {
+        docElement.removeChild(ruby);
+        // the removed child node still exists in memory, so ...
+        ruby = null;
+        rt = null;
+        rp = null;
+    }
 
 });