Restore </li>

edit: although tag technically creates a closed tag by default, leaving in the </li> doesn't seem to hurt. However we should actually use content_tag instead show: equal number of opening and closing tags, presumably it matches something
autolab · Jun 20, 2022 · e704b09 · e704b09
1 parent 0a1efee
commit e704b09
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/app/views/assessments/edit.html.erb b/app/views/assessments/edit.html.erb
@@ -21,6 +21,7 @@
     <% [ "basic", "handin", "penalties", "problems", "advanced" ].each do |tab_name| %>
       <%= (tab_name == params[:active_tab]) ? tag(:li, {:class => "active tab"}) : tag(:li, class: :tab) %>
         <%= link_to tab_name.titleize, "#tab_#{tab_name}" %>
+      </li>
     <% end %>
     </ul>
   </div>

diff --git a/app/views/assessments/show.html.erb b/app/views/assessments/show.html.erb
@@ -327,7 +327,7 @@
         locals: { submission: submission,
         download_access: download_access } %>
     <% end %>
-    <%= link_to "See all submissions", {:action => "history"} %>
+    <%= link_to "See all submissions", {:action => "history"} %></li>
 
   <% end %>
 </div>