From 41f9bf109ba2bb19def66a147303ad0fc8b09e70 Mon Sep 17 00:00:00 2001
From: Jessie Keck <jessie.keck@gmail.com>
Date: Thu, 31 Jan 2019 14:38:27 -0800
Subject: [PATCH] Add an action state on the Info icon when the Info panel is
open.
---
src/components/WindowSideBar.js | 2 +-
src/components/WindowSideBarButtons.js | 16 +++++++++++++++-
2 files changed, 16 insertions(+), 2 deletions(-)
diff --git a/src/components/WindowSideBar.js b/src/components/WindowSideBar.js
index 3804eb814..4221c0fcb 100644
--- a/src/components/WindowSideBar.js
+++ b/src/components/WindowSideBar.js
@@ -38,7 +38,7 @@ class WindowSideBar extends Component {
}}
>
<List>
- <WindowSideBarButtons windowId={windowId} />
+ <WindowSideBarButtons windowId={windowId} sideBarPanel={sideBarPanel} />
</List>
</Drawer>
<Drawer
diff --git a/src/components/WindowSideBarButtons.js b/src/components/WindowSideBarButtons.js
index 955ace9cb..037349f63 100644
--- a/src/components/WindowSideBarButtons.js
+++ b/src/components/WindowSideBarButtons.js
@@ -7,6 +7,16 @@ import InfoIcon from '@material-ui/icons/Info';
*
*/
class WindowSideBarButtons extends Component {
+ /**
+ * sideBarPanelCurrentlySelected - return if the given sideBarPanel is currently selected
+ * @return Boolean
+ */
+ sideBarPanelCurrentlySelected(panelType) {
+ const { sideBarPanel } = this.props;
+
+ return sideBarPanel === panelType;
+ }
+
/**
* render
*
@@ -21,7 +31,9 @@ class WindowSideBarButtons extends Component {
color="inherit"
onClick={() => (toggleWindowSideBarPanel('info'))}
>
- <InfoIcon />
+ <InfoIcon
+ color={this.sideBarPanelCurrentlySelected('info') ? 'action' : 'inherit'}
+ />
</IconButton>
</>
);
@@ -30,10 +42,12 @@ class WindowSideBarButtons extends Component {
WindowSideBarButtons.propTypes = {
toggleWindowSideBarPanel: PropTypes.func,
+ sideBarPanel: PropTypes.string,
};
WindowSideBarButtons.defaultProps = {
toggleWindowSideBarPanel: () => {},
+ sideBarPanel: 'closed',
};
export default WindowSideBarButtons;
--
GitLab