From 004b0ef4901dd75759a9234302555ce85217b77f Mon Sep 17 00:00:00 2001
From: Eugene Samonenko <esamonenko@agilevision.io>
Date: Sun, 7 Feb 2021 12:41:30 +0200
Subject: [PATCH] feat(fitView): use option includeHiddenNodes false by default

---
 src/hooks/useZoomPanHelper.ts | 6 +++---
 src/types/index.ts            | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/hooks/useZoomPanHelper.ts b/src/hooks/useZoomPanHelper.ts
index 7fd54891..ba04196b 100644
--- a/src/hooks/useZoomPanHelper.ts
+++ b/src/hooks/useZoomPanHelper.ts
@@ -13,7 +13,7 @@ const initialZoomPanHelper: ZoomPanHelperFunctions = {
   zoomOut: () => {},
   zoomTo: (_: number) => {},
   transform: (_: FlowTransform) => {},
-  fitView: (_: FitViewParams = { padding: DEFAULT_PADDING, excludeHidden: false }) => {},
+  fitView: (_: FitViewParams = { padding: DEFAULT_PADDING, includeHiddenNodes: false }) => {},
   setCenter: (_: number, __: number) => {},
   fitBounds: (_: Rect) => {},
   initialized: false,
@@ -55,14 +55,14 @@ const useZoomPanHelper = (): ZoomPanHelperFunctions => {
 
           d3Zoom.transform(d3Selection, nextTransform);
         },
-        fitView: (options: FitViewParams = { padding: DEFAULT_PADDING, excludeHidden: false }) => {
+        fitView: (options: FitViewParams = { padding: DEFAULT_PADDING, includeHiddenNodes: false }) => {
           const { nodes, width, height, minZoom, maxZoom } = store.getState();
 
           if (!nodes.length) {
             return;
           }
 
-          const bounds = getRectOfNodes(options.excludeHidden ? nodes.filter(node => !node.isHidden) : nodes);
+          const bounds = getRectOfNodes(options.includeHiddenNodes ? nodes : nodes.filter(node => !node.isHidden));
           const padding = options.padding || DEFAULT_PADDING
           const [x, y, zoom] = getTransformForBounds(bounds, width, height, minZoom, maxZoom, padding);
           const transform = zoomIdentity.translate(x, y).scale(zoom);
diff --git a/src/types/index.ts b/src/types/index.ts
index e3a602a5..46dcfa37 100644
--- a/src/types/index.ts
+++ b/src/types/index.ts
@@ -236,7 +236,7 @@ export interface WrapNodeProps<T = any> {
 
 export type FitViewParams = {
   padding?: number;
-  excludeHidden?: boolean;
+  includeHiddenNodes?: boolean;
 };
 
 export type FlowExportObject<T = any> = {