From 1d2a40f69b0d835fea531c4a81d96ef5bf3d7f3b Mon Sep 17 00:00:00 2001 From: xavier-GitHub76 <82046143+xavier-GitHub76@users.noreply.github.com> Date: Mon, 30 Jun 2025 18:21:20 +0200 Subject: [PATCH] fix(explore): Change dataset icon on explore to match datasets view (#33977) --- .../controls/DatasourceControl/index.jsx | 19 ++++++++++++++----- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/superset-frontend/src/explore/components/controls/DatasourceControl/index.jsx b/superset-frontend/src/explore/components/controls/DatasourceControl/index.jsx index ee1187894a3..45acc2b65d2 100644 --- a/superset-frontend/src/explore/components/controls/DatasourceControl/index.jsx +++ b/superset-frontend/src/explore/components/controls/DatasourceControl/index.jsx @@ -73,6 +73,16 @@ const defaultProps = { isEditable: true, }; +const getDatasetType = datasource => { + if (datasource.type === 'query') { + return 'query'; + } + if (datasource.type === 'table' && datasource.sql) { + return 'virtual_dataset'; + } + return 'physical_dataset'; +}; + const Styles = styled.div` .data-container { display: flex; @@ -134,10 +144,9 @@ const VISIBLE_TITLE_LENGTH = 25; // Assign icon for each DatasourceType. If no icon assignment is found in the lookup, no icon will render export const datasourceIconLookup = { - [DatasourceType.Query]: ( - - ), - [DatasourceType.Table]: , + query: , + physical_dataset: , + virtual_dataset: , }; // Render title for datasource with tooltip only if text is longer than VISIBLE_TITLE_LENGTH @@ -397,7 +406,7 @@ class DatasourceControl extends PureComponent { return ( - {datasourceIconLookup[datasource?.type]} + {datasourceIconLookup[getDatasetType(datasource)]} {renderDatasourceTitle(titleText, tooltip)} {healthCheckMessage && (