Header html nesting fixes (publiclab#2199)

Souravirus · Mar 12, 2018 · 608de83 · 608de83
1 parent b2a79b7
commit 608de83
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/app/views/layouts/_header.html.erb b/app/views/layouts/_header.html.erb
@@ -125,7 +125,7 @@
             <ul class="dropdown-menu">
               <li class="dropdown-header"><%= current_user.username %><% if current_user.role === 'moderator' || current_user.role === 'admin'%>: <%= current_user.role %><% end %></li>
               <% if params[:action] == "dashboard" %>
-                <li><a href="/home"></a>Home</li>
+                <li><a href="/home">Home</a></li>
               <% else %>
                 <li><a href="/dashboard"><%= t('layout._header.dashboard') %></a></li>
               <% end %>
@@ -135,13 +135,13 @@
               <li><a href="/subscriptions"><%= t('layout._header.subscriptions') %></a></li>
               <li><a href="/profile/<%= current_user.username %>/likes"><%= t('layout._header.notes_liked') %></a></li>
               <li class="divider"></li>
-	       <% if current_user.role == 'admin' %>
-    <li><a href="/useremail">Email search</a></li>
-    <li><a href="/wiki/locked">Locked pages</a></li>
-    <li><a href="/spam">Spam moderation</a></li>
-    <li><a href="/people">Contributors</a></li>
-	           <li class="divider"></li>
-	       <% end %>
+	      <% if current_user.role == 'admin' %>
+                <li><a href="/useremail">Email search</a></li>
+                <li><a href="/wiki/locked">Locked pages</a></li>
+                <li><a href="/spam">Spam moderation</a></li>
+                <li><a href="/people">Contributors</a></li>
+	        <li class="divider"></li>
+	      <% end %>
               <li><%= link_to t('layout._header.logout'), logout_path %></li>
               <li><a href="/logoutRemotely">Logout from all devices</a></li>
             </ul>