exposed w.overthrow.enabledClassName to fix a failing test

filamentgroup · Aug 2, 2013 · 5853220 · 5853220
1 parent a862f1b
commit 5853220
Show file tree

Hide file tree

Showing 6 changed files with 16 additions and 6 deletions.
diff --git a/README.md b/README.md
@@ -79,6 +79,7 @@ Overthrow exposes various properties you can access via the `overthrow` object:
 
 Many `overthrow` settings are exposed and configurable via the `overthrow` object. If needed, you can redefine these before calling `set()`.
 
+- `overthrow.enabledClassName`: The class name added to the `html` element in supported browsers. Default is `overthrow-enabled`
 - `overthrow.scrollIndicatorClassName`: The class name used to identify scrollable `overthrow` elements. Default is `overthrow`
 
 ## Methods

diff --git a/dist/overthrow.js b/dist/overthrow.js
@@ -57,6 +57,8 @@
 	// Expose overthrow API
 	w.overthrow = {};
 
+	w.overthrow.enabledClassName = enabledClassName;
+
 	w.overthrow.addClass = function(){
 		if( docElem.className.indexOf( enabledClassName ) === -1 ){
 			docElem.className += " " + enabledClassName;

diff --git a/dist/overthrow.sidescroller.js b/dist/overthrow.sidescroller.js
@@ -57,6 +57,8 @@
 	// Expose overthrow API
 	w.overthrow = {};
 
+	w.overthrow.enabledClassName = enabledClassName;
+
 	w.overthrow.addClass = function(){
 		if( docElem.className.indexOf( enabledClassName ) === -1 ){
 			docElem.className += " " + enabledClassName;

diff --git a/src/overthrow-detect.js b/src/overthrow-detect.js
@@ -55,14 +55,16 @@
 	// Expose overthrow API
 	w.overthrow = {};
 
+	w.overthrow.enabledClassName = enabledClassName;
+
 	w.overthrow.addClass = function(){
-		if( docElem.className.indexOf( enabledClassName ) === -1 ){
-			docElem.className += " " + enabledClassName;
+		if( docElem.className.indexOf( w.overthrow.enabledClassName ) === -1 ){
+			docElem.className += " " + w.overthrow.enabledClassName;
 		}
 	};
 
 	w.overthrow.removeClass = function(){
-		docElem.className = docElem.className.replace( enabledClassName, "" );
+		docElem.className = docElem.className.replace( w.overthrow.enabledClassName, "" );
 	};
 
 	// Enable and potentially polyfill overflow

diff --git a/test/index.html b/test/index.html
@@ -6,7 +6,10 @@
 	<link rel="stylesheet" href="qunit/qunit.css" media="screen">
 	<script src="qunit/qunit.js"></script>
 	<link href="test.css" rel="stylesheet" />
-	<script src="../overthrow.js"></script>
+	<script src="../src/overthrow-detect.js"></script>
+	<script src="../src/overthrow-polyfill.js"></script>
+	<script src="../src/overthrow-toss.js"></script>
+	<script src="../src/overthrow-init.js"></script>
 	<script src="tests.js"></script>
 
 </head>

diff --git a/test/tests.js b/test/tests.js
@@ -74,7 +74,7 @@ window.onload = function(){
 
 		test( 'overthrow.set adds HTML class with custom name', function() { 
 			overthrow.forget();
-			overthrow.configure({enabledClassName: "prefixed-overthrow-enabled"});
+			overthrow.enabledClassName = "prefixed-overthrow-enabled";
 			overthrow.set();
 			ok( document.documentElement.className.indexOf( "prefixed-overthrow-enabled" ) > -1 );
 		});
@@ -84,7 +84,7 @@ window.onload = function(){
 			ok( document.documentElement.className.indexOf( "prefixed-overthrow-enabled" ) === -1 );
 
 			//since the following tests assume the class name to be the default, we set it back
-			overthrow.configure({enabledClassName: "overthrow-enabled"});
+			overthrow.enabledClassName = "overthrow-enabled";
 			overthrow.set();
 		});