Skip to content

Add scroll locking feature with proportional sync#132

Merged
jmthomas merged 4 commits intomasterfrom
scroll_lock
Jan 9, 2026
Merged

Add scroll locking feature with proportional sync#132
jmthomas merged 4 commits intomasterfrom
scroll_lock

Conversation

@jmthomas
Copy link
Collaborator

@jmthomas jmthomas commented Jan 9, 2026

Implements the lockScrolling option (now enabled by default) that synchronizes scrolling between left and right editors. Uses proportional positioning to handle documents of different sizes - scrolling to 50% in one editor scrolls the other to 50% as well.

Closes #21

Implements the lockScrolling option (now enabled by default) that
synchronizes scrolling between left and right editors. Uses proportional
positioning to handle documents of different sizes - scrolling to 50%
in one editor scrolls the other to 50% as well.

Closes #21

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
jmthomas and others added 3 commits January 8, 2026 17:19
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Use Cypress retry mechanism instead of fixed wait for more reliable
scroll sync assertion.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Wait for Ace gutter cells before testing scroll sync
- Force Ace editor resize in headless mode to ensure lineHeight is calculated
- Add defensive lineHeight fallback for headless environments
- Upload Cypress screenshots/videos as artifacts when tests fail

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@jmthomas jmthomas merged commit 91bff26 into master Jan 9, 2026
2 checks passed
@jmthomas jmthomas deleted the scroll_lock branch January 9, 2026 23:04
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.

Any updates with scroll locking?

1 participant