Allow css to be disabled on the menu for those who don't need it and …

…want the increased speed without all the lookups.
refinery · Sep 6, 2010 · 69cd6f0 · 69cd6f0
1 parent d24ff1d
commit 69cd6f0
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/vendor/refinerycms/core/app/views/shared/_menu.html.erb b/vendor/refinerycms/core/app/views/shared/_menu.html.erb
@@ -14,7 +14,8 @@
                      :collection => collection,
                      :locals => {
                        :hide_children => hide_children ||= RefinerySetting.find_or_set(:menu_hide_children, false),
-                       :sibling_count => sibling_count
+                       :sibling_count => sibling_count,
+                       :apply_css => true #if you don't care about class='first' class='last' or class='selected' set apply_css to false for speed.
                      }
           -%>
         </ul>

diff --git a/vendor/refinerycms/core/app/views/shared/_menu_branch.html.erb b/vendor/refinerycms/core/app/views/shared/_menu_branch.html.erb
@@ -1,5 +1,8 @@
 <%
-  css = (css = css_for_menu_branch(menu_branch, menu_branch_counter, sibling_count||=nil)).any? ? " class='#{css.join(' ')}'" : nil
+  css = if ((defined?(apply_css) && apply_css) || !defined?(apply_css)) and
+           (classes = css_for_menu_branch(menu_branch, menu_branch_counter, sibling_count||=nil.any?))
+    " class='#{classes.join(' ')}'"
+  end
   dom_id = (menu_branch.parent_id.nil? and menu_branch.title.present?) ? " id='item_#{menu_branch_counter}'" : nil
 -%>
 <li<%= css -%><%= dom_id -%>>
@@ -12,7 +15,7 @@
         render :partial => "/shared/menu_branch",
                :collection => children,
                :locals => {
-                 :hide_children => hide_children,
+                 :hide_children => false, # hide children can't be true or we wouldn't get here.
                  :sibling_count => children.size - 1
                } -%>
     </ul>