Logflare / logflare / 34886283300-1

83%

main: 83%

Build Type

Pull #3980

github

Committed by chasers

Commit Message

fix: let the LiveView drive every scroll

A chart click scrolled the page to the newest results, but a search did not.
Scrolling was also split between the server and a `data-tailing` attribute
that the hook read on every update. A fresh load with a querystring never
scrolled. The follow-up diffs of a search (aggregates, pagination buttons)
each ran `restoreScrollAnchor/0` and undid the scroll.

- Every initial event page pushes `scroll-to-bottom`. The initial page is
  always the newest events in range.
- Tail appends push `scroll-to-bottom` too, so `scrollToLatest/0` and
  `data-tailing` come out.
- The hook holds a pending scroll until a frame runs it. It skips the anchor
  restore until then.

The server owns scroll intent. The hook owns viewport stability, which needs
element geometry that the server cannot see.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YBPF1knvgfghTSdxuYryGL

Pull Request Pull Request #3980: fix: let the LiveView drive every scroll

Coverage Stats

3 of 4 new or added lines in 2 files covered. (75.0%)

2 existing lines in 2 files now uncovered.

14997 of 18167 relevant lines covered (82.55%)

1618.32 hits per line