Skip to content

Conversation

@roomote
Copy link
Contributor

@roomote roomote bot commented Jan 11, 2026

This PR fixes the nested scrolling issue in expanded thinking sections by explicitly setting maxHeight: "none" and overflow: "visible" on the content container.

Changes

  • Modified ReasoningBlock.tsx to ensure expanded thinking content grows naturally with the page scroll instead of creating a nested scrollable area

Testing

  • Linting: ✓ Passed
  • Type checking: ✓ Passed

View task on Roo Code Cloud


Important

Fixes nested scrolling in ReasoningBlock.tsx by adjusting CSS properties for expanded content.

  • Behavior:
    • Fixes nested scrolling issue in ReasoningBlock.tsx by setting maxHeight: "none" and overflow: "visible" on the content container.
  • Testing:
    • Linting and type checking passed successfully.

This description was created by Ellipsis for 5b8ecae. You can customize this summary. It will automatically update as commits are pushed.

@roomote
Copy link
Contributor Author

roomote bot commented Jan 11, 2026

Rooviewer Clock   See task on Roo Cloud

Review complete. No issues found.

The change correctly adds inline styles to prevent nested scrolling in the expanded thinking sections. The implementation is minimal, targeted, and consistent with how similar overflow issues are handled elsewhere in the codebase.

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

@mrubens mrubens closed this Jan 11, 2026
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jan 11, 2026
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jan 11, 2026
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jan 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants