feat/auto-scroll #17

Merged

nrupard merged 12 commits from feat/auto-scroll into main

2026-04-14 22:16:44 +02:00

nrupard commented

2026-04-14 22:16:40 +02:00

Owner

No description provided.

nrupard added 12 commits

2026-04-14 22:16:40 +02:00

docs(auto-scroll): spec for chat auto-scroll behavior 09769b5f59

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

docs(auto-scroll): implementation plan for chat auto-scroll d3ab5967a6

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

feat(chat): scaffold use_auto_scroll hook 71eee96771

feat(chat): add localStorage last-seen helpers 7442b7db84

feat(chat): add DOM scroll primitives for auto-scroll hook 0c7e2acd2e

feat(chat): implement auto-scroll open + sticky-bottom logic 8b2020d60a

feat(chat): auto-dismiss pill and advance last-seen on manual scroll a808741344

feat(chat): integrate auto-scroll into room view 3ce27a704d

feat(chat): integrate auto-scroll into DM view 3a96d37a77

fix(chat): use Signal.read() for struct-field access and wrap message group for keying b2a34de5d5

fix(chat): sticky-bottom scroll + guard stale signal in visibilitychange 2336d70e7f

Sticky-bottom was misfiring because is_near_bottom was sampled after
Dioxus appended the new row, inflating scrollHeight. Track at_bottom
continuously via the scroll listener and read the remembered value on
append instead.

Also guard the pre-existing visibilitychange listener in dm_view.rs:
it's registered on document and leaked via cb.forget, so it outlives
the component and panicked when visibility_tick dropped.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

fix(chat): always sticky-scroll for own messages b8f2222975

Opening a chat with unseen messages leaves at_bottom=false after
scrolling to the first unseen. Without this, the sender's own
next message would show the "new messages" pill instead of
sticking to the bottom.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>