[PR #742] feat(DiffRenderable): synchronize scroll between split view panes in diff.ts #749

New issue

Open

opened 2026-03-02 23:47:57 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 23:47:57 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/anomalyco/opentui/pull/742
Author: @viralcodex
Created: 2/25/2026
Status: 🔄 Open

Base: main ← Head: sync-scroll-diff

📝 Commits (7)

3d8df00 feat(DiffRenderable): synchronize scroll between split view panes in diff.ts
2ed5649 fix(DiffRenderable): update container reference for side detection in isInsideSide method
fd23733 fix(DiffRenderable): remove unnecessary blank line in split view scroll test
386c622 Merge branch 'main' into sync-scroll-diff
608e200 feat(DiffRenderable): add syncScroll option for split view synchronization
11e4c43 Merge branch 'sync-scroll-diff' of https://github.com/viralcodex/opentui into sync-scroll-diff
7ad7f34 Merge branch 'main' into sync-scroll-diff

📊 Changes

2 files changed (+123 additions, -1 deletions)

View changed files

📝 packages/core/src/renderables/Diff.test.ts (+82 -1)
📝 packages/core/src/renderables/Diff.ts (+41 -0)

📄 Description

Fixes: #563
Hi @kommander and @simonklee,

Added this synced scroll in code diff as mentioned in this issue...

I was thinking to keep both separate and synced scroll which can be toggled through hotkeys depending on user preferences.

Whatever you guys say 😄

https://github.com/user-attachments/assets/bdc27805-d5e4-4f1c-b1f4-d31b94da4601

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/anomalyco/opentui/pull/742 **Author:** [@viralcodex](https://github.com/viralcodex) **Created:** 2/25/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `sync-scroll-diff` --- ### 📝 Commits (7) - [`3d8df00`](https://github.com/anomalyco/opentui/commit/3d8df00672b2aa655df982be2768b3e1739043c3) feat(DiffRenderable): synchronize scroll between split view panes in diff.ts - [`2ed5649`](https://github.com/anomalyco/opentui/commit/2ed56490adefc16a2e104b1b737b4e2b43a7d090) fix(DiffRenderable): update container reference for side detection in isInsideSide method - [`fd23733`](https://github.com/anomalyco/opentui/commit/fd23733f6050b1b24fdb9abf916d73c6e6eb7925) fix(DiffRenderable): remove unnecessary blank line in split view scroll test - [`386c622`](https://github.com/anomalyco/opentui/commit/386c62288b6a1c7144faf10c882910d78a685a84) Merge branch 'main' into sync-scroll-diff - [`608e200`](https://github.com/anomalyco/opentui/commit/608e2001930a75971a1e5be10c2b7ca8773f1de6) feat(DiffRenderable): add syncScroll option for split view synchronization - [`11e4c43`](https://github.com/anomalyco/opentui/commit/11e4c4381adfcc3c6fe5f9fbc1a1eb1053cdaffe) Merge branch 'sync-scroll-diff' of https://github.com/viralcodex/opentui into sync-scroll-diff - [`7ad7f34`](https://github.com/anomalyco/opentui/commit/7ad7f3434ce9c8cb844644e7a01c5f4b95f6f72e) Merge branch 'main' into sync-scroll-diff ### 📊 Changes **2 files changed** (+123 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `packages/core/src/renderables/Diff.test.ts` (+82 -1) 📝 `packages/core/src/renderables/Diff.ts` (+41 -0) </details> ### 📄 Description Fixes: #563 Hi @kommander and @simonklee, Added this synced scroll in code diff as mentioned in this issue... I was thinking to keep both separate and synced scroll which can be toggled through hotkeys depending on user preferences. Whatever you guys say 😄 https://github.com/user-attachments/assets/bdc27805-d5e4-4f1c-b1f4-d31b94da4601 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>