From e5cdea8daf94f58f46a956a1e6bc16daaa4b2ed0 Mon Sep 17 00:00:00 2001 From: Jake McDermott Date: Thu, 3 Jan 2019 11:56:42 -0500 Subject: [PATCH] use constructor bound methods for logo --- src/components/TowerLogo/TowerLogo.jsx | 20 ++++++++++++++------ 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/src/components/TowerLogo/TowerLogo.jsx b/src/components/TowerLogo/TowerLogo.jsx index 6b9d0e3a30..10925a3046 100644 --- a/src/components/TowerLogo/TowerLogo.jsx +++ b/src/components/TowerLogo/TowerLogo.jsx @@ -12,18 +12,26 @@ class TowerLogo extends Component { super(props); this.state = { hover: false }; + + this.onClick = this.onClick.bind(this); + this.onHover = this.onHover.bind(this); } - onClick = () => { - const { history } = this.props; - history.push('/'); - }; + onClick () { + const { history, onClick: handleClick } = this.props; - onHover = () => { + if (!handleClick) return; + + history.push('/'); + + handleClick(); + } + + onHover () { const { hover } = this.state; this.setState({ hover: !hover }); - }; + } render () { const { hover } = this.state;