Remove unnecessary <a/> from the toggle button

ferrous26 · Feb 26, 2012 · 8b3e474 · 8b3e474
1 parent 16ab84f
commit 8b3e474
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/formattedJSON.js b/src/formattedJSON.js
@@ -126,7 +126,7 @@
       var toolbar = this._html( html );
       document.body.insertBefore( toolbar, document.body.firstChild );
 
-      var toggle = document.getElementById( "toolbar" ).getElementsByTagName( "a" )[0];
+      var toggle = document.getElementById( "toolbar" ).getElementsByTagName( "li" )[0];
 
       toggle.addEventListener( "click", function() {
         formatJSON._toggleClass( document.body, "before" );

diff --git a/src/proxy.html b/src/proxy.html
@@ -61,9 +61,10 @@
       -webkit-user-select: none;
       z-index: 9999;
     }
-    #toolbar a {
+    #toolbar li {
       background: -webkit-gradient( linear, 0 0, 0 100%, from(#f4f4f4), to(#bebebe) );
       color: #3f3f3f;
+      cursor: pointer;
       display: inline-block;
       font-size: 11px;
       font-weight: bold;
@@ -72,7 +73,7 @@
       text-decoration: none;
       text-shadow: 0 1px rgba(255, 255, 255, 0.8);
     }
-    #toolbar a:active {
+    #toolbar li:active {
       background: -webkit-gradient( linear, 0 0, 0 100%, from(#bebebe), to(#f4f4f4) );
     }
 
@@ -148,7 +149,7 @@
 </head>
 <body>
   <ul id="toolbar">
-    <li><a href="#">Toggle Formatting</a></li>
+    <li>Toggle Formatting</li>
   </ul>
 </body>
 </html>