diff options
| -rw-r--r-- | CHANGELOG.md | 3 | ||||
| -rw-r--r-- | web/js/map-OpenLayers.js | 11 |
2 files changed, 9 insertions, 5 deletions
diff --git a/CHANGELOG.md b/CHANGELOG.md index 9089a1823..f37ca7088 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -10,7 +10,8 @@ - Fix issue with red bars on bar graph of many categories. - Prefetch translations in /reports list of bodies. - Ignore deleted/area-less bodies in dashboard list. - - Add missing CSS class from final questionnaire question. + - Add missing CSS class from final questionnaire question. #1953 + - Fix JavaScript error on /my calculating bounds #1954 - Change text on /reports to match lower down (fix translation). - Ensure all reports graph can't dip downward. #1956 - UK: diff --git a/web/js/map-OpenLayers.js b/web/js/map-OpenLayers.js index 00f3e8e70..1d417f68e 100644 --- a/web/js/map-OpenLayers.js +++ b/web/js/map-OpenLayers.js @@ -236,7 +236,8 @@ $.extend(fixmystreet.utils, { * we already have the pins when the page loaded */ function zoomToBounds(bounds) { if (!bounds) { return; } - fixmystreet.markers.strategies[0].deactivate(); + var strategy = fixmystreet.markers.strategies[0]; + strategy.deactivate(); var center = bounds.getCenterLonLat(); var z = fixmystreet.map.getZoomForExtent(bounds); if ( z < 13 && $('html').hasClass('mobile') ) { @@ -244,9 +245,11 @@ $.extend(fixmystreet.utils, { } fixmystreet.map.setCenter(center, z); // Reactivate the strategy and make it think it's done an update - fixmystreet.markers.strategies[0].activate(); - fixmystreet.markers.strategies[0].calculateBounds(); - fixmystreet.markers.strategies[0].resolution = fixmystreet.map.getResolution(); + strategy.activate(); + if (strategy instanceof OpenLayers.Strategy.BBOX) { + strategy.calculateBounds(); + strategy.resolution = fixmystreet.map.getResolution(); + } } function sidebar_highlight(problem_id) { |