Toggle the button's text & icon in a better way.

metacpan · Jul 29, 2014 · 3b4874a · 3b4874a
1 parent 9cbedb8
commit 3b4874a
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/root/static/js/cpan.js b/root/static/js/cpan.js
@@ -342,21 +342,18 @@ function starModule(form) {
             var count = 0;
             if (button.hasClass('active')) {
                 count = 1;
-		button.remove();
-		form.append('<button type="submit" class="stargazer active"><i class="icon-star"></i> Unstar</button>');
+		button.html('<i class="icon-star"></i> Unstar');
                 form.append('<input type="hidden" name="remove" value="1">');
                 if (!count){
                     button.toggleClass('highlight');
-		    button.remove();
-		    form.append('<button type="submit" class="stargazer highlight"><i class="icon-star-empty"></i> Star</button>');
+		    button.html('<i class="icon-star-empty"></i> Star');
 		}            
 		} else {
                 count=0;
                 form.find('input[name="remove"]').remove();
                 if (count === 0) {
                     button.toggleClass('highlight');
-		    button.remove();
-		    form.append('<button type="submit" class="stargazer highlight"><i class="icon-star-empty"></i> Star</button>');
+		    button.html('<i class="icon-star-empty"></i> Star');
                 }
             }
         },