Patch for issue #2 Shouldn't alter location.hash if no default target…

… id set
ucsf-ckm · May 31, 2012 · 7dcd038 · 7dcd038
1 parent b605f94
commit 7dcd038
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 4 deletions.
diff --git a/LightningTouch.js b/LightningTouch.js
@@ -166,7 +166,7 @@
             }
         }
 
-        if (!location.hash) {
+        if ((!location.hash) && (defaultTargetId!=='')) {
             location.hash = '#/' + defaultTargetId;
         }
         showContent(location.hash.substring(2), [defaultTargetId]);

diff --git a/demo/LightningTouch-1.0.1.min.js → demo/LightningTouch-1.0.2.min.js b/demo/LightningTouch-1.0.1.min.js → demo/LightningTouch-1.0.2.min.js
diff --git a/demo/index.html b/demo/index.html
@@ -1,7 +1,7 @@
 <!DOCTYPE html>
 <title>Lightning Touch Demos</title>
 <meta name="viewport" content="width=device-width">
-<script src="LightningTouch-1.0.1.min.js"></script>
+<script src="LightningTouch-1.0.2.min.js"></script>
 
 <!-- 
     Function for onclick() handling so it can be compared to Lightning Touch.
@@ -53,4 +53,4 @@ <h2>That was slow.</h2>
             <a href="#" onclick="swap('onclick-main','less-awesome'); return false;">Tap to reset onclick listener.</a>
 
         </div>
-    </section>
+    </section>
diff --git a/tests/no_default_hash.html b/tests/no_default_hash.html
@@ -0,0 +1,9 @@
+<!DOCTYPE html>
+<title>Lightning Touch Test</title>
+<meta name="viewport" content="width=device-width">
+<script src="../LightningTouch.js"></script>
+
+<body>
+    <p>There should be no hash inserted in the URL on this page.</p>
+</body>
+