Fix notifications being unnecessarily redrawn on state change (#15312)
Co-authored-by: Claire <claire.github-309c@sitedethib.com>
This commit is contained in:
		
							parent
							
								
									127c543a6e
								
							
						
					
					
						commit
						9669167aae
					
				
					 1 changed files with 7 additions and 1 deletions
				
			
		| 
						 | 
				
			
			@ -32,10 +32,16 @@ const messages = defineMessages({
 | 
			
		|||
  markAsRead : { id: 'notifications.mark_as_read', defaultMessage: 'Mark every notification as read' },
 | 
			
		||||
});
 | 
			
		||||
 | 
			
		||||
const getExcludedTypes = createSelector([
 | 
			
		||||
  state => state.getIn(['settings', 'notifications', 'shows']),
 | 
			
		||||
], (shows) => {
 | 
			
		||||
  return ImmutableList(shows.filter(item => !item).keys());
 | 
			
		||||
});
 | 
			
		||||
 | 
			
		||||
const getNotifications = createSelector([
 | 
			
		||||
  state => state.getIn(['settings', 'notifications', 'quickFilter', 'show']),
 | 
			
		||||
  state => state.getIn(['settings', 'notifications', 'quickFilter', 'active']),
 | 
			
		||||
  state => ImmutableList(state.getIn(['settings', 'notifications', 'shows']).filter(item => !item).keys()),
 | 
			
		||||
  getExcludedTypes,
 | 
			
		||||
  state => state.getIn(['notifications', 'items']),
 | 
			
		||||
], (showFilterBar, allowedType, excludedTypes, notifications) => {
 | 
			
		||||
  if (!showFilterBar || allowedType === 'all') {
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		
		Reference in a new issue