Changeset 33290 for main/trunk/greenstone3

Timestamp:

2019-07-02T21:09:46+12:00 (5 years ago)

Author:

wy59

Message:

Minor cleanup

File:

• main/trunk/greenstone3/web/interfaces/default/js/map-scripts.js (modified) (3 diffs)

main/trunk/greenstone3/web/interfaces/default/js/map-scripts.js

@@ -437 +437 @@
 }
 
-// In order for fitBounds to work out a non-zero bounds, have to use visibility, not display when hiding/showing the map.
+// Refer to https://developers.google.com/maps/documentation/javascript/reference/map#Map.fitBounds
+// In order for fitBounds() to work out a non-zero bounds, have to use visibility, not display when hiding/showing the map.
 // AND should not set height to 0px when using visibility hidden. But not setting height to 0 means an invisible map takes up space.
 // In order for the resulting invisible map, that still takes up space, to not break the flow of the visible items
 // on the page, need to swap map between position relative when map visible versus position absolute when the map is hidden.
-// https://developers.google.com/maps/documentation/javascript/reference/map#Map.fitBounds
 function showMap(callingFunction) {
     
@@ -904 +904 @@
             showMap("httpMapBrowseRequest");
             
-            //var jsonNodesStr = "";            
             for(var i = 0; i < jsonNodes.length; i++)
             {
@@ -911 +910 @@
                 _docList.ids.push(doc.nodeID);
     
-                //$("#div"+doc.nodeID).attr("data-gps-map-json", JSON.stringify(jsonNodes[i]));
-                //jsonNodesStr += JSON.stringify(jsonNodes[i]); // can't do this after createOverlayItems, as we get cyclical ref error
                 var options = {
                     "mainDoc": true // TODO: should this be true or false???