X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/6debc9dd19b3441a974c8eae9788cfa80227badd..6477e004fb67681b8b64bcdbca187dbc32c6cd60:/services/workbench2/src/components/data-explorer/data-explorer.tsx diff --git a/services/workbench2/src/components/data-explorer/data-explorer.tsx b/services/workbench2/src/components/data-explorer/data-explorer.tsx index 27e46d5849..ba710bc783 100644 --- a/services/workbench2/src/components/data-explorer/data-explorer.tsx +++ b/services/workbench2/src/components/data-explorer/data-explorer.tsx @@ -84,7 +84,6 @@ interface DataExplorerDataProps { defaultViewIcon?: IconType; defaultViewMessages?: string[]; working?: boolean; - currentRefresh?: string; currentRoute?: string; hideColumnSelector?: boolean; paperProps?: PaperProps; @@ -97,6 +96,7 @@ interface DataExplorerDataProps { elementPath?: string; isMSToolbarVisible: boolean; checkedList: TCheckedList; + isNotFound: boolean; } interface DataExplorerActionProps { @@ -120,52 +120,13 @@ type DataExplorerProps = DataExplorerDataProps & DataExplorerActionProps extends React.Component> { - state = { - showLoading: false, - prevRefresh: "", - prevRoute: "", - }; multiSelectToolbarInTitle = !this.props.title && !this.props.progressBar; - componentDidUpdate(prevProps: DataExplorerProps) { - const currentRefresh = this.props.currentRefresh || ""; - const currentRoute = this.props.currentRoute || ""; - - if (currentRoute !== this.state.prevRoute) { - // Component already mounted, but the user comes from a route change, - // like browsing through a project hierarchy. - this.setState({ - showLoading: this.props.working, - prevRoute: currentRoute, - }); - } - - if (currentRefresh !== this.state.prevRefresh) { - // Component already mounted, but the user just clicked the - // refresh button. - this.setState({ - showLoading: this.props.working, - prevRefresh: currentRefresh, - }); - } - if (this.state.showLoading && !this.props.working) { - this.setState({ - showLoading: false, - }); - } - } - componentDidMount() { if (this.props.onSetColumns) { this.props.onSetColumns(this.props.columns); } - // Component just mounted, so we need to show the loading indicator. - this.setState({ - showLoading: this.props.working, - prevRefresh: this.props.currentRefresh || "", - prevRoute: this.props.currentRoute || "", - }); } render() { @@ -207,6 +168,7 @@ export const DataExplorer = withStyles(styles)( toggleMSToolbar, setCheckedListOnStore, checkedList, + working, } = this.props; return (