Merge pull request #9705 from darkwing/avoid-selected-account-memory-leak

Prevent memory leak from selected account copy tooltip
feature/default_network_editable
David Walsh 4 years ago committed by GitHub
commit f15b4949a1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 13
      ui/app/components/app/selected-account/selected-account.component.js

@ -18,6 +18,17 @@ class SelectedAccount extends Component {
selectedIdentity: PropTypes.object.isRequired, selectedIdentity: PropTypes.object.isRequired,
} }
componentDidMount () {
this.copyTimeout = null
}
componentWillUnmount () {
if (this.copyTimeout) {
clearTimeout(this.copyTimeout)
this.copyTimeout = null
}
}
render () { render () {
const { t } = this.context const { t } = this.context
const { selectedIdentity } = this.props const { selectedIdentity } = this.props
@ -34,7 +45,7 @@ class SelectedAccount extends Component {
className="selected-account__clickable" className="selected-account__clickable"
onClick={() => { onClick={() => {
this.setState({ copied: true }) this.setState({ copied: true })
setTimeout(() => this.setState({ copied: false }), 3000) this.copyTimeout = setTimeout(() => this.setState({ copied: false }), 3000)
copyToClipboard(checksummedAddress) copyToClipboard(checksummedAddress)
}} }}
> >

Loading…
Cancel
Save