style(dashboard): minor improvements to chart context menu (apache#11170

) Bumping "Explore/edit chart" as the first option. Getting rid of the old "edit chart properties" deprecated CRUD link
auxten · Nov 20, 2020 · e94f1b0 · e94f1b0
1 parent bf5ae4f
commit e94f1b0
Showing 1 changed file with 3 additions and 9 deletions.
diff --git a/superset-frontend/src/dashboard/components/SliceHeaderControls.jsx b/superset-frontend/src/dashboard/components/SliceHeaderControls.jsx
@@ -158,22 +158,16 @@ class SliceHeaderControls extends React.PureComponent {
             </MenuItem>
           )}
 
-          {this.props.sliceCanEdit && (
-            <MenuItem href={slice.edit_url} target="_blank">
-              {t('Edit chart metadata')}
+          {this.props.supersetCanExplore && (
+            <MenuItem onClick={this.exploreChart}>
+              {t('Explore chart')}
             </MenuItem>
           )}
 
           {this.props.supersetCanCSV && (
             <MenuItem onClick={this.exportCSV}>{t('Export CSV')}</MenuItem>
           )}
 
-          {this.props.supersetCanExplore && (
-            <MenuItem onClick={this.exploreChart}>
-              {t('Explore chart')}
-            </MenuItem>
-          )}
-
           <MenuItem onClick={this.handleToggleFullSize}>{resizeLabel}</MenuItem>
 
           <URLShortLinkModal