Loading indicator on message list when data is stale

[anh2111htd]

Issue #3387. The check if data is stale is const hasCaughtUp = caughtUp.newer; where caughtUp is getCaughtUpForNarrow(state, props.narrow). Based on the use case describe in #3387 , I understand we want to indicate to the users if they have had all new message (when they open from a notification for example), so I check for caughtUp.newer here. Let me know if this check is not sufficient or incorrect.

Screenshot:

[chrisbobbe]

Hi @anh2111htd, thanks for your work on this, and I'm sorry we didn't get around to reviewing this — not your fault! 🙂

I'm closing this, as #3901 also shows the notice on the unreads screen (far left on the tabs at the bottom; an "inbox" icon), so it also fixes #3025, and several other screens that use data from the event system. Also, the loading state we're concerned with here is stored in session.loading, not state.caughtUp; you can read the JSDoc about state.caughtUp in reduxTypes.js. 🙂