Merge remote-tracking branch 'openstreetmap/pull/954'

[rails.git] / app / assets / javascripts / welcome.js

diff --git a/app/assets/javascripts/welcome.js b/app/assets/javascripts/welcome.js
index 5773bfd6dbc6260c935561f93e072816f25276f6..5ba0d7cd2183e28f3ffcbaa88d5901eefece4531 100644 (file)
--- a/app/assets/javascripts/welcome.js
+++ b/app/assets/javascripts/welcome.js
@@ -2,58 +2,46 @@ $(document).ready(function() {
   var params = OSM.params();
 
   if (params.lat && params.lon) {
-    $('.edit-located').show();
-
-    $.ajax({
-      url: "http://nominatim.openstreetmap.org/reverse",
-      data: {
-        lat: params.lat,
-        lon: params.lon,
-        zoom: 10
-      },
-      success: function(xml) {
-        var result = $(xml).find('result');
-        if (result.length) {
-          $('.edit-located').hide();
-          $('.edit-geocoded').show();
-          $('.edit-geocoded-location').text(result.text());
-        }
-      }
-    });
+    params.lat = parseFloat(params.lat);
+    params.lon = parseFloat(params.lon);
+    params.zoom = params.zoom || 17;
 
-    params = {
-      lat: params.lat,
-      lon: params.lon,
-      zoom: params.zoom || 17,
-      editor: params.editor
-    };
+    var url = '/edit';
 
-    $('.start-mapping').attr('href', '/edit?' + $.param(params));
+    if (params.editor) {
+      url += '?editor=' + params.editor;
+    }
 
-  } else if (navigator.geolocation) {
-    $('.edit-geolocated').show();
+    url += OSM.formatHash(params);
 
-    function geoSuccess(position) {
-      window.location = '/edit?zoom=17&lat=' + position.coords.latitude + '&lon=' + position.coords.longitude;
-    }
+    $('.start-mapping').attr('href', url);
 
-    function geoError() {
-      $('.start-mapping')
-        .removeClass('loading')
-        .addClass('error');
-    }
+  } else {
+    var geoSuccess = function (position) {
+      window.location = '/edit' + OSM.formatHash({
+        zoom: 17,
+        lat: position.coords.latitude,
+        lon: position.coords.longitude
+      });
+    };
 
     $('.start-mapping').on('click', function(e) {
       e.preventDefault();
+      $('.start-mapping').addClass('loading');
 
-      $('.start-mapping')
-        .addClass('loading');
-
-      // handle firefox's weird implementation
-      // https://bugzilla.mozilla.org/show_bug.cgi?id=675533
-      window.setTimeout(geoError, 4000);
+      if (navigator.geolocation) {
+        // handle firefox's weird implementation
+        // https://bugzilla.mozilla.org/show_bug.cgi?id=675533
+        window.setTimeout(manualEdit, 4000);
 
-      navigator.geolocation.getCurrentPosition(geoSuccess, geoError);
+        navigator.geolocation.getCurrentPosition(geoSuccess, manualEdit);
+      } else {
+        manualEdit();
+      }
     });
   }
+
+  function manualEdit() {
+    window.location = '/?edit_help=1';
+  }
 });