[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