[openstreetmap/openstreetmap-website] Add links to diaries in preferred languages (PR #4451)

Tom Hughes notifications at github.com
Tue Jan 2 18:53:48 UTC 2024


@tomhughes requested changes on this pull request.



>  
-            <% if @user && @user == current_user || !@user && current_user %>
-              <li><%= link_to image_tag("new.png", :class => "small_icon") + t(".new"), new_diary_entry_path, :title => t(".new_title") %></li>
-            <% end %>
+          <% @languages&.each do |language| %>

I think these should probably go after "New Diary Entry" and before "My Diary" they're links to a subset of the diary in much the same way that "My Diary" is so they should probably be together.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4451#pullrequestreview-1800866966
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4451/review/1800866966 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240102/36d35d29/attachment.htm>


More information about the rails-dev mailing list