diff options
author | Thomas Huang <tmashuang@users.noreply.github.com> | 2019-08-07 04:35:18 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-08-07 04:35:18 +0800 |
commit | 86ad9564a064fd6158dab6a3c9e5b10614ef6e68 (patch) | |
tree | 6032d7a4ae67371889eece1d8490c26d5a119dd5 /ui | |
parent | 835d4fbb139185a0026a24bb2f25ae55d7ee2baf (diff) | |
parent | 372241e93588be00ecf675107028a6e8536ca7d6 (diff) | |
download | tangerine-wallet-browser-86ad9564a064fd6158dab6a3c9e5b10614ef6e68.tar tangerine-wallet-browser-86ad9564a064fd6158dab6a3c9e5b10614ef6e68.tar.gz tangerine-wallet-browser-86ad9564a064fd6158dab6a3c9e5b10614ef6e68.tar.bz2 tangerine-wallet-browser-86ad9564a064fd6158dab6a3c9e5b10614ef6e68.tar.lz tangerine-wallet-browser-86ad9564a064fd6158dab6a3c9e5b10614ef6e68.tar.xz tangerine-wallet-browser-86ad9564a064fd6158dab6a3c9e5b10614ef6e68.tar.zst tangerine-wallet-browser-86ad9564a064fd6158dab6a3c9e5b10614ef6e68.zip |
Merge pull request #6957 from MetaMask/Version-v7.0.0
Version 7.0.0
Diffstat (limited to 'ui')
-rw-r--r-- | ui/app/components/app/multiple-notifications/index.scss | 2 | ||||
-rw-r--r-- | ui/app/components/app/multiple-notifications/multiple-notifications.component.js | 32 |
2 files changed, 19 insertions, 15 deletions
diff --git a/ui/app/components/app/multiple-notifications/index.scss b/ui/app/components/app/multiple-notifications/index.scss index e3ee39a74..e8d064bc0 100644 --- a/ui/app/components/app/multiple-notifications/index.scss +++ b/ui/app/components/app/multiple-notifications/index.scss @@ -3,7 +3,6 @@ display: flex; flex-direction: column; width: 472px; - height: 116px; position: absolute; bottom: 0; right: 0; @@ -37,7 +36,6 @@ } .home-notification-wrapper--show-all { - height: 356px;; justify-content: flex-end; margin-bottom: 0; diff --git a/ui/app/components/app/multiple-notifications/multiple-notifications.component.js b/ui/app/components/app/multiple-notifications/multiple-notifications.component.js index 09020c467..040890e18 100644 --- a/ui/app/components/app/multiple-notifications/multiple-notifications.component.js +++ b/ui/app/components/app/multiple-notifications/multiple-notifications.component.js @@ -18,21 +18,27 @@ export default class MultipleNotifications extends PureComponent { const notificationsToBeRendered = notifications.filter(notificationConfig => notificationConfig.shouldBeRendered) - return (<div - className={classnames(...classNames, { - 'home-notification-wrapper--show-all': showAll, - 'home-notification-wrapper--show-first': !showAll, - })} - > - { notificationsToBeRendered.map(notificationConfig => notificationConfig.component) } + if (notificationsToBeRendered.length === 0) { + return null + } + + return ( <div - className="home-notification-wrapper__i-container" - onClick={() => this.setState({ showAll: !showAll })} + className={classnames(...classNames, { + 'home-notification-wrapper--show-all': showAll, + 'home-notification-wrapper--show-first': !showAll, + })} > - {notificationsToBeRendered.length > 1 ? <i className={classnames('fa fa-sm fa-sort-amount-asc', { - 'flipped': !showAll, - })} /> : null} + { notificationsToBeRendered.map(notificationConfig => notificationConfig.component) } + <div + className="home-notification-wrapper__i-container" + onClick={() => this.setState({ showAll: !showAll })} + > + {notificationsToBeRendered.length > 1 ? <i className={classnames('fa fa-sm fa-sort-amount-asc', { + 'flipped': !showAll, + })} /> : null} + </div> </div> - </div>) + ) } } |