oiupoyt/discordo
1
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fix(ui/chat): preserve cursor behavior when loading older messages (#780)

Browse source
This commit is contained in:
ayn2op 2026-03-21 17:38:56 -04:00 • committed by GitHub
parent 3d014dd6d2
commit b2f079b700
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
internal/ui/chat/messages_list.go
View file

@ -896,6 +896,7 @@ func (ml *messagesList) HandleEvent(event tview.Event) tview.Command {
if selectedChannel == nil || selectedChannel.ID != event.ChannelID {
return nil
}
prevCursor := ml.Cursor()
// Defensive invalidation if Discord returns overlapping windows.
for _, message := range event.Older {
@ -903,7 +904,17 @@ func (ml *messagesList) HandleEvent(event tview.Event) tview.Command {
}
ml.messages = slices.Concat(event.Older, ml.messages)
ml.invalidateRows()
ml.SetCursor(len(event.Older) - 1)
switch {
case prevCursor == 0:
// Preserve "SelectUp at top" semantics: move to the next older message.
ml.SetCursor(len(event.Older) - 1)
case prevCursor > 0:
// Keep selection on the same message after prepend shifts indexes.
ml.SetCursor(prevCursor + len(event.Older))
default:
ml.SetCursor(prevCursor)
}
return nil
}
return ml.Model.HandleEvent(event)