X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/10ce16c28de952f6533ca3cc9df909269e3d2a53..8c6209c45e4720c66dd53640142cbc9111748246:/src/components/tree/tree.tsx

diff --git a/src/components/tree/tree.tsx b/src/components/tree/tree.tsx
index e4d8c72ce6..799e821704 100644
--- a/src/components/tree/tree.tsx
+++ b/src/components/tree/tree.tsx
@@ -3,43 +3,43 @@
 // SPDX-License-Identifier: AGPL-3.0
 
 import * as React from 'react';
-import List from "@material-ui/core/List/List";
-import ListItem from "@material-ui/core/ListItem/ListItem";
+import { List, ListItem, ListItemIcon, Collapse } from "@material-ui/core";
 import { StyleRulesCallback, withStyles, WithStyles } from '@material-ui/core/styles';
 import { ReactElement } from "react";
-import Collapse from "@material-ui/core/Collapse/Collapse";
 import CircularProgress from '@material-ui/core/CircularProgress';
+import * as classnames from "classnames";
+
 import { ArvadosTheme } from '../../common/custom-theme';
+import { SidePanelRightArrowIcon } from '../icon/icon';
 
-type CssRules = 'list' | 'activeArrow' | 'inactiveArrow' | 'arrowRotate' | 'arrowTransition' | 'loader' | 'arrowVisibility';
+type CssRules = 'list' | 'active' | 'loader' | 'toggableIconContainer' | 'iconClose' | 'iconOpen' | 'toggableIcon';
 
 const styles: StyleRulesCallback<CssRules> = (theme: ArvadosTheme) => ({
     list: {
-        paddingBottom: '3px',
-        paddingTop: '3px',
+        padding: '3px 0px'
     },
-    activeArrow: {
-        color: theme.palette.primary.main,
+    loader: {
         position: 'absolute',
+        transform: 'translate(0px)',
+        top: '3px'
     },
-    inactiveArrow: {
+    toggableIconContainer: {
         color: theme.palette.grey["700"],
-        position: 'absolute',
+        height: '14px',
+        position: 'absolute'
     },
-    arrowTransition: {
-        transition: 'all 0.1s ease',
+    toggableIcon: {
+        fontSize: '14px'
     },
-    arrowRotate: {
-        transition: 'all 0.1s ease',
-        transform: 'rotate(-90deg)',
+    active: {
+        color: theme.palette.primary.main,
     },
-    arrowVisibility: {
-        opacity: 0,
+    iconClose: {
+        transition: 'all 0.1s ease',
     },
-    loader: {
-        position: 'absolute',
-        transform: 'translate(0px)',
-        top: '3px'
+    iconOpen: {
+        transition: 'all 0.1s ease',
+        transform: 'rotate(90deg)',
     }
 });
 
@@ -73,39 +73,44 @@ export const Tree = withStyles(styles)(
         render(): ReactElement<any> {
             const level = this.props.level ? this.props.level : 0;
             const { classes, render, toggleItemOpen, items, toggleItemActive, onContextMenu } = this.props;
-            const { list, inactiveArrow, activeArrow, loader } = classes;
+            const { list, loader, toggableIconContainer } = classes;
             return <List component="div" className={list}>
                 {items && items.map((it: TreeItem<T>, idx: number) =>
                     <div key={`item/${level}/${idx}`}>
                         <ListItem button className={list} style={{ paddingLeft: (level + 1) * 20 }}
-                                  onClick={() => toggleItemActive(it.id, it.status)}
-                                  onContextMenu={this.handleRowContextMenu(it)}>
+                            onClick={() => toggleItemActive(it.id, it.status)}
+                            onContextMenu={this.handleRowContextMenu(it)}>
                             {it.status === TreeItemStatus.Pending ?
-                                <CircularProgress size={10} className={loader}/> : null}
-                            {it.toggled && it.items && it.items.length === 0 ? null : this.renderArrow(it.status, it.active ? activeArrow : inactiveArrow, it.open, it.id)}
+                                <CircularProgress size={10} className={loader} /> : null}
+                            <i onClick={() => this.props.toggleItemOpen(it.id, it.status)}
+                                className={toggableIconContainer}>
+                                <ListItemIcon className={this.getToggableIconClassNames(it.open, it.active)}>
+                                    {it.toggled && it.items && it.items.length === 0 ? <span /> : <SidePanelRightArrowIcon />}
+                                </ListItemIcon>
+                            </i>
                             {render(it, level)}
                         </ListItem>
                         {it.items && it.items.length > 0 &&
-                        <Collapse in={it.open} timeout="auto" unmountOnExit>
-                            <Tree
-                                items={it.items}
-                                render={render}
-                                toggleItemOpen={toggleItemOpen}
-                                toggleItemActive={toggleItemActive}
-                                level={level + 1}
-                                onContextMenu={onContextMenu}/>
-                        </Collapse>}
+                            <Collapse in={it.open} timeout="auto" unmountOnExit>
+                                <Tree
+                                    items={it.items}
+                                    render={render}
+                                    toggleItemOpen={toggleItemOpen}
+                                    toggleItemActive={toggleItemActive}
+                                    level={level + 1}
+                                    onContextMenu={onContextMenu} />
+                            </Collapse>}
                     </div>)}
             </List>;
         }
 
-        renderArrow(status: TreeItemStatus, arrowClass: string, open: boolean, id: string) {
-            const { arrowTransition, arrowVisibility, arrowRotate } = this.props.classes;
-            return <i onClick={() => this.props.toggleItemOpen(id, status)}
-                      className={`
-                        ${arrowClass}
-                        ${status === TreeItemStatus.Pending ? arrowVisibility : ''}
-                        ${open ? `fas fa-caret-down ${arrowTransition}` : `fas fa-caret-down ${arrowRotate}`}`}/>;
+        getToggableIconClassNames = (isOpen?: boolean, isActive?: boolean) => {
+            const { iconOpen, iconClose, active, toggableIcon } = this.props.classes;
+            return classnames(toggableIcon, {
+                [iconOpen]: isOpen,
+                [iconClose]: !isOpen,
+                [active]: isActive
+            });
         }
 
         handleRowContextMenu = (item: TreeItem<T>) =>