Display loading spinner while switching between helm charts (#4269)

This commit is contained in:
Sebastian Malton authored and GitHub committed 2021-11-08 10:12:17 -05:00
1 parent 90d4899ece
commit 3666c56319
1 file changed
+7 -3
@@ -49,9 +49,9 @@ const LargeTooltip = withStyles({
@observer @observer
export class HelmChartDetails extends Component<Props> { export class HelmChartDetails extends Component<Props> {
@observable chartVersions: HelmChart[]; @observable chartVersions: HelmChart[];
@observable selectedChart: HelmChart; @observable selectedChart?: HelmChart;
@observable readme: string = null; @observable readme?: string;
@observable error: string = null; @observable error?: string;
private abortController?: AbortController; private abortController?: AbortController;
@@ -68,6 +68,10 @@ export class HelmChartDetails extends Component<Props> {
disposeOnUnmount(this, [ disposeOnUnmount(this, [
reaction(() => this.props.chart, async ({ name, repo, version }) => { reaction(() => this.props.chart, async ({ name, repo, version }) => {
try { try {
this.selectedChart = undefined;
this.chartVersions = undefined;
this.readme = undefined;
const { readme, versions } = await getChartDetails(repo, name, { version }); const { readme, versions } = await getChartDetails(repo, name, { version });
this.readme = readme; this.readme = readme;