@nertc commented on this pull request.


> @@ -76,6 +85,15 @@
           <li><%= link_to t("layouts.copyright"), copyright_path, :class => 
"dropdown-item" %></li>
           <li><%= link_to t("layouts.help"), help_path, :class => 
"dropdown-item" %></li>
           <li><%= link_to t("layouts.about"), about_path, :class => 
"dropdown-item" %></li>
+          <li>
+            <% if current_user && current_user.id %>
+              <%= link_to(preferences_path) do %>
+                <%= render "shared/language_selector", :hoverable => true, 
:black => true, :classes => "dropdown-item", :disabled => true %>

I'll try to move it out of the "More" and test with different resolutions. If 
it doesn't break any resolution, I'll keep it out of the "More". It will need 
less HTML code.

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

Message ID: 
<openstreetmap/openstreetmap-website/pull/5201/review/2325967...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to