@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 solve the focusing issue without any custom classes. In terms of
collapsibility of the translation icon, purpose was to take as little space as
possible when the screen is smaller. Without collapsing, HTML code will be
shorter and with less "if-else"-es.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5201#discussion_r1772865699
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5201/review/2324460...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
