Skip to content

Fixed terminal scrolling to the beginning when a choice menu or diff panel appears (regression since 0.20.20) - #69

Merged
oleg68 merged 1 commit into
nbplugins:mainfrom
oleg68:bugfix/fix-terminal-scroll-regression
Apr 18, 2026
Merged

oleg68 merged 1 commit into
nbplugins:mainfrom
oleg68:bugfix/fix-terminal-scroll-regression

Conversation

@oleg68

@oleg68 oleg68 commented Apr 18, 2026

Copy link
Copy Markdown
Collaborator

Restored the scrollToShowAllOutput() call at the end of switchSouthCard(). This call was removed in commit 58c7f59 to fix a blank terminal after model selection, but the blank terminal was subsequently fixed properly via forceTerminalRedraw() in ClaudeSessionController (commit 970f359), making the removal unnecessary. The scroll regression was introduced as a side effect.

…panel appears (regression since 0.20.20)

Restored the scrollToShowAllOutput() call at the end of switchSouthCard().
This call was removed in commit 58c7f59 to fix a blank terminal after model
selection, but the blank terminal was subsequently fixed properly via
forceTerminalRedraw() in ClaudeSessionController (commit 970f359), making
the removal unnecessary. The scroll regression was introduced as a side effect.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@oleg68
oleg68 merged commit cc30ee9 into nbplugins:main Apr 18, 2026
3 checks passed
@oleg68
oleg68 deleted the bugfix/fix-terminal-scroll-regression branch April 18, 2026 10:00
oleg68 added a commit to oleg68/NetbeansPluginClaudeCodeGui that referenced this pull request Apr 19, 2026
…panel appears (regression since 0.20.20) (nbplugins#69)

Restored the scrollToShowAllOutput() call at the end of
switchSouthCard(). This call was removed in commit 58c7f59 to fix a
blank terminal after model selection, but the blank terminal was
subsequently fixed properly via forceTerminalRedraw() in
ClaudeSessionController (commit 970f359), making the removal
unnecessary. The scroll regression was introduced as a side effect.

Co-authored-by: Oleg Samarin <osamarin@openintegration.inc>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant