<p>In app/views/diary_entry/_diary_entry.html.erb:</p>
<pre style='color:#555'>> @@ -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'>
</pre>
<p>What's win here? Do we just think it's "prettier" if the title isn't a link on the diary entry view? It's adding a  very kludgy complication to the template, and removing a handy link that can be used to grab a link to the diary entry so I'd like there to be a decent upside to make it a worthwhile change.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href='https://github.com/openstreetmap/openstreetmap-website/pull/218/files#r3504482'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/uTRSc6ihLa7Shf84BpiOpqKgP4QyOfojJAw_AjmPPVivRvjblH6TJr8s6cCO8Fhk.gif' height='1' width='1'></p>