Skip to content
Snippets Groups Projects
Commit 4815f375 authored by Karishma Chadha's avatar Karishma Chadha
Browse files

Address PR comments.

parent b8928afa
No related merge requests found
......@@ -134,7 +134,8 @@ class MenuBar extends React.Component {
constructor (props) {
super(props);
bindAll(this, [
'handleLanguageMouseUp'
'handleLanguageMouseUp',
'handleRestoreOption'
]);
}
handleLanguageMouseUp (e) {
......@@ -142,6 +143,12 @@ class MenuBar extends React.Component {
this.props.onClickLanguage(e);
}
}
handleRestoreOption (restoreFun) {
return () => {
restoreFun();
this.props.onRequestCloseEdit();
};
}
render () {
return (
<Box className={styles.menuBar}>
......@@ -283,7 +290,7 @@ class MenuBar extends React.Component {
<DeletionRestorer>{(handleRestore, {restorable, deletedItem}) => (
<MenuItem
className={classNames({[styles.disabled]: !restorable})}
onClick={handleRestore}
onClick={this.handleRestoreOption(handleRestore)}
>
{deletedItem === 'Sprite' ?
<FormattedMessage
......
......@@ -38,9 +38,11 @@ class DeletionRestorer extends React.Component {
/* eslint-enable no-unused-vars */
...props
} = this.props;
props.restorable = typeof this.props.restore === 'function';
props.deletedItem = this.props.deletedItem;
return this.props.children(this.restoreDeletion, props);
const restorable = typeof this.props.restore === 'function';
return this.props.children(this.restoreDeletion, {
...props,
restorable
});
}
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment