UI: minimap causes main map not to initialize

[nominatim.git] / website / js / nominatim-ui.js

diff --git a/website/js/nominatim-ui.js b/website/js/nominatim-ui.js
index 0560b45ff3b4625930e2baa94cc54738af0dacab..abb23bebdd60a94a2553a0781449b674255d49fc 100644 (file)
--- a/website/js/nominatim-ui.js
+++ b/website/js/nominatim-ui.js
@@ -17,7 +17,7 @@ function parse_and_normalize_geojson_string(raw_string){
     return parsed_geojson;
 }
 
-jQuery(document).on('ready', function(){
+jQuery(document).ready(function(){
 
     if ( !$('#search-page,#reverse-page').length ){ return; }
     
@@ -37,8 +37,6 @@ jQuery(document).on('ready', function(){
         attribution: (nominatim_map_init.tile_attribution || null ) //'&copy; <a href="http://osm.org/copyright">OpenStreetMap</a> contributors'
     }).addTo(map);
 
-    map.setView([nominatim_map_init.lat, nominatim_map_init.lon], nominatim_map_init.zoom);
-
     var osm2 = new L.TileLayer(nominatim_map_init.tile_url, {minZoom: 0, maxZoom: 13, attribution: (nominatim_map_init.tile_attribution || null )});
     var miniMap = new L.Control.MiniMap(osm2, {toggleDisplay: true}).addTo(map);
 
@@ -198,15 +196,18 @@ jQuery(document).on('ready', function(){
             }
         }
         else {
-            if ( is_reverse_search ){
-                // make sure the search coordinates are in the map view as well
-                map.fitBounds([[result.lat,result.lon], [nominatim_map_init.lat,nominatim_map_init.lon]], {padding: [50,50], maxZoom: map.getZoom()});
-
-                // better, but causes a leaflet warning
-                // map.panInsideBounds([[result.lat,result.lon], [nominatim_map_init.lat,nominatim_map_init.lon]], {animate: false});
-            }
-            else {
-                map.panTo([result.lat,result.lon], result.zoom || nominatim_map_init.zoom);
+            var result_coord = L.latLng(result.lat, result.lon);
+            if ( result_coord ){
+                if ( is_reverse_search ){
+                    // make sure the search coordinates are in the map view as well
+                    map.fitBounds([result_coord, [nominatim_map_init.lat,nominatim_map_init.lon]], {padding: [50,50], maxZoom: map.getZoom()});
+
+                    // better, but causes a leaflet warning
+                    // map.panInsideBounds([[result.lat,result.lon], [nominatim_map_init.lat,nominatim_map_init.lon]], {animate: false});
+                }
+                else {
+                    map.panTo(result_coord, result.zoom || nominatim_map_init.zoom);
+                }
             }
         }
 
@@ -238,6 +239,8 @@ jQuery(document).on('ready', function(){
         });
 
         $('#switch-coords').on('click', function(e){
+            e.preventDefault();
+            e.stopPropagation();
             var lat = $('form input[name=lat]').val();
             var lon = $('form input[name=lon]').val();
             $('form input[name=lat]').val(lon);
@@ -248,11 +251,19 @@ jQuery(document).on('ready', function(){
 
     highlight_result(0, false);
 
+    // common mistake is to copy&paste latitude and longitude into the 'lat' search box
+    $('form input[name=lat]').on('change', function(){
+        var coords = $(this).val().split(',');
+        if (coords.length == 2) {
+            $(this).val(L.Util.trim(coords[0]));
+            $(this).siblings('input[name=lon]').val(L.Util.trim(coords[1]));
+        }
+    });
 
 });
 
 
-jQuery(document).on('ready', function(){
+jQuery(document).ready(function(){
 
     if ( !$('#details-page').length ){ return; }
 
@@ -271,9 +282,6 @@ jQuery(document).on('ready', function(){
             attribution: (nominatim_map_init.tile_attribution || null ) //'&copy; <a href="http://osm.org/copyright">OpenStreetMap</a> contributors'
         }).addTo(map);
 
-        var osm2 = new L.TileLayer(nominatim_map_init.tile_url, {minZoom: 0, maxZoom: 13, attribution: (nominatim_map_init.tile_attribution || null )});
-        var miniMap = new L.Control.MiniMap(osm2, {toggleDisplay: true}).addTo(map);
-
         var layerGroup = new L.layerGroup().addTo(map);
 
         var circle = L.circleMarker([nominatim_result.lat,nominatim_result.lon], { radius: 10, weight: 2, fillColor: '#ff7800', color: 'blue', opacity: 0.75});
@@ -296,6 +304,8 @@ jQuery(document).on('ready', function(){
             map.setView([nominatim_result.lat,nominatim_result.lon],10);
         }
 
+        var osm2 = new L.TileLayer(nominatim_map_init.tile_url, {minZoom: 0, maxZoom: 13, attribution: (nominatim_map_init.tile_attribution || null )});
+        var miniMap = new L.Control.MiniMap(osm2, {toggleDisplay: true}).addTo(map);
 
 
 });