m/metamask-extension
1
0
Fork 0
mirror of https://github.com/kremalicious/metamask-extension.git synced 2024-11-22 18:00:18 +01:00
Code

Fix threebox last updated proptype (#7375)

Browse Source 
* Use child components for multiple notifications component

The multiple notifications component has been updated to take its child
components as children rather than as a props array, so that the child
components are never executed in the case where they aren't needed.

* Fix threebox last updated proptype
This commit is contained in:
Mark Stacey 2019-11-10 01:30:07 -05:00 committed by GitHub
parent 02aebc2e03
commit ec1f3fa19a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 28 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
ui/app/components/app/multiple-notifications/multiple-notifications.component.js
View File

@ -4,7 +4,7 @@ import PropTypes from 'prop-types'
export default class MultipleNotifications extends PureComponent {
static propTypes = {
notifications: PropTypes.array,
children: PropTypes.array,
classNames: PropTypes.array,
}
@ -14,11 +14,10 @@ export default class MultipleNotifications extends PureComponent {
render () {
const { showAll } = this.state
const { notifications, classNames = [] } = this.props
const { children, classNames = [] } = this.props
const notificationsToBeRendered = notifications.filter(notificationConfig => notificationConfig.shouldBeRendered)
if (notificationsToBeRendered.length === 0) {
const childrenToRender = children.filter(child => child)
if (childrenToRender.length === 0) {
return null
}
@ -29,12 +28,12 @@ export default class MultipleNotifications extends PureComponent {
'home-notification-wrapper--show-first': !showAll,
})}
>
{ notificationsToBeRendered.map(notificationConfig => notificationConfig.component) }
{ childrenToRender }
<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', {
{childrenToRender.length > 1 ? <i className={classnames('fa fa-sm fa-sort-amount-asc', {
'flipped': !showAll,
})} /> : null}
</div>

41
ui/app/pages/home/home.component.js
View File

@ -42,7 +42,7 @@ export default class Home extends PureComponent {
selectedAddress: PropTypes.string,
restoreFromThreeBox: PropTypes.func,
setShowRestorePromptToFalse: PropTypes.func,
threeBoxLastUpdated: PropTypes.string,
threeBoxLastUpdated: PropTypes.number,
}
componentWillMount () {
@ -119,10 +119,10 @@ export default class Home extends PureComponent {
<TransactionView>
<MultipleNotifications
className
notifications={[
{
shouldBeRendered: showPrivacyModeNotification,
component: <HomeNotification
>
{
showPrivacyModeNotification
? <HomeNotification
descriptionText={t('privacyModeDefault')}
acceptText={t('learnMore')}
onAccept={() => {
@ -134,11 +134,12 @@ export default class Home extends PureComponent {
unsetMigratedPrivacyMode()
}}
key="home-privacyModeDefault"
/>,
},
{
shouldBeRendered: shouldShowSeedPhraseReminder,
component: <HomeNotification
/>
: null
}
{
shouldShowSeedPhraseReminder
? <HomeNotification
descriptionText={t('backupApprovalNotice')}
acceptText={t('backupNow')}
onAccept={() => {
@ -150,12 +151,13 @@ export default class Home extends PureComponent {
}}
infoText={t('backupApprovalInfo')}
key="home-backupApprovalNotice"
/>,
},
{
shouldBeRendered: threeBoxLastUpdated && showRestorePrompt,
component: <HomeNotification
descriptionText={t('restoreWalletPreferences', [ formatDate(parseInt(threeBoxLastUpdated), 'M/d/y') ])}
/>
: null
}
{
threeBoxLastUpdated && showRestorePrompt
? <HomeNotification
descriptionText={t('restoreWalletPreferences', [ formatDate(threeBoxLastUpdated, 'M/d/y') ])}
acceptText={t('restore')}
ignoreText={t('noThanks')}
infoText={t('dataBackupFoundInfo')}
@ -169,9 +171,10 @@ export default class Home extends PureComponent {
setShowRestorePromptToFalse()
}}
key="home-privacyModeDefault"
/>,
},
]}/>
/>
: null
}
</MultipleNotifications>
</TransactionView>
)
: null }