[GH-ISSUE #716] Add `scrollChildIntoView` method to ScrollBoxRenderable #195

New issue

Open

opened 2026-03-02 23:45:11 +03:00 by kerem · 1 comment

kerem commented

2026-03-02 23:45:11 +03:00

Owner

Originally created by @jorgeraad on GitHub (Feb 19, 2026).
Original GitHub issue: https://github.com/anomalyco/opentui/issues/716

Originally assigned to: @simonklee on GitHub.

ScrollBoxRenderable has scrollTo and scrollBy for position-based scrolling, but no way to say "make this child visible." Consumers end up writing the same ~15 lines of boilerplate every time they have a keyboard-navigable list inside a scrollbox: find the child by ID, calculate its position relative to the viewport, and scrollBy the right delta.

scrollBox.scrollChildIntoView(childId: string)

The class already has everything it needs internally (findDescendantById, viewport.height, scrollTop, scrollBy), so this would be a thin convenience method on the same level as scrollTo/scrollBy.

Originally created by @jorgeraad on GitHub (Feb 19, 2026). Original GitHub issue: https://github.com/anomalyco/opentui/issues/716 Originally assigned to: @simonklee on GitHub. ScrollBoxRenderable has `scrollTo` and `scrollBy` for position-based scrolling, but no way to say "make this child visible." Consumers end up writing the same ~15 lines of boilerplate every time they have a keyboard-navigable list inside a scrollbox: find the child by ID, calculate its position relative to the viewport, and `scrollBy` the right delta. ```ts scrollBox.scrollChildIntoView(childId: string) ``` The class already has everything it needs internally (`findDescendantById`, `viewport.height`, `scrollTop`, `scrollBy`), so this would be a thin convenience method on the same level as `scrollTo`/`scrollBy`.

kerem added the

feature

core

labels

2026-03-02 23:45:11 +03:00

kerem commented

2026-03-02 23:45:12 +03:00

Author

Owner

@simonklee commented on GitHub (Feb 21, 2026):

Thanks, i think the poc impl was mostly correct. I adjusted it a little and created a PR.

@simonklee commented on GitHub (Feb 21, 2026): Thanks, i think the poc impl was mostly correct. I adjusted it a little and created a PR.

kerem referenced this issue

2026-03-02 23:46:09 +03:00

[PR #195] [MERGED] document testing utils #364

kerem referenced this issue

2026-03-14 09:20:46 +03:00

[PR #195] [MERGED] document testing utils #1144