Merge pull request #2906 from weaveworks/2858-prevent-zooming-nan-errors

Try to prevent zooming NaN errors

client/app/scripts/actions/app-actions.js

@@ -405,7 +405,8 @@ export function clickTopology(topologyId) {
 export function cacheZoomState(zoomState) {
   return {
     type: ActionTypes.CACHE_ZOOM_STATE,
-    zoomState
+    // Make sure only proper numerical values are cached.
+    zoomState: zoomState.filter(value => !window.isNaN(value)),
   };
 }
 

client/app/scripts/reducers/root.js

@@ -88,7 +88,8 @@ export const initialState = makeMap({
   topologyViewMode: GRAPH_VIEW_MODE,
   version: '...',
   versionUpdate: null,
-  viewport: makeMap(),
+  // Set some initial numerical values to prevent NaN in case of edgy race conditions.
+  viewport: makeMap({ width: 0, height: 0 }),
   websocketClosed: false,
   zoomCache: makeMap(),
   serviceImages: makeMap()