Set OpenID tabindex higher than username/password form tabindexes

[rails.git] / app / views / layouts / map.html.erb

diff --git a/app/views/layouts/map.html.erb b/app/views/layouts/map.html.erb
index 7a0ca0ed714c3bde95ade620844d7dfee23d97f9..f6a7473a4f6f777d3977a4c4026a19de0efbb12d 100644 (file)
--- a/app/views/layouts/map.html.erb
+++ b/app/views/layouts/map.html.erb
@@ -4,7 +4,7 @@
 
 <% content_for(:body_class) { "map-layout" } %>
 
 
 <% content_for(:body_class) { "map-layout" } %>
 

-<% if current_user and !current_user.home_lon.nil? and !current_user.home_lat.nil? %>
+<% if current_user&.home_location? %>

   <% content_for :greeting do %>
     <%= link_to t("layouts.home"),
                 "#",
   <% content_for :greeting do %>
     <%= link_to t("layouts.home"),
                 "#",


@@ -30,43 +30,38 @@
 
     <div id="browse_status"></div>
 
 
     <div id="browse_status"></div>
 

-    <div id="sidebar_loader">
-      <img alt="<%= t("browse.start_rjs.loading") %>" class="loader" src="<%= image_path("searching.gif") %>">
+    <div id="sidebar_loader" class="my-3 text-center loader">
+      <div class="spinner-border" role="status">
+        <span class="visually-hidden"><%= t("browse.start_rjs.loading") %></span>
+      </div>

     </div>
 
     </div>
 

-    <div id="sidebar_content">
+    <div id="sidebar_content" class="p-3">

       <%= yield %>
     </div>
 
     <% unless current_user %>
       <%= yield %>
     </div>
 
     <% unless current_user %>

-      <div class="welcome p-3">
+      <div class="welcome p-3" hidden>

         <%= render "sidebar_header", :title => t("layouts.intro_header") %>
         <%= render "sidebar_header", :title => t("layouts.intro_header") %>

-        <div>
-          <p><%= t "layouts.intro_text" %></p>
-          <p><%= t "layouts.hosting_partners_html",
-                   :ucl => link_to(t("layouts.partners_ucl"), "https://www.ucl.ac.uk"),
-                   :bytemark => link_to(t("layouts.partners_bytemark"), "https://www.bytemark.co.uk"),
-                   :partners => link_to(t("layouts.partners_partners"), "https://hardware.openstreetmap.org/thanks/") %>
-          </p>
-          <div class="d-flex mx-n1">
-            <div class="w-50 px-1">
-              <a class="btn btn-primary w-100" href="<%= about_path %>"><%= t("layouts.learn_more") %></a>
-            </div>
-            <div class="w-50 px-1">
-              <a class="btn btn-primary w-100" href="<%= user_new_path %>"><%= t("layouts.start_mapping") %></a>
-            </div>
-          </div>
+        <p class="fs-6 fw-light"><%= t "layouts.intro_text" %></p>
+        <p class="fs-6 fw-light"><%= t "layouts.hosting_partners_html",
+                                       :ucl => link_to(t("layouts.partners_ucl"), "https://www.ucl.ac.uk"),
+                                       :fastly => link_to(t("layouts.partners_fastly"), "https://www.fastly.com/"),
+                                       :bytemark => link_to(t("layouts.partners_bytemark"), "https://www.bytemark.co.uk"),
+                                       :partners => link_to(t("layouts.partners_partners"), "https://hardware.openstreetmap.org/thanks/") %>
+        </p>
+        <div class="d-flex gap-2">
+          <a class="btn btn-primary w-100 d-flex align-items-center justify-content-center" href="<%= about_path %>"><%= t("layouts.learn_more") %></a>
+          <a class="btn btn-primary w-100 d-flex align-items-center justify-content-center" href="<%= user_new_path %>"><%= t("layouts.start_mapping") %></a>

         </div>
       </div>
     <% end %>
 
         </div>
       </div>
     <% end %>
 

-    <div id="banner">
-      <%= render :partial => "layouts/banner" %>
-    </div>
+    <%= render :partial => "layouts/banner" %>

   </div>
 
   <noscript>
   </div>
 
   <noscript>

-    <div id="noscript">
+    <div class="mt-5 p-3">

       <p><%= t "site.index.js_1" %></p>
       <p><%= t "site.index.js_2" %></p>
     </div>
       <p><%= t "site.index.js_1" %></p>
       <p><%= t "site.index.js_2" %></p>
     </div>


@@ -78,11 +73,11 @@
   <div id="map" tabindex="2">
   </div>
 
   <div id="map" tabindex="2">
   </div>
 

-  <div id="attribution">
+  <div id="attribution" class="d-none">

     <table>
       <tr>
     <table>
       <tr>

-        <td class="attribution_license"><%= t "site.index.license.license_url" %></td>
-        <td class="attribution_project"><%= t "site.index.license.project_url" %></td>
+        <td><%= t "site.index.license.license_url" %></td>
+        <td><%= t "site.index.license.project_url" %></td>

       </tr>
       <tr>
         <td colspan="2" class="attribution_notice"><%= t "site.index.license.copyright" %></td>
       </tr>
       <tr>
         <td colspan="2" class="attribution_notice"><%= t "site.index.license.copyright" %></td>