Fix HTML for #77. Ids are unique across the dom and bad things happen…

… if they repeat.
concerto · Jul 14, 2012 · 6f38d28 · 6f38d28
1 parent 9be81a1
commit 6f38d28
Showing 1 changed file with 20 additions and 19 deletions.
diff --git a/app/views/submissions/_index_body.html.erb b/app/views/submissions/_index_body.html.erb
@@ -78,17 +78,17 @@
         <p>Currently <b>Pending Moderation</b></p>
         <div class="form-stacked">
           <h4>Moderate Content</h4>
-          <div id="moderate-select" class="btn-toolbar">
+          <div class="moderate-select" class="btn-toolbar">
             <div class="btn-group" data-toggle="buttons-radio">
               <button class="btn approve">Approve</button>
               <button class="btn deny">Deny</button>
             </div>
           </div>
 
-          <div id="moderate-true">
+          <div class="moderate-true">
             <%= render :partial => "submissions/moderate_form", :locals => { :submission => submission, :moderate => true } %>
           </div>
-          <div id="moderate-false">
+          <div class="moderate-false">
             <%= render :partial => "submissions/moderate_form", :locals => { :submission => submission, :moderate => false } %>
           </div>
         </div>
@@ -148,17 +148,17 @@
         <p>Currently <b>Pending Moderation</b></p>
         <div class="form-stacked">
           <h4>Moderate Content</h4>
-          <div id="moderate-select" class="btn-toolbar">
+          <div class="moderate-select" class="btn-toolbar">
             <div class="btn-group" data-toggle="buttons-radio">
               <button class="btn approve">Approve</button>
               <button class="btn deny">Deny</button>
             </div>
           </div>
 
-          <div id="moderate-true">
+          <div class="moderate-true">
             <%= render :partial => "submissions/moderate_form", :locals => { :submission => submission, :moderate => true } %>
           </div>
-          <div id="moderate-false">
+          <div class="moderate-false">
             <%= render :partial => "submissions/moderate_form", :locals => { :submission => submission, :moderate => false } %>
           </div>
         </div>
@@ -214,17 +214,17 @@
         <p>Currently <b>Pending Moderation</b></p>
         <div class="form-stacked">
           <h4>Moderate Content</h4>
-          <div id="moderate-select" class="btn-toolbar">
+          <div class="moderate-select" class="btn-toolbar">
             <div class="btn-group" data-toggle="buttons-radio">
               <button class="btn approve">Approve</button>
               <button class="btn deny">Deny</button>
             </div>
           </div>
 
-          <div id="moderate-true">
+          <div class="moderate-true">
             <%= render :partial => "submissions/moderate_form", :locals => { :submission => submission, :moderate => true } %>
           </div>
-          <div id="moderate-false">
+          <div class="moderate-false">
             <%= render :partial => "submissions/moderate_form", :locals => { :submission => submission, :moderate => false } %>
           </div>
         </div>
@@ -280,17 +280,17 @@
         <p>Currently <b>Pending Moderation</b></p>
         <div class="form-stacked">
           <h4>Moderate Content</h4>
-          <div id="moderate-select" class="btn-toolbar">
+          <div class="moderate-select" class="btn-toolbar">
             <div class="btn-group" data-toggle="buttons-radio">
               <button class="btn approve">Approve</button>
               <button class="btn deny">Deny</button>
             </div>
           </div>
 
-          <div id="moderate-true">
+          <div class="moderate-true">
             <%= render :partial => "submissions/moderate_form", :locals => { :submission => submission, :moderate => true } %>
           </div>
-          <div id="moderate-false">
+          <div class="moderate-false">
             <%= render :partial => "submissions/moderate_form", :locals => { :submission => submission, :moderate => false } %>
           </div>
         </div>
@@ -308,18 +308,19 @@
 <script type="text/javascript">
 
 $(document).ready(function() {
+  $(".moderate-true").hide();
+  $(".moderate-false").hide();
 
-  $("#moderate-true").hide();
-  $("#moderate-false").hide();
-
-  $(document).on("click", "#moderate-select button.approve", function(event) {
+  $(document).on("click", ".moderate-select button.approve", function(event) {
     event.preventDefault();
-    $("#moderate-true").show();
+    $(".moderate-false").hide();
+    $(".moderate-true").show();
   });
 
-  $(document).on("click", "#moderate-select button.deny", function(event) {
+  $(document).on("click", ".moderate-select button.deny", function(event) {
     event.preventDefault();
-    $("#moderate-false").hide();
+    $(".moderate-true").hide()
+    $(".moderate-false").show();
   });
 
   $('.dropdown-control').click(function(event) { event.preventDefault(); });