Skip to content

fix: 🐛 Fix autoscrolling terminal output #6538

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

chezsmithy
Copy link
Contributor

@chezsmithy chezsmithy commented Jul 9, 2025

Description

A previous change to auto-scroll behavior broke the autoscroll when terminal tools became taller due to streaming output.

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screenshots

[ For visual changes, include screenshots. Screen recordings are particularly helpful, and appreciated! ]

Tests

Visual inspection of auto scroll working. I've used a prompts like running vite or jest tests that steam output over time, and then also using terminal output like echo commands in a foreground loop.


Summary by cubic

Fixed autoscrolling in the terminal output so that the view stays at the bottom as new streaming output appears.

  • Bug Fixes
    • Improved scroll detection and handling for terminal tools that grow in height.
    • Added resize and mutation observers to track dynamic content changes.

@chezsmithy chezsmithy requested a review from a team as a code owner July 9, 2025 18:41
@chezsmithy chezsmithy requested review from Patrick-Erichsen and removed request for a team July 9, 2025 18:41
Copy link

netlify bot commented Jul 9, 2025

👷 Deploy request for continuedev pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 247eeb3

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jul 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:L This PR changes 100-499 lines, ignoring generated files.
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

1 participant