fix polyfill detection for @hidden

Author: bboyle

Readme.md

@@ -66,15 +66,7 @@ Elements are hidden by toggling the HTML5 `hidden` attribute. Modern browsers su
 }
 ```
 
-Hopefully you're beyond needing IE6 support, but just in case:
-
-```css
-* {
-    _display: expression(this.hidden?'none':'inherit');
-}
-```
-
-However, the script will check whether 'hidden' elements are visible. If they are, it will use the jQuery `.toggle()` method to control relevance. This results in the style `display` being change to `none` when elements are not relevant.
+However, the script will check whether 'hidden' elements remain visible. If they are, it will use the jQuery `.hide()` and `.slideDown()` methods to control relevance. This results in the style `display` being changed to `none` when elements are not relevant, and provides a fallback animation.
 
 You can override `hidden` in modern browsers to implement CSS transitions. For example:
 

dist/relevance.js

@@ -1,4 +1,4 @@
-/*! relevance - v2.0.0 - 2015-02-26
+/*! relevance - v2.0.0 - 2015-02-27
 * https://github.com/bboyle/relevance
 * Copyright (c) 2015 Ben Boyle; Licensed MIT */
 if ( jQuery !== 'undefined' ) {
@@ -9,10 +9,10 @@ if ( jQuery !== 'undefined' ) {
 			irrelevantEvent = 'irrelevant',
 			elementsToDisable = 'button, input, select, textarea',
 			polyfillHidden = (function() {
-				var hidden = $( '<div hidden/>' );
+				var hidden = $( '<div hidden></div>' );
 				var isHidden = hidden.appendTo( 'body' ).is( ':visible' );
 				hidden.remove();
-				return isHidden;
+				return ! isHidden;
 			}()),
 
 			formElementsByName = function( form, name ) {

dist/relevance.min.js

@@ -7,10 +7,10 @@ if ( jQuery !== 'undefined' ) {
 			irrelevantEvent = 'irrelevant',
 			elementsToDisable = 'button, input, select, textarea',
 			polyfillHidden = (function() {
-				var hidden = $( '<div hidden/>' );
+				var hidden = $( '<div hidden></div>' );
 				var isHidden = hidden.appendTo( 'body' ).is( ':visible' );
 				hidden.remove();
-				return isHidden;
+				return ! isHidden;
 			}()),
 
 			formElementsByName = function( form, name ) {

src/relevance.js

@@ -26,10 +26,6 @@
 			color: #111;
 		}
 
-		[hidden] {
-			display: none;
-		}
-
 		#qunit-fixture {
 			position: static;
 		}

test/events.html

@@ -36,13 +36,6 @@
 			font-weight: bold;
 		}
 
-		[hidden] {
-			display: none;
-		}
-		* {
-			_display: expression(this.hidden?'none':'inherit');
-		}
-
 		#qunit-fixture {
 			position: static;
 		}

test/instructions.html

@@ -74,7 +74,7 @@
 			// start qunit
 			start();
 
-			strictEqual( $( '#foo' ).is( '[hidden]' ), true, '#foo is hidden' );
+			strictEqual( $( '#foo' ).is( ':hidden' ), true, '#foo is hidden' );
 			strictEqual( $( '#foo' ).attr( 'aria-hidden' ), 'true', '#foo is aria-hidden when irrelevant' );
 			ok( $( '#foo' ).attr( 'hidden' ), '#foo is @hidden when irrelevant' );
 			strictEqual( $( '#foo' )[ 0 ].disabled, true, '#foo is @disabled when irrelevant' );

test/plugin.js

@@ -51,13 +51,6 @@
 			background: black;
 		}
 
-		[hidden] {
-			display: none;
-		}
-		* {
-			_display: expression(this.hidden?'none':'inherit');
-		}
-
 		#qunit-fixture {
 			position: static;
 		}