Browse Source

Revert "Change timelines to add new items to pending items when scrolled down #11867" (#11921)

Fix #11912
master^2
Eugen Rochko 4 years ago
committed by GitHub
parent
commit
8c4b5f7cef
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 6 deletions
  1. +3
    -4
      app/javascript/mastodon/reducers/notifications.js
  2. +3
    -2
      app/javascript/mastodon/reducers/timelines.js

+ 3
- 4
app/javascript/mastodon/reducers/notifications.js View File

@@ -38,10 +38,9 @@ const notificationToMap = notification => ImmutableMap({
});

const normalizeNotification = (state, notification, usePendingItems) => {
const top = state.get('top');
const mounted = state.get('mounted');
const top = state.get('top');

if (usePendingItems || (!top && mounted) || !state.get('pendingItems').isEmpty()) {
if (usePendingItems || !state.get('pendingItems').isEmpty()) {
return state.update('pendingItems', list => list.unshift(notificationToMap(notification))).update('unread', unread => unread + 1);
}

@@ -67,7 +66,7 @@ const expandNormalizedNotifications = (state, notifications, next, isLoadingRece

return state.withMutations(mutable => {
if (!items.isEmpty()) {
usePendingItems = isLoadingRecent && (usePendingItems || (!mutable.get('top') && mutable.get('mounted')) || !mutable.get('pendingItems').isEmpty());
usePendingItems = isLoadingRecent && (usePendingItems || !mutable.get('pendingItems').isEmpty());

mutable.update(usePendingItems ? 'pendingItems' : 'items', list => {
const lastIndex = 1 + list.findLastIndex(


+ 3
- 2
app/javascript/mastodon/reducers/timelines.js View File

@@ -40,7 +40,8 @@ const expandNormalizedTimeline = (state, timeline, statuses, next, isPartial, is
if (timeline.endsWith(':pinned')) {
mMap.set('items', statuses.map(status => status.get('id')));
} else if (!statuses.isEmpty()) {
usePendingItems = isLoadingRecent && (usePendingItems || !mMap.get('top') || !mMap.get('pendingItems').isEmpty());
usePendingItems = isLoadingRecent && (usePendingItems || !mMap.get('pendingItems').isEmpty());

mMap.update(usePendingItems ? 'pendingItems' : 'items', ImmutableList(), oldIds => {
const newIds = statuses.map(status => status.get('id'));

@@ -63,7 +64,7 @@ const expandNormalizedTimeline = (state, timeline, statuses, next, isPartial, is
const updateTimeline = (state, timeline, status, usePendingItems) => {
const top = state.getIn([timeline, 'top']);

if (usePendingItems || !top || !state.getIn([timeline, 'pendingItems']).isEmpty()) {
if (usePendingItems || !state.getIn([timeline, 'pendingItems']).isEmpty()) {
if (state.getIn([timeline, 'pendingItems'], ImmutableList()).includes(status.get('id')) || state.getIn([timeline, 'items'], ImmutableList()).includes(status.get('id'))) {
return state;
}


Loading…
Cancel
Save