diff options
Diffstat (limited to 'src/containers/settings/SettingsWindow.js')
-rw-r--r-- | src/containers/settings/SettingsWindow.js | 25 |
1 files changed, 12 insertions, 13 deletions
diff --git a/src/containers/settings/SettingsWindow.js b/src/containers/settings/SettingsWindow.js index 9bb64b6fe..e03c4c1d2 100644 --- a/src/containers/settings/SettingsWindow.js +++ b/src/containers/settings/SettingsWindow.js | |||
@@ -11,17 +11,19 @@ import ErrorBoundary from '../../components/util/ErrorBoundary'; | |||
11 | import { workspaceStore } from '../../features/workspaces'; | 11 | import { workspaceStore } from '../../features/workspaces'; |
12 | import UIStore from '../../stores/UIStore'; | 12 | import UIStore from '../../stores/UIStore'; |
13 | 13 | ||
14 | export default @inject('stores', 'actions') @observer class SettingsContainer extends Component { | 14 | @inject('stores', 'actions') |
15 | @observer | ||
16 | class SettingsContainer extends Component { | ||
15 | portalRoot = document.querySelector('#portalContainer'); | 17 | portalRoot = document.querySelector('#portalContainer'); |
16 | 18 | ||
17 | el = document.createElement('div'); | 19 | el = document.createElement('div'); |
18 | 20 | ||
19 | componentDidMount() { | 21 | componentDidMount() { |
20 | this.portalRoot.appendChild(this.el); | 22 | this.portalRoot.append(this.el); |
21 | } | 23 | } |
22 | 24 | ||
23 | componentWillUnmount() { | 25 | componentWillUnmount() { |
24 | this.portalRoot.removeChild(this.el); | 26 | this.el.remove(); |
25 | } | 27 | } |
26 | 28 | ||
27 | render() { | 29 | render() { |
@@ -36,16 +38,11 @@ export default @inject('stores', 'actions') @observer class SettingsContainer ex | |||
36 | ); | 38 | ); |
37 | 39 | ||
38 | return ReactDOM.createPortal( | 40 | return ReactDOM.createPortal( |
39 | ( | 41 | <ErrorBoundary> |
40 | <ErrorBoundary> | 42 | <Layout navigation={navigation} closeSettings={closeSettings}> |
41 | <Layout | 43 | {children} |
42 | navigation={navigation} | 44 | </Layout> |
43 | closeSettings={closeSettings} | 45 | </ErrorBoundary>, |
44 | > | ||
45 | {children} | ||
46 | </Layout> | ||
47 | </ErrorBoundary> | ||
48 | ), | ||
49 | this.el, | 46 | this.el, |
50 | ); | 47 | ); |
51 | } | 48 | } |
@@ -60,3 +57,5 @@ SettingsContainer.wrappedComponent.propTypes = { | |||
60 | ui: PropTypes.instanceOf(UIStore).isRequired, | 57 | ui: PropTypes.instanceOf(UIStore).isRequired, |
61 | }).isRequired, | 58 | }).isRequired, |
62 | }; | 59 | }; |
60 | |||
61 | export default SettingsContainer; | ||