Question

Right now I have a navigation partial that looks like this (x10 buttons)…

<% if current_controller == 'territories' %>     <li><%= link_to 'Territories', {:controller => 'territories'}, :class => 'active'  %></li> <% else %>     <li><%= link_to 'Territories', {:controller => 'territories'}  %></li> <% end %> <% if current_controller == 'contacts'  %>     <li><%= link_to 'Contacts', {:controller => 'Contacts'}, :class => 'active'  %></li> <% else %>     <li><%= link_to 'Contacts', {:controller => 'Contacts'}  %></li> <% end %> 

Is there a more elegant/DRY solution for doing this?

Answer 1

In a similar vein to what Chuck said:

<% TARGETS.each do |target| %>   <li>     <%= link_to target.humanize,        { :controller => target },        class => ('active' if current_controller == target)) %>   </li> <% end %>