diff --git a/superset-frontend/src/explore/components/DisplayQueryButton.jsx b/superset-frontend/src/explore/components/DisplayQueryButton.jsx index dbb5d8441b..e292839248 100644 --- a/superset-frontend/src/explore/components/DisplayQueryButton.jsx +++ b/superset-frontend/src/explore/components/DisplayQueryButton.jsx @@ -36,7 +36,6 @@ import { getChartDataRequest } from '../../chart/chartAction'; import downloadAsImage from '../../utils/downloadAsImage'; import Loading from '../../components/Loading'; import ModalTrigger from '../../components/ModalTrigger'; -import PropertiesModal from './PropertiesModal'; import { sliceUpdated } from '../actions/exploreActions'; import { CopyButton } from './DataTableControl'; @@ -46,6 +45,7 @@ SyntaxHighlighter.registerLanguage('sql', sqlSyntax); SyntaxHighlighter.registerLanguage('json', jsonSyntax); const propTypes = { + onOpenPropertiesModal: PropTypes.func, onOpenInEditor: PropTypes.func, queryResponse: PropTypes.object, chartStatus: PropTypes.string, @@ -76,7 +76,6 @@ export const DisplayQueryButton = props => { const [sqlSupported] = useState( datasource && datasource.split('__')[1] === 'table', ); - const [isPropertiesModalOpen, setIsPropertiesModalOpen] = useState(false); const [menuVisible, setMenuVisible] = useState(false); const beforeOpen = resultType => { @@ -103,20 +102,12 @@ export const DisplayQueryButton = props => { }); }; - const openPropertiesModal = () => { - setIsPropertiesModalOpen(true); - }; - - const closePropertiesModal = () => { - setIsPropertiesModalOpen(false); - }; - const handleMenuClick = ({ key, domEvent }) => { const { chartHeight, slice, onOpenInEditor, latestQueryFormData } = props; setMenuVisible(false); switch (key) { case MENU_KEYS.EDIT_PROPERTIES: - openPropertiesModal(); + props.onOpenPropertiesModal(); break; case MENU_KEYS.RUN_IN_SQL_LAB: onOpenInEditor(latestQueryFormData); @@ -182,17 +173,11 @@ export const DisplayQueryButton = props => { onToggle={setMenuVisible} > - {slice && [ + {slice && ( {t('Edit properties')} - , - , - ]} + + )} diff --git a/superset-frontend/src/explore/components/ExploreChartHeader.jsx b/superset-frontend/src/explore/components/ExploreChartHeader.jsx index 52af3df178..88a54b3b18 100644 --- a/superset-frontend/src/explore/components/ExploreChartHeader.jsx +++ b/superset-frontend/src/explore/components/ExploreChartHeader.jsx @@ -94,7 +94,7 @@ export class ExploreChartHeader extends React.PureComponent { this.state = { isPropertiesModalOpen: false, }; - this.openProperiesModal = this.openProperiesModal.bind(this); + this.openPropertiesModal = this.openPropertiesModal.bind(this); this.closePropertiesModal = this.closePropertiesModal.bind(this); } @@ -111,7 +111,7 @@ export class ExploreChartHeader extends React.PureComponent { ); } - openProperiesModal() { + openPropertiesModal() { this.setState({ isPropertiesModalOpen: true, }); @@ -167,7 +167,7 @@ export class ExploreChartHeader extends React.PureComponent { role="button" tabIndex={0} className="edit-desc-icon" - onClick={this.openProperiesModal} + onClick={this.openPropertiesModal} > @@ -202,7 +202,10 @@ export class ExploreChartHeader extends React.PureComponent { status={CHART_STATUS_MAP[chartStatus]} />