bit of UI cleanup

app/helpers/application_helper.rb

@@ -21,20 +21,14 @@ module ApplicationHelper
     nav = [
         nav_item('Recipes', recipes_path, 'recipes'),
         nav_item('Ingredients', ingredients_path, 'ingredients'),
-        nav_item('Logs', logs_path, 'logs'),
+        nav_item('Logs', logs_path, 'logs', current_user?),
         nav_item('Calculator', calculator_path, 'calculator'),
-        nav_item('About', about_path, 'home')
+        nav_item('About', about_path, 'home'),
+        nav_item('Notes', notes_path, 'notes', current_user?),
+        nav_item('Admin', admin_users_path, 'admin/users', current_user? && current_user.admin?)
     ]
 
-    if current_user
+    nav.compact
-      nav << nav_item('Notes', notes_path, 'notes')
-    end
-
-    if current_user && current_user.admin?
-      nav << nav_item('Admin', admin_users_path, 'admin/users')
-    end
-
-    nav
   end
 
   def profile_nav_items
@@ -57,8 +51,8 @@ module ApplicationHelper
     end
   end
 
-  def nav_item(name, url, controller = nil)
+  def nav_item(name, url, controller = nil, visible = true)
-    content_tag('li', link_to(name, url), class: active_for_controller(controller))
+    !visible ? nil : content_tag('li', link_to(name, url), class: active_for_controller(controller))
   end
 
   def active_for_controller(controller)

app/views/ingredients/index.html.erb

@@ -8,7 +8,9 @@
       <p>No Ingredients</p>
     <% else %>
 
+      <% if current_user? %>
         <%= link_to 'New Ingredient', new_ingredient_path, class: 'btn btn-default' %>
+      <% end %>
 
       <br/>
 
@@ -19,22 +21,26 @@
           <th>USDA</th>
           <th>KCal per 100g</th>
           <th>Density (oz/cup)</th>
+          <% if current_user? %>
             <th></th>
+          <% end %>
         </tr>
         </thead>
 
         <tbody>
         <% decorate(@ingredients, IngredientDecorator).each do |ingredient| %>
           <tr>
-            <td><%= link_to ingredient.name, edit_ingredient_path(ingredient) %></td>
+            <td><%= link_to_if current_user?, ingredient.name, edit_ingredient_path(ingredient) %></td>
             <td><%= ingredient.ndbn_check %></td>
             <td><%= ingredient.kcal %></td>
             <td><%= ingredient.density %></td>
+            <% if current_user? %>
               <td>
                 <%= link_to ingredient, method: :delete, data: { confirm: 'Are you sure?' }, class: 'btn btn-xs btn-danger' do %>
                   <span class="glyphicon glyphicon-remove"></span>
                 <% end %>
               </td>
+            <% end %>
           </tr>
         <% end %>
         </tbody>
@@ -43,7 +49,9 @@
 
     <br/>
 
+    <% if current_user? %>
       <%= link_to 'New Ingredient', new_ingredient_path, class: 'btn btn-default' %>
+    <% end %>
 
 
   </div>

app/views/kaminari/_first_page.html.erb

@@ -0,0 +1,3 @@
+<li>
+  <%= link_to_unless current_page.first?, raw(t 'views.pagination.first'), url, :remote => remote %>
+</li>

app/views/kaminari/_gap.html.erb

@@ -0,0 +1,3 @@
+<li class='disabled'>
+  <%= content_tag :a, raw(t 'views.pagination.truncate') %>
+</li>

app/views/kaminari/_last_page.html.erb

@@ -0,0 +1,3 @@
+<li>
+  <%= link_to_unless current_page.last?, raw(t 'views.pagination.last'), url, {:remote => remote} %>
+</li>

app/views/kaminari/_next_page.html.erb

@@ -0,0 +1,3 @@
+<li>
+  <%= link_to_unless current_page.last?, raw(t 'views.pagination.next'), url, :rel => 'next', :remote => remote %>
+</li>

app/views/kaminari/_page.html.erb

@@ -0,0 +1,9 @@
+<% if page.current? %>
+  <li class='active'>
+    <%= content_tag :a, page, remote: remote, rel: (page.next? ? 'next' : (page.prev? ? 'prev' : nil)) %>
+  </li>
+<% else %>
+  <li>
+    <%= link_to page, url, remote: remote, rel: (page.next? ? 'next' : (page.prev? ? 'prev' : nil)) %>
+  </li>
+<% end %>

app/views/kaminari/_paginator.html.erb

@@ -0,0 +1,15 @@
+<%= paginator.render do -%>
+  <ul class="pagination">
+    <%= first_page_tag unless current_page.first? %>
+    <%= prev_page_tag unless current_page.first? %>
+    <% each_page do |page| -%>
+      <% if page.left_outer? || page.right_outer? || page.inside_window? -%>
+        <%= page_tag page %>
+      <% elsif !page.was_truncated? -%>
+        <%= gap_tag %>
+      <% end -%>
+    <% end -%>
+    <%= next_page_tag unless current_page.last? %>
+    <%= last_page_tag unless current_page.last? %>
+  </ul>
+<% end -%>

app/views/kaminari/_prev_page.html.erb

@@ -0,0 +1,3 @@
+<li>
+  <%= link_to_unless current_page.first?, raw(t 'views.pagination.previous'), url, :rel => 'prev', :remote => remote %>
+</li>

app/views/recipes/index.html.erb

@@ -9,7 +9,9 @@
       <p>No Recipes</p>
     <% else %>
 
+      <% if current_user? %>
         <%= link_to 'New Recipe', new_recipe_path, class: 'btn btn-default' %><br/>
+      <% end %>
 
       <%= paginate @recipes %>
 
@@ -69,7 +71,9 @@
 
     <br>
 
-    <%= link_to 'New Recipe', new_recipe_path, class: 'btn btn-default' %>
+    <% if current_user? %>
+      <%= link_to 'New Recipe', new_recipe_path, class: 'btn btn-default' %><br/>
+    <% end %>
 
   </div>
 </div>