[openstreetmap-website] Site cleanup (#150)

John Firebaugh notifications at github.com
Sun Jan 6 21:19:41 GMT 2013


> -<% if @this_user %>
> -  <%= user_image @this_user %>
> -<% end %>
> -
> -<h2><%= h(@title) %></h2>
> -
> -<% if @this_user %>
> -  <%= if_user(@this_user) do %>
> -    <%= link_to image_tag("new.png", :border=>0) + t('diary_entry.list.new'), {:controller => 'diary_entry', :action => 'new'}, {:title => t('diary_entry.list.new_title')} %>
> -  <% end %>
> -<% else %>
> -  <%= if_logged_in do %>
> -    <%= link_to image_tag("new.png", :border=>0) + t('diary_entry.list.new'), {:controller => 'diary_entry', :action => 'new'}, {:title => t('diary_entry.list.new_title')} %>
> +<% content_for :heading do %>
> +
> +  <% if @this_user %><div id="userinformation"><% end %>

This is confusing. Can we conditionalize just the presence/absence of the `id` attribute rather than the entire tag structure?

---
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/150/files#r2560592
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20130106/d9ce05d3/attachment.html>


More information about the rails-dev mailing list