fix(url-center): round the latlng to 4 digits as suggested by @fbuchi…

…nger here: #283
tombatossals · Mar 19, 2014 · 4a6d755 · 4a6d755
1 parent 21bacce
commit 4a6d755
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 7 deletions.
diff --git a/dist/angular-leaflet-directive.js b/dist/angular-leaflet-directive.js
@@ -115,7 +115,7 @@
           return;
         }
         var center = map.getCenter();
-        var centerUrlHash = center.lat + ':' + center.lng + ':' + map.getZoom();
+        var centerUrlHash = center.lat.toFixed(4) + ':' + center.lng.toFixed(4) + ':' + map.getZoom();
         var search = $location.search();
         if (!isDefined(search.c) || search.c !== centerUrlHash) {
           //$log.debug("notified new center...");

diff --git a/dist/angular-leaflet-directive.min.js b/dist/angular-leaflet-directive.min.js
diff --git a/src/directives/center.js b/src/directives/center.js
@@ -16,7 +16,7 @@ angular.module("leaflet-directive").directive('center',
             return;
         }
         var center = map.getCenter();
-        var centerUrlHash = center.lat + ":" + center.lng + ":" + map.getZoom();
+        var centerUrlHash = (center.lat).toFixed(4) + ":" + (center.lng).toFixed(4) + ":" + map.getZoom();
         var search = $location.search();
         if (!isDefined(search.c) || search.c !== centerUrlHash) {
             //$log.debug("notified new center...");

diff --git a/test/unit/centerDirectiveSpec.js b/test/unit/centerDirectiveSpec.js
@@ -107,7 +107,7 @@ describe('Directive: leaflet center', function() {
                 centerUrlHash = u;
             });
             scope.$digest();
-            expect(centerUrlHash).toBe('9.5:-1.8:8');
+            expect(centerUrlHash).toBe('9.5000:-1.8000:8');
         });
     });
 });
diff --git a/test/unit/layersDirectiveSpec.js b/test/unit/layersDirectiveSpec.js
@@ -921,7 +921,6 @@ describe('Directive: leaflet', function() {
             layer: 'cars'
         };
         scope.$digest();
-        console.log(scope.markers.m1);
         expect(map.hasLayer(markers.m1)).toBe(true); //Fail
         expect(layers.overlays.cars.hasLayer(markers.m1)).toBe(true);