update failed test and use visibleSidebar instead

Signed-off-by: Huong Nguyen <huongg1409@gmail>
kedro-org · Jul 29, 2024 · 8306bcc · 8306bcc
1 parent ead4fad
commit 8306bcc
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 5 deletions.
diff --git a/src/components/flowchart/components/sliced-pipeline-action-bar/sliced-pipeline-action-bar.js b/src/components/flowchart/components/sliced-pipeline-action-bar/sliced-pipeline-action-bar.js
@@ -3,11 +3,14 @@ import Button from '../../../ui/button';
 
 import './sliced-pipeline-action-bar.scss';
 
-export const SlicedPipelineActionBar = ({ chartSize, slicedPipeline }) => {
-  const { outerWidth, sidebarWidth } = chartSize;
-  const sidebarVisible = sidebarWidth > 140;
+export const SlicedPipelineActionBar = ({
+  chartSize,
+  slicedPipeline,
+  visibleSidebar,
+}) => {
+  const { outerWidth } = chartSize;
 
-  const transformX = sidebarVisible ? outerWidth / 2 + 100 : outerWidth / 2;
+  const transformX = visibleSidebar ? outerWidth / 2 + 100 : outerWidth / 2;
   if (slicedPipeline.length > 0) {
     return (
       <div

diff --git a/src/components/flowchart/flowchart.js b/src/components/flowchart/flowchart.js
@@ -718,6 +718,7 @@ export class FlowChart extends Component {
       visibleGraph,
       displayGlobalNavigation,
       displaySidebar,
+      visibleSidebar,
     } = this.props;
     const { outerWidth = 0, outerHeight = 0 } = chartSize;
     const { slicedPipelineState } = this.state;
@@ -786,6 +787,7 @@ export class FlowChart extends Component {
           <SlicedPipelineActionBar
             chartSize={chartSize}
             slicedPipeline={slicedPipelineState.range}
+            visibleSidebar={visibleSidebar}
           />
         </div>
         <Tooltip

diff --git a/src/components/flowchart/flowchart.test.js b/src/components/flowchart/flowchart.test.js
@@ -487,7 +487,7 @@ describe('FlowChart', () => {
       displayGlobalNavigation: expect.any(Boolean),
       displaySidebar: expect.any(Boolean),
       displayMetadataPanel: expect.any(Boolean),
-      filteredPipeline: expect.any(Object),
+      slicedPipeline: expect.any(Object),
     };
     expect(mapStateToProps(mockState.spaceflights)).toEqual(expectedResult);
   });