Skip to content

Reduce terminal flicker in the dor file picker - #1092

Merged
nedtwigg merged 2 commits into
mainfrom
dor-o-flicker
Oct 9, 2026
Merged

nedtwigg merged 2 commits into
mainfrom
dor-o-flicker

Conversation

@nedtwigg

@nedtwigg nedtwigg commented Oct 9, 2026

Copy link
Copy Markdown
Member

Typing in dor o briefly cleared the result list before deferred ranking caught up, and every update rewrote the whole terminal. Seed each query with one bounded ranking slice, redraw only changed rows, and emit nothing for unchanged frames. Preserve full-width characters when clearing row tails and repaint after terminal resizing.

Validation: pnpm --filter dor test (260 passed, 1 skipped), spec and Markdown lints, and live search/navigation in the Dormouse innerdogfood harness. Regression coverage includes redraw scope, unchanged frames, handler replies, right-margin clearing, query edits, and resizing between compact and side-panel layouts.

nedtwigg and others added 2 commits October 9, 2026 09:47
Erase each changed row before writing it, so a character in the last
column survives without measuring the row's width. Key the frame cache
on the terminal size directly. Seed ranking once per input chunk from
the picker rather than on every query edit, so a chunk of several edits
pays one bounded slice. Share test helpers for painted rows and output
since the last check, and trim the spec rule to its contract.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying mouseterm with  Cloudflare Pages  Cloudflare Pages

Latest commit: 739d3fb
Status: ✅  Deploy successful!
Preview URL: https://2c6fa2f5.mouseterm.pages.dev
Branch Preview URL: https://dor-o-flicker.mouseterm.pages.dev

View logs

@nedtwigg
nedtwigg merged commit 589ab1c into main Oct 9, 2026
13 checks passed
@nedtwigg
nedtwigg deleted the dor-o-flicker branch October 9, 2026 17:03

This branch is waiting to be deployed

1 waiting deployment
hosted-preview — 739d3fb8 Waiting Oct 9, 2026 by nedtwigg via cleanup #1274
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants