Merge pull request #13451 from annando/no-jump

Avoid jumps in the timeline after "received" has been selected
This commit is contained in:
Hypolite Petovan 2023-09-20 09:08:14 -04:00 committed by GitHub
commit e153787ac7
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -348,14 +348,16 @@ class Network extends Timeline
$this->order = 'commented'; $this->order = 'commented';
} }
$this->selectedTab = $this->selectedTab ?? $this->order;
// Upon updates in the background and order by last comment we order by received date, // Upon updates in the background and order by last comment we order by received date,
// since otherwise the feed will optically jump, when some already visible thread has been updated. // since otherwise the feed will optically jump, when some already visible thread has been updated.
if ($this->update && ($this->selectedTab == TimelineEntity::COMMENTED)) { if ($this->update && ($this->selectedTab == TimelineEntity::COMMENTED)) {
$this->order = 'received'; $this->order = 'received';
$request['last_received'] = $request['last_commented'] ?? null;
$request['first_received'] = $request['first_commented'] ?? null;
} }
$this->selectedTab = $this->selectedTab ?? $this->order;
// Prohibit combined usage of "star" and "mention" // Prohibit combined usage of "star" and "mention"
if ($this->selectedTab == TimelineEntity::STAR) { if ($this->selectedTab == TimelineEntity::STAR) {
$this->mention = false; $this->mention = false;