#SDSV-31 - Cleanup, add tooltips

MetaCell · Jun 7, 2024 · ee3dfb0 · ee3dfb0
1 parent b846680
commit ee3dfb0
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 14 deletions.
diff --git a/src/components/GraphViewer/GraphViewer.js b/src/components/GraphViewer/GraphViewer.js
@@ -368,11 +368,11 @@ const GraphViewer = (props) => {
               <MenuItem selected={TOP_DOWN.layout === selectedLayout.layout} onClick={() => handleLayoutChange(TOP_DOWN)}>{TOP_DOWN.label}</MenuItem>
               <MenuItem selected={LEFT_RIGHT.layout === selectedLayout.layout} onClick={() => handleLayoutChange(LEFT_RIGHT)}>{LEFT_RIGHT.label}</MenuItem>
             </Menu>
+            <Tooltip id="button-report" title="Change Graph Layout">
             <IconButton area-label="GraphLayout" aria-controls="layout-menu" aria-haspopup="true" onClick={handleLayoutClick}>
-              <Tooltip id="button-report" title="Change Graph Layout">
                 <ViewTypeIcon />
-              </Tooltip>
             </IconButton>
+            </Tooltip>
             <IconButton area-label="ZoomIn" onClick={(e) => zoomIn()}>
               <Tooltip id="button-report" title="Zoom In">
                 <AddRoundedIcon />

diff --git a/src/components/NodeDetailView/NodeDetailView.js b/src/components/NodeDetailView/NodeDetailView.js
@@ -2,7 +2,7 @@ import {Box} from "@material-ui/core";
 import NodeFooter from "./Footers/Footer";
 import DetailsFactory from './factory';
 import Breadcrumbs from "./Details/Views/Breadcrumbs";
-import { IconButton } from '@material-ui/core';
+import { IconButton, Tooltip } from '@material-ui/core';
 import { subject_key, protocols_key, contributors_key } from '../../constants';
 import {TuneRounded} from "@material-ui/icons";
 import { useSelector, useDispatch } from 'react-redux'
@@ -79,7 +79,9 @@ const NodeDetailView = (props) => {
       }
       <NodeFooter />
       { !showSettingsContent && <Box className='overlay-button-container'>
+      <Tooltip id="button-report" title="Open Metadata Settings">
         <IconButton className="overlay-button" onClick={toggleContent}><TuneRounded /></IconButton>
+      </Tooltip>
       </Box> }
     </Box>
   );

diff --git a/src/components/NodeDetailView/settings/SettingsItem.js b/src/components/NodeDetailView/settings/SettingsItem.js
@@ -67,17 +67,18 @@ const SettingsItem = props => {
           onClick={toggleItemDisabled}
           disableRipple
         >
-          <Tooltip id={item.label} title={"Toggle Property Visibility"}>
-            {!item.visible ? (
-              <AddCircleOutlineIcon
-                style={{ color: "#3779E1", fontSize: "1rem" }}
-              />
-            ) : (
-              <RemoveCircleOutlineIcon
-                style={{ color: "#ED745D", fontSize: "1rem" }}
-              />
-            )}
-          </Tooltip>
+
+        {!item.visible ? (
+          <Tooltip id={item.label} title={"Show Property"}>
+          <AddCircleOutlineIcon
+            style={{ color: "#3779E1", fontSize: "1rem" }}
+          /></Tooltip>
+        ) : (
+          <Tooltip id={item.label} title={"Hide Property"}>
+          <RemoveCircleOutlineIcon
+            style={{ color: "#ED745D", fontSize: "1rem" }}
+          /></Tooltip>
+        )}
         </IconButton>
       </ListItemSecondaryAction>
     </ListItem>