Use flex in routing form to fix marker column width

diff --git a/app/assets/javascripts/index/directions.js b/app/assets/javascripts/index/directions.js
index 89eccb8cbd7d9074fb08d188a7d3875730969a6d..f6b0581caef1d7ba5ce98a21da8990ac05638949 100644 (file)
--- a/app/assets/javascripts/index/directions.js
+++ b/app/assets/javascripts/index/directions.js
@@ -357,7 +357,7 @@ OSM.Directions = function (map) {
     getRoute(true, true);
   });
 
-  $(".routing_marker").on("dragstart", function (e) {
+  $(".routing_marker_column img").on("dragstart", function (e) {
     var dt = e.originalEvent.dataTransfer;
     dt.effectAllowed = "move";
     var dragData = { type: $(this).data("type") };

diff --git a/app/assets/stylesheets/common.scss b/app/assets/stylesheets/common.scss
index 757da56fec30f495a332f758b87eb6ebfb5c1cf7..4cc31d611b6d7dd0187ca475e08b298d0636e1c9 100644 (file)
--- a/app/assets/stylesheets/common.scss
+++ b/app/assets/stylesheets/common.scss
@@ -568,7 +568,13 @@ tr.turn {
     cursor: pointer;
 }
 
-.routing_marker { width: 15px; cursor: move; }
+.routing_marker_column {
+  width: 15px;
+
+  img {
+    cursor: move;
+  }
+}
 
 /* Rules for the history sidebar */
 

diff --git a/app/views/layouts/_search.html.erb b/app/views/layouts/_search.html.erb
index 11b7c7b1e1ce5091c9eb11bdd3b5f81cba960dcb..00cf60b7a0bca80c47150720dd976285e014cb3a 100644 (file)
--- a/app/views/layouts/_search.html.erb
+++ b/app/views/layouts/_search.html.erb
@@ -33,34 +33,26 @@
   <form method="GET" action="<%= directions_path %>" class="directions_form bg-body-secondary pb-3">
     <div class="d-flex flex-row-reverse px-3 py-3"><button type="button" class="btn-close" aria-label="<%= t("javascripts.close") %>"></button></div>
 
-    <div class="row gx-2 m-1">
-      <div class="col-1">
-        <%= image_tag "marker-green.png", :class => "routing_marker mx-auto d-block", :data => { :type => "from" }, :draggable => "true" %>
-      </div>
-      <div class="col">
-        <%= text_field_tag "route_from", params[:from], :placeholder => t("site.search.from"), :autocomplete => "on", :class => "form-control py-1 px-2", :dir => "auto" %>
+    <div class="d-flex my-1 mx-2 gap-2 align-items-center">
+      <div class="routing_marker_column flex-shrink-0">
+        <%= image_tag "marker-green.png", :class => "img-fluid", :data => { :type => "from" }, :draggable => "true" %>
       </div>
+      <%= text_field_tag "route_from", params[:from], :placeholder => t("site.search.from"), :autocomplete => "on", :class => "form-control py-1 px-2", :dir => "auto" %>
     </div>
-    <div class="row gx-2 m-1">
-      <div class="col-1">
-        <%= image_tag "marker-red.png", :class => "routing_marker mx-auto d-block", :data => { :type => "to" }, :draggable => "true" %>
-      </div>
-      <div class="col">
-        <%= text_field_tag "route_to", params[:to], :placeholder => t("site.search.to"), :autocomplete => "on", :class => "form-control py-1 px-2", :dir => "auto" %>
+    <div class="d-flex my-1 mx-2 gap-2 align-items-center">
+      <div class="routing_marker_column flex-shrink-0">
+        <%= image_tag "marker-red.png", :class => "img-fluid", :data => { :type => "to" }, :draggable => "true" %>
       </div>
+      <%= text_field_tag "route_to", params[:to], :placeholder => t("site.search.to"), :autocomplete => "on", :class => "form-control py-1 px-2", :dir => "auto" %>
     </div>
-    <div class="row gx-2 m-1">
-      <div class="col offset-1">
-        <select class="routing_engines form-select py-1 px-2" name="routing_engines"></select>
-      </div>
-      <div class="col-auto">
-        <%= submit_tag t("site.search.submit_text"), :class => "routing_go btn btn-primary py-1 px-2", :data => { :disable_with => false } %>
-      </div>
+    <div class="d-flex my-1 mx-2 gap-2 align-items-center">
+      <div class="routing_marker_column flex-shrink-0"></div>
+      <select class="routing_engines form-select py-1 px-2" name="routing_engines"></select>
+      <%= submit_tag t("site.search.submit_text"), :class => "routing_go btn btn-primary py-1 px-2", :data => { :disable_with => false } %>
     </div>
-    <div class="row gx-2 m-1">
-      <div class="col offset-1">
-        <button class="btn btn-link py-1 px-2 reverse_directions"><%= t("site.search.reverse_directions_text") %></button>
-      </div>
+    <div class="d-flex my-1 mx-2 gap-2 align-items-center">
+      <div class="routing_marker_column flex-shrink-0"></div>
+      <button class="btn btn-link py-1 px-2 reverse_directions"><%= t("site.search.reverse_directions_text") %></button>
     </div>
 
     <div class="loader_copy d-none">