[openstreetmap-website] Diary cleanup (#218)

Saman Bemel-Benrud notifications at github.com
Sun Mar 24 21:34:26 UTC 2013


> @@ -4,7 +4,16 @@
>        <%= user_thumbnail diary_entry.user %>
>      <% end %>
>  
> -    <h2><%= link_to h(diary_entry.title), :action => 'view', :display_name => diary_entry.user.display_name, :id => diary_entry.id %></h2>
> +    <h2>
> +      <% if params[:action] == 'list' %>
> +      <%= link_to h(diary_entry.title), :action => 'view', :display_name => diary_entry.user.display_name, :id => diary_entry.id %>
> +       <% else %>
> +      <%= diary_entry.title %>
> +      <% end %>
> +    </h2>
> +    <small class='deemphasize'>

Yep, I will revert this. 

Visually, that strong "link" blue on post on titles doesn't work. It feels a little weird. There are other alternatives, like adding a little permalink icon or having more nuanced link styles across the board, would be more appropriate.

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


More information about the rails-dev mailing list