Change timelines to add new items to pending items when scrolled down (#11867)
This commit is contained in:
		
							parent
							
								
									605b78c717
								
							
						
					
					
						commit
						57c7ce5e7f
					
				
					 2 changed files with 9 additions and 5 deletions
				
			
		|  | @ -35,12 +35,12 @@ const notificationToMap = notification => ImmutableMap({ | ||||||
| }); | }); | ||||||
| 
 | 
 | ||||||
| const normalizeNotification = (state, notification, usePendingItems) => { | const normalizeNotification = (state, notification, usePendingItems) => { | ||||||
|   if (usePendingItems) { |   const top = state.get('top'); | ||||||
|  | 
 | ||||||
|  |   if (usePendingItems || !top || !state.get('pendingItems').isEmpty()) { | ||||||
|     return state.update('pendingItems', list => list.unshift(notificationToMap(notification))).update('unread', unread => unread + 1); |     return state.update('pendingItems', list => list.unshift(notificationToMap(notification))).update('unread', unread => unread + 1); | ||||||
|   } |   } | ||||||
| 
 | 
 | ||||||
|   const top = state.get('top'); |  | ||||||
| 
 |  | ||||||
|   if (!top) { |   if (!top) { | ||||||
|     state = state.update('unread', unread => unread + 1); |     state = state.update('unread', unread => unread + 1); | ||||||
|   } |   } | ||||||
|  | @ -63,6 +63,8 @@ const expandNormalizedNotifications = (state, notifications, next, usePendingIte | ||||||
| 
 | 
 | ||||||
|   return state.withMutations(mutable => { |   return state.withMutations(mutable => { | ||||||
|     if (!items.isEmpty()) { |     if (!items.isEmpty()) { | ||||||
|  |       usePendingItems = usePendingItems || !mutable.get('top') || !mutable.get('pendingItems').isEmpty(); | ||||||
|  | 
 | ||||||
|       mutable.update(usePendingItems ? 'pendingItems' : 'items', list => { |       mutable.update(usePendingItems ? 'pendingItems' : 'items', list => { | ||||||
|         const lastIndex = 1 + list.findLastIndex( |         const lastIndex = 1 + list.findLastIndex( | ||||||
|           item => item !== null && (compareId(item.get('id'), items.last().get('id')) > 0 || item.get('id') === items.last().get('id')) |           item => item !== null && (compareId(item.get('id'), items.last().get('id')) > 0 || item.get('id') === items.last().get('id')) | ||||||
|  |  | ||||||
|  | @ -40,6 +40,7 @@ const expandNormalizedTimeline = (state, timeline, statuses, next, isPartial, is | ||||||
|     if (timeline.endsWith(':pinned')) { |     if (timeline.endsWith(':pinned')) { | ||||||
|       mMap.set('items', statuses.map(status => status.get('id'))); |       mMap.set('items', statuses.map(status => status.get('id'))); | ||||||
|     } else if (!statuses.isEmpty()) { |     } else if (!statuses.isEmpty()) { | ||||||
|  |       usePendingItems = usePendingItems || !mMap.get('top') || !mMap.get('pendingItems').isEmpty(); | ||||||
|       mMap.update(usePendingItems ? 'pendingItems' : 'items', ImmutableList(), oldIds => { |       mMap.update(usePendingItems ? 'pendingItems' : 'items', ImmutableList(), oldIds => { | ||||||
|         const newIds = statuses.map(status => status.get('id')); |         const newIds = statuses.map(status => status.get('id')); | ||||||
| 
 | 
 | ||||||
|  | @ -60,7 +61,9 @@ const expandNormalizedTimeline = (state, timeline, statuses, next, isPartial, is | ||||||
| }; | }; | ||||||
| 
 | 
 | ||||||
| const updateTimeline = (state, timeline, status, usePendingItems) => { | const updateTimeline = (state, timeline, status, usePendingItems) => { | ||||||
|   if (usePendingItems) { |   const top = state.getIn([timeline, 'top']); | ||||||
|  | 
 | ||||||
|  |   if (usePendingItems || !top || !state.getIn([timeline, 'pendingItems']).isEmpty()) { | ||||||
|     if (state.getIn([timeline, 'pendingItems'], ImmutableList()).includes(status.get('id')) || state.getIn([timeline, 'items'], ImmutableList()).includes(status.get('id'))) { |     if (state.getIn([timeline, 'pendingItems'], ImmutableList()).includes(status.get('id')) || state.getIn([timeline, 'items'], ImmutableList()).includes(status.get('id'))) { | ||||||
|       return state; |       return state; | ||||||
|     } |     } | ||||||
|  | @ -68,7 +71,6 @@ const updateTimeline = (state, timeline, status, usePendingItems) => { | ||||||
|     return state.update(timeline, initialTimeline, map => map.update('pendingItems', list => list.unshift(status.get('id'))).update('unread', unread => unread + 1)); |     return state.update(timeline, initialTimeline, map => map.update('pendingItems', list => list.unshift(status.get('id'))).update('unread', unread => unread + 1)); | ||||||
|   } |   } | ||||||
| 
 | 
 | ||||||
|   const top        = state.getIn([timeline, 'top']); |  | ||||||
|   const ids        = state.getIn([timeline, 'items'], ImmutableList()); |   const ids        = state.getIn([timeline, 'items'], ImmutableList()); | ||||||
|   const includesId = ids.includes(status.get('id')); |   const includesId = ids.includes(status.get('id')); | ||||||
|   const unread     = state.getIn([timeline, 'unread'], 0); |   const unread     = state.getIn([timeline, 'unread'], 0); | ||||||
|  |  | ||||||
		Loading…
	
		Reference in a new issue