X-Git-Url: https://git.openstreetmap.org/rails.git/blobdiff_plain/8a9261ea0749ce9bf714073542a017133de01378..b46655b16762573305cde0fce1b4d885649cf594:/app/views/browse/start.rjs

diff --git a/app/views/browse/start.rjs b/app/views/browse/start.rjs
index 11f9edfd4..783c387b2 100644
--- a/app/views/browse/start.rjs
+++ b/app/views/browse/start.rjs
@@ -22,6 +22,7 @@ page << <<EOJ
         map.addControl(box);
         map.events.register("moveend", map, showData);
         map.events.triggerEvent("moveend");
+        map.dataLayer.stopBrowse = stopBrowse;
         browsing = true;
     }
 
@@ -35,20 +36,24 @@ page << <<EOJ
     }
     
     function stopBrowse() {
-        if (gml) {
-            gml.destroy();
-            gml = null; 
-        } 
-        if (sf) {   
-            sf.destroy();  
-            sf = null;
-        } 
-        if (currentFeature) {
-            currentFeature.destroy(); 
-            currentFeature = null; 
-        } 
-        map.events.unregister("moveend", map, showData);
-        browsing = false; 
+        if (browsing) {
+            browsing = false; 
+            map.dataLayer.stopBrowse = null;
+            if (gml) {
+                gml.destroy();
+                gml = null; 
+            } 
+            if (sf) {   
+                sf.destroy();  
+                sf = null;
+            } 
+            if (currentFeature) {
+                currentFeature.destroy(); 
+                currentFeature = null; 
+            }
+            map.dataLayer.setVisibility(false);
+            map.events.unregister("moveend", map, showData);
+        }    
     }
     
     function startDrag() {
@@ -74,7 +79,6 @@ page << <<EOJ
             currentBounds = tileBounds;
             getData(tileBounds);
             mode = "auto";
-            $("drag_box").style.display="inline";
             $("use_map").style.display="none";
         }
         return false;
@@ -84,11 +88,11 @@ page << <<EOJ
     function endDrag(bbox) {
         var bounds = bbox.getBounds();
         box.deactivate();
+        currentBounds = bounds;
         getData(bounds);
         $("drag_box").innerHTML = "Manually select a different area";
         mode = "manual";  
         $("use_map").style.display="inline";
-        $("drag_box").style.display="none";
     }
     
     function displayFeatureWarning() {
@@ -168,6 +172,7 @@ page << <<EOJ
             gml = new OpenLayers.Layer.GML("Data",url, 
                     {format: OpenLayers.Format.OSM, formatOptions: {checkTags: true},
                      maxFeatures: 100, requestSuccess: customDataLoader,
+                     displayInLayerSwitcher: false,
                      styleMap: new OpenLayers.StyleMap({'default': style, 'select': {'strokeColor': '#0000ff', strokeWidth: 8}})
                     }
             );
@@ -188,9 +193,11 @@ page << <<EOJ
     }
     function dataLoaded() {
         if (!browsing) { return; } 
-        $("status").innerHTML = "Loaded " + this.features.length + " features. (<a href='"+ this.url+"'>API</a>)";
+        $("status").innerHTML = "Loaded."
         
-        objList = document.createElement("ul");
+        objList = document.createElement("div")
+
+        list = document.createElement("ul");
         for (var i = 0; i < this.features.length; i++) {
             var feature = this.features[i]; 
             
@@ -212,10 +219,16 @@ page << <<EOJ
             link.onclick = OpenLayers.Function.bind(viewFeatureLink, link);   
             li.appendChild(link);
 
-            objList.appendChild(li);
+            list.appendChild(li);
         }
+        objList.appendChild(list)
+        var link = document.createElement("a");
+        link.href = this.url;
+        link.appendChild(document.createTextNode("API"));
+        objList.appendChild(link)
         $("object").innerHTML = "";
         $("object").appendChild(objList); 
+
     }
     
     function viewFeatureLink() {
@@ -266,18 +279,23 @@ page << <<EOJ
         div.appendChild(link)
         $("object").appendChild(div);    
         
-        // Now the list of attributes
-        var ul = document.createElement("ul");
         var type = "way";
         if (feature.geometry.CLASS_NAME == "OpenLayers.Geometry.Point") {
             type = "node";
         }    
-        var li = document.createElement("li");
+        
         var link = document.createElement("a");   
         link.href =  "/browse/"+type+"/"+feature.osm_id;
-        link.appendChild(document.createTextNode(feature.osm_id));
-        li.appendChild(link);
-        ul.appendChild(li);
+        link.appendChild(document.createTextNode("Database entry for " + feature.osm_id));
+        
+        var div = document.createElement("div");
+        div.style.marginTop = "20px"
+        div.appendChild(link);
+
+        $("object").appendChild(div);
+
+        // Now the list of attributes
+        var ul = document.createElement("ul");
         for (var key in feature.attributes) {
             var li = document.createElement("li");
             var b = document.createElement("b");
@@ -287,16 +305,16 @@ page << <<EOJ
             ul.appendChild(li);
         }
         
-        var li = document.createElement("li");
+        $("object").appendChild(ul);
+        
         var link = document.createElement("a");   
         link.href =  "/browse/"+type+"/"+feature.osm_id+"/history";
         link.appendChild(document.createTextNode("History"));
-        li.appendChild(link);
         ul.appendChild(li);
         link.onclick = OpenLayers.Function.bind(loadHistory, {type: type, feature: feature, link: link});
-        $("object").appendChild(ul);
-        
         
+        $("object").appendChild(link);
+
         // Stash the currently drawn feature
         currentFeature = feature; 
     }   
@@ -310,7 +328,7 @@ page << <<EOJ
         if (currentFeature.osm_id != this.feature.osm_id || $("object").firstChild == objList)  { 
             return false;
         } 
-        this.link.parentNode.parentNode.removeChild(this.link.parentNode);
+        this.link.parentNode.removeChild(this.link);
         var doc = request.responseXML;
         var div = document.createElement("div");
         var h3 = document.createElement("h3"); 
@@ -327,7 +345,7 @@ page << <<EOJ
         }
         div.appendChild(history);
         var link = document.createElement("a");
-        link.appendChild(document.createTextNode("History"));
+        link.appendChild(document.createTextNode("History entry for " + this.feature.osm_id));
         link.href = "/browse/"+this.type+"/"+this.feature.osm_id+"/history";
         div.appendChild(link);
         $("object").appendChild(div);