On the chart editing screen, the icon of dataset should be similar to those used on the “Dataset” screen.

[xavier-GitHub76]

Hello,

Physical and virtual data sets do not have the same pictogram.

On the chart editing screen, the icon should be similar to those used on the “Dataset” screen.

[kasiazjc]

Hello,

Physical and virtual data sets do not have the same pictogram.

On the chart editing screen, the icon should be similar to those used on the “Dataset” screen.

Hi, thanks for raising this! I definitely agree, do you think it's something you would be able to change? 🙏

[xavier-GitHub76]

I can try but I'm a novice.
Could you help me to identify files to changed ?

After, I will try to see how to add the control on the kind of dataset and apply the good SVG :

Physical dataset :

<svg xmlns="http://www.w3.org/2000/svg" width="1em" height="1em" fill="currentColor" aria-hidden="true" focusable="false" class="" viewBox="0 0 24 24">
    <path fill="currentColor" fill-rule="evenodd" d="M21 2.5H3a1 1 0 0 0-1 1v18a1 1 0 0 0 1 1h18a1 1 0 0 0 1-1v-18a1 1 0 0 0-1-1m-13 18H4v-4h4zm0-6H4v-4h4zm0-6H4v-4h4zm6 12h-4v-4h4zm0-6h-4v-4h4zm0-6h-4v-4h4zm6 12h-4v-4h4zm0-6h-4v-4h4zm0-6h-4v-4h4z" clip-rule="evenodd"></path>
</svg>

Virtual Dataset :

<svg xmlns="http://www.w3.org/2000/svg" width="1em" height="1em" fill="currentColor" aria-hidden="true" focusable="false" class="" viewBox="0 0 24 24"> 
    <path fill="#136478" d="M4 4.5h16v16H4z"></path>
    <path fill="#A5DBE9" fill-rule="evenodd" d="M21 2.5H3a1 1 0 0 0-1 1v18a1 1 0 0 0 1 1h18a1 1 0 0 0 1-1v-18a1 1 0 0 0-1-1m-13 18H4v-4h4zm0-6H4v-4h4zm0-6H4v-4h4zm6 12h-4v-4h4zm0-6h-4v-4h4zm0-6h-4v-4h4zm6 12h-4v-4h4zm0-6h-4v-4h4zm0-6h-4v-4h4z" clip-rule="evenodd"></path>
</svg>

[qingshan007]

Problem Investigation:

After investigation, it was found that the rendering type was incorrect
Create a chart type that is always table
datasource type: table

On the chart construction page, switch datasets (virtual or physical):

datasource type: virtual
datasource type: physical

Without modifying or destroying the original logic of the newly created dataset:
Locate to: superset-frontend\src\explore\components\controls\DatasourceControl\index.jsx

1. Add judgment logic, where the physical dataset SQL field is always='', and the virtual one is the SQL during construction:

const dynamicDatasourceType = datasource?.sql
      ? DatasourceType.Query
      : DatasourceType.Table;

2. Use a new judgment:

{/* {datasourceIconLookup[datasource?.type]} */}
↓
{datasourceIconLookup[dynamicDatasourceType]}

3. Modify the icon:

// Assign icon for each DatasourceType.   If no icon assingment is found in the lookup, no icon will render
// export const datasourceIconLookup = {
//   [DatasourceType.Query]: (
//     <Icons. ConsoleSqlOutlined className="datasource-svg" />
//   ),
//   [DatasourceType.Table]: <Icons.DatasetPhysical className="datasource-svg" />,
// };
↓
export const datasourceIconLookup = {
[DatasourceType.Query]: <Icons.DatasetVirtual className="datasource-svg" />,
[DatasourceType.Table]: <Icons.DatasetPhysical className="datasource-svg" />,
};

4. Result:
   Switch datasets and display icons of corresponding types
   Create a regular chart OR by entering it from the dataset page and displaying the corresponding type of icon

[qingshan007]

@xavier-GitHub76 @kasiazjc

[rusackas]

I agree with this... and I would encourage anyone to open a PR to this effect. Meanwhile, since it's not a bug, but rather an enhancement (and it's been silent for a while), I'll convert this to an Ideas Discussion where it can live on in hopes of finding a volunteer :D