Update review display to show up even in HTML without AJAX

MacKLess · Dec 7, 2017 · 281fa50 · 281fa50
1 parent 7e2f6e4
commit 281fa50
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 3 deletions.
diff --git a/app/assets/stylesheets/application.scss b/app/assets/stylesheets/application.scss
@@ -35,3 +35,7 @@
 .hidelink {
   display: none;
 }
+
+.reviews {
+  display: none;
+}
diff --git a/app/views/products/_show.html.erb b/app/views/products/_show.html.erb
@@ -55,4 +55,5 @@
   <% end %>
   <h5><%= link_to "Hide Reviews", products_path, class: "hidelink", id: "hide_reviews_#{@product.id}", remote: true %></h5>
   <h5><%= link_to "Show Reviews", product_reviews_path(@product), id: "show_reviews_#{@product.id}", remote: true %></h5>
+  <%= render "reviews/index" %>
 </div>
diff --git a/app/views/reviews/_index.html.erb b/app/views/reviews/_index.html.erb
@@ -1,4 +1,4 @@
-<div id="reviews-<%= @product.id %>">
+<div class="reviews" id="reviews-<%= @product.id %>">
   <h2>Reviews</h2>
   <% if @product.reviews.any? %>
     <div id="reviews">

diff --git a/app/views/reviews/index.js.erb b/app/views/reviews/index.js.erb
@@ -1,7 +1,8 @@
-$('#show_reviews_<%= @product.id %>').hide().after("<%= j render('reviews/index') %>");
+$('#show_reviews_<%= @product.id %>').hide();
+$("#reviews-<%= @product.id %>").show();
 $('#hide_reviews_<%= @product.id %>').show().click(function(event) {
   event.preventDefault();
   $('#show_reviews_<%= @product.id %>').show();
   $('#hide_reviews_<%= @product.id %>').hide();
-  $('#reviews-<%= @product.id %>').remove();
+  $('#reviews-<%= @product.id %>').hide();
 });