Only show notification expand icon when there are > 1 notifications that 'shouldBeRendered' (#6965)

feature/default_network_editable
Dan J Miller 5 years ago committed by GitHub
parent 010e3927da
commit 3136dd39ea
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 9
      ui/app/components/app/multiple-notifications/multiple-notifications.component.js

@ -16,21 +16,20 @@ export default class MultipleNotifications extends PureComponent {
const { showAll } = this.state
const { notifications, classNames = [] } = this.props
const notificationsToBeRendered = notifications.filter(notificationConfig => notificationConfig.shouldBeRendered)
return (<div
className={classnames(...classNames, {
'home-notification-wrapper--show-all': showAll,
'home-notification-wrapper--show-first': !showAll,
})}
>
{notifications
.filter(notificationConfig => notificationConfig.shouldBeRendered)
.map(notificationConfig => notificationConfig.component)
}
{ notificationsToBeRendered.map(notificationConfig => notificationConfig.component) }
<div
className="home-notification-wrapper__i-container"
onClick={() => this.setState({ showAll: !showAll })}
>
{notifications.length > 1 ? <i className={classnames('fa fa-sm fa-sort-amount-asc', {
{notificationsToBeRendered.length > 1 ? <i className={classnames('fa fa-sm fa-sort-amount-asc', {
'flipped': !showAll,
})} /> : null}
</div>

Loading…
Cancel
Save