@1ec5 commented on this pull request.


> -          <% end %>
-        <% else %>
-          <%= render "layouts/select_language_list" %>
-        <% end %>
+        <div class="mb-3 px-2">
+          <%= text_field_tag :language_search,
+                             params[:language_search],
+                             :placeholder => 
t("languages_panes.select_language_list.search_language", :default => 
"Search..."),
+                             :autocomplete => "off",
+                             :dir => "auto",
+                             :class => "form-control py-2 px-3" %>
+        </div>
+        <turbo-frame id="select_language_list" src="<%= languages_pane_path 
%>?source=<%= ERB::Util.url_encode(request.fullpath) %>" loading="lazy">
+          <div class="text-center py-3">
+            <div class="spinner-border text-secondary" role="status">
+              <span class="visually-hidden">Loading...</span>

Screenreader users might be using a locale other than English. We also localize 
the [hidden attribution 
text](https://github.com/openstreetmap/openstreetmap-website/blob/801aeaa43a7bc5ec3b6f9fba530980cf55b33621/app/views/layouts/map.html.erb#L77)
 on the homepage and `<title>` tags inside SVGs, so localizing the “Loading” 
indicators would be consistent. Granted, this would probably affect a small 
segment of the user base.

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

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

Reply via email to