From 98665fef91f7b42e1458bec9dc6ecf9faa46be69 Mon Sep 17 00:00:00 2001 From: Bryan Housel Date: Sat, 28 Feb 2015 22:59:10 -0500 Subject: [PATCH] Don't allow zoomTo on an entity with degenrate extent (because sending the map to NaN/NaN/NaN is not cool) --- js/id/renderer/map.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/js/id/renderer/map.js b/js/id/renderer/map.js index 8d9529561..3ff258a10 100644 --- a/js/id/renderer/map.js +++ b/js/id/renderer/map.js @@ -341,8 +341,10 @@ iD.Map = function(context) { }; map.zoomTo = function(entity, zoomLimits) { - var extent = entity.extent(context.graph()), - zoom = map.trimmedExtentZoom(extent); + var extent = entity.extent(context.graph()); + if (!isFinite(extent.area())) return; + + var zoom = map.trimmedExtentZoom(extent); zoomLimits = zoomLimits || [context.minEditableZoom(), 20]; map.centerZoom(extent.center(), Math.min(Math.max(zoom, zoomLimits[0]), zoomLimits[1])); };