On Fri, Apr 16, 2010 at 3:26 PM, Pito Salas <li...@ruby-forum.com> wrote:
> Hi all
>
> This isn't doing what I expect it to. Can you give it a quick look-see
> if you have a moment?
>
> def user_navigation_helper
> content_tag(:div, :class =>"banner_right") do
> content_tag(:ul, :class => "wat-cf") do
> if current_user()
> content_tag(:li) { current_user.email }
> content_tag(:li) { link_to("Edit profile",
> edit_user_path(:current)) }
> content_tag(:li) { link_to("Logout", logout_path) }
> else
> content_tag(:li) { link_to("Login", login_path) }
> content_tag(:li) { link_to("Register", new_user_path) }
> end
> end
> end
> end
>
> I am getting just one of the two or thee <li> items. Is there a
> better/cleaner/correct way of doing this? Thanks!
one way:
def user_navigation_helper
content_tag(:div, :class =>"banner_right") do
content_tag(:ul, :class => "wat-cf") do
if current_user()
[content_tag(:li) { current_user.email },
content_tag(:li) { link_to("Edit profile",
edit_user_path(:current)) },
content_tag(:li) { link_to("Logout", logout_path) }]].join
else
[content_tag(:li) { link_to("Login", login_path) },
content_tag(:li) { link_to("Register", new_user_path) }].join
end
end
end
end
--
Rick DeNatale
Blog: http://talklikeaduck.denhaven2.com/
Github: http://github.com/rubyredrick
Twitter: @RickDeNatale
WWR: http://www.workingwithrails.com/person/9021-rick-denatale
LinkedIn: http://www.linkedin.com/in/rickdenatale
--
You received this message because you are subscribed to the Google Groups "Ruby
on Rails: Talk" group.
To post to this group, send email to rubyonrails-t...@googlegroups.com.
To unsubscribe from this group, send email to
rubyonrails-talk+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en.
