X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/a1eff4d25c72460e1c3e52e523ba5029012d2ce9..af7233e3695ad99ae500b51390e1bf9c1c8c5a5c:/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 1f9fb42d6b..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 { @@ -118,64 +118,15 @@ interface DataExplorerActionProps { type DataExplorerProps = DataExplorerDataProps & DataExplorerActionProps & WithStyles & MPVPanelProps; -type DataExplorerState = { - prevRefresh: string; - prevRoute: string; - notFound: boolean; -}; - export const DataExplorer = withStyles(styles)( class DataExplorerGeneric extends React.Component> { - state: DataExplorerState = { - prevRefresh: "", - prevRoute: "", - notFound: false, - }; 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({ - prevRoute: currentRoute, - }); - } - - if (currentRefresh !== this.state.prevRefresh) { - // Component already mounted, but the user just clicked the - // refresh button. - this.setState({ - prevRefresh: currentRefresh, - }); - } - - if(this.state.notFound === true && this.props.itemsAvailable > 0) { - this.setState({ - notFound: false, - }); - } - - if (prevProps.working === true && this.props.working === false) { - this.setState({ - notFound: this.props.itemsAvailable === 0, - }); - } - } - componentDidMount() { if (this.props.onSetColumns) { this.props.onSetColumns(this.props.columns); } - // Component just mounted, so we need to show the loading indicator. - this.setState({ - prevRefresh: this.props.currentRefresh || "", - prevRoute: this.props.currentRoute || "", - }); } render() { @@ -333,7 +284,7 @@ export const DataExplorer = withStyles(styles)( setCheckedListOnStore={setCheckedListOnStore} checkedList={checkedList} working={working} - notFound={this.state.notFound} + isNotFound={this.props.isNotFound} />