Create linkloader iframe in script

openstreetmap · Aug 6, 2013 · d413539 · d413539
1 parent b125981
commit d413539
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 9 deletions.
diff --git a/app/assets/javascripts/application.js b/app/assets/javascripts/application.js
@@ -44,12 +44,21 @@ function remoteEditHandler(bbox, select) {
       };
 
   if (select) query.select = select;
-  $("#linkloader")
+
+  var iframe = $('<iframe>')
+    .hide()
+    .appendTo('body')
     .attr("src", "http://127.0.0.1:8111/load_and_zoom?" + querystring.stringify(query))
-    .load(function() { loaded = true; });
+    .on('load', function() {
+      $(this).remove();
+      loaded = true;
+    });
 
   setTimeout(function () {
-    if (!loaded) alert(I18n.t('site.index.remote_failed'));
+    if (!loaded) {
+      alert(I18n.t('site.index.remote_failed'));
+      iframe.remove();
+    }
   }, 1000);
 
   return false;

diff --git a/app/views/browse/_map.html.erb b/app/views/browse/_map.html.erb
@@ -1,6 +1,3 @@
-<iframe id="linkloader" style="display: none">
-</iframe>
-
 <div id="browse_map" class='clearfix content_map'>
   <% if map.instance_of? Changeset or (map.instance_of? Node and map.version > 1) or map.visible? %>
 

diff --git a/app/views/site/index.html.erb b/app/views/site/index.html.erb
@@ -19,9 +19,6 @@
 <div id="map" tabindex="2">
 </div>
 
-<iframe id="linkloader" style="display: none">
-</iframe>
-
 <div id="attribution">
   <table width="100%">
     <tr>