menu dividers

dagster-io · Sep 15, 2023 · 518c1e9 · 518c1e9
1 parent 4dab527
commit 518c1e9
Showing 1 changed file with 3 additions and 16 deletions.
diff --git a/js_modules/dagster-ui/packages/ui-core/src/asset-graph/AssetGraphExplorerSidebar.tsx b/js_modules/dagster-ui/packages/ui-core/src/asset-graph/AssetGraphExplorerSidebar.tsx
@@ -12,6 +12,7 @@ import {
   TextInput,
   MiddleTruncate,
   useViewport,
+  MenuDivider,
 } from '@dagster-io/ui-components';
 import {useVirtualizer} from '@tanstack/react-virtual';
 import React from 'react';
@@ -361,6 +362,7 @@ const Node = ({
                 content={
                   <Menu>
                     <MenuItem icon="materialization" text="Materialize" />
+                    {upstream.length || downstream.length ? <MenuDivider /> : null}
                     {upstream.length ? (
                       <MenuItem
                         text="Select upstream"
@@ -381,6 +383,7 @@ const Node = ({
                         }}
                       />
                     ) : null}
+                    {upstream.length || downstream.length ? <MenuDivider /> : null}
                     {upstream.length ? (
                       <MenuItem
                         text="Show upstream graph"
@@ -401,22 +404,6 @@ const Node = ({
                         }}
                       />
                     ) : null}
-                    {upstream.length ? (
-                      <MenuItem
-                        text={`View upstream (${upstream.length})`}
-                        onClick={() => {
-                          setShowUpstreamDialog(true);
-                        }}
-                      />
-                    ) : null}
-                    {downstream.length ? (
-                      <MenuItem
-                        text={`View downstream (${downstream.length})`}
-                        onClick={() => {
-                          setShowDownstreamDialog(true);
-                        }}
-                      />
-                    ) : null}
                   </Menu>
                 }
                 hoverOpenDelay={100}