From: Tom Hughes <tom@compton.nu>
Date: Mon, 28 Apr 2008 09:49:27 +0000 (+0000)
Subject: IE was getting confused by having a div with an id of "map" and a
X-Git-Tag: live~8390
X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/42da12e81c98d12a5cb8acf993c6b3a113798387

IE was getting confused by having a div with an id of "map" and a
link anchor with a name of "map". Closes #865.
---

diff --git a/app/views/diary_entry/new.rhtml b/app/views/diary_entry/new.rhtml
index b46bb4d9f..6c40eaeb5 100644
--- a/app/views/diary_entry/new.rhtml
+++ b/app/views/diary_entry/new.rhtml
@@ -11,10 +11,13 @@
       <td><%= f.text_area :body, :cols => 80 %></td>
     </tr>
     <tr valign="top">
-	    <th>Location</th>
-	    <td><a name="map"></a><div id="map" style="border: 1px solid black; position: relative; width : 90%; height : 400px; display: none;"></div>
-		  <span class="location">Latitude: <%= f.text_field :latitude, :size => 20, :id => "latitude" %> Longitude: <%= f.text_field :longitude, :size => 20, :id => "longitude" %></span> <a href="#map" id="usemap" onclick="document.getElementById('map').style.display = 'block'; document.getElementById('usemap').style.display = 'none';">use map</a> </td>
-	  </tr>
+      <th>Location</th>
+      <td>
+        <div id="map" style="border: 1px solid black; position: relative; width : 90%; height : 400px; display: none;"></div>
+        <span class="location">Latitude: <%= f.text_field :latitude, :size => 20, :id => "latitude" %> Longitude: <%= f.text_field :longitude, :size => 20, :id => "longitude" %></span>
+        <a href="javascript:openMap()" id="usemap">use map</a>
+      </td>
+    </tr>
     <tr>
       <th></th>
       <td><%= submit_tag 'Save' %></td>
@@ -40,7 +43,7 @@
   <!--
   var marker;
 
-  function init(){
+  function init() {
     var centre = new OpenLayers.LonLat(<%= lon %>, <%= lat %>);
     var zoom = <%= zoom %>;
 
@@ -66,6 +69,11 @@
     marker = addMarkerToMap(merc, null, "Diary entry location");
   }
 
+  function openMap() {
+    $("map").style.display = "block";
+    $("usemap").style.display = "none";
+  }
+
   window.onload = init;
 // -->
 </script>