Skip to content

Fix find match scrolling#158

Merged
ThisIs-Developer merged 1 commit into
mainfrom
codex/fix-find-box-scrolling
Jun 4, 2026
Merged

Fix find match scrolling#158
ThisIs-Developer merged 1 commit into
mainfrom
codex/fix-find-box-scrolling

Conversation

@ThisIs-Developer
Copy link
Copy Markdown
Owner

Summary

  • Fix find navigation so active matches scroll into view in long wrapped documents.
  • Use the editor highlight layer's real wrapped match position instead of estimating by logical line number.
  • Keep scroll state tied to the textarea so highlights and line numbers stay aligned with editor scrolling.

Root Cause

Find navigation estimated the target position as lineIndex * lineHeight, which works for short documents but fails when long paragraphs wrap across many visual rows.

Fixes #154.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
markdown-viwer Ready Ready Preview, Comment Jun 4, 2026 5:39pm

@ThisIs-Developer ThisIs-Developer merged commit 20b6474 into main Jun 4, 2026
6 checks passed
@ThisIs-Developer ThisIs-Developer deleted the codex/fix-find-box-scrolling branch June 4, 2026 17:39
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.

Find box has scrolling problem

1 participant