Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Sep 20, 2025

This reverts a change from #7985 to the default wrapping setting for code blocks. IMO this looks better, at least for trying to see what commands it wants to run.

Before:
Screenshot 2025-09-20 at 11 37 01 AM

After:
Screenshot 2025-09-20 at 11 36 26 AM


Important

Reverts default initialWordWrap to true in CodeBlock component, making code blocks wrap text by default.

  • Behavior:
    • Reverts default initialWordWrap to true in CodeBlock component in CodeBlock.tsx.
    • Code blocks now wrap text by default, improving readability.
  • Misc:

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

@mrubens mrubens requested review from cte and jr as code owners September 20, 2025 15:38
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. UI/UX UI/UX related or focused labels Sep 20, 2025
@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Review] in Roo Code Roadmap Sep 20, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 20, 2025
Copy link
Contributor

@roomote roomote bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a good UX improvement that enhances the default readability of code blocks. The change from initialWordWrap = false to initialWordWrap = true makes code blocks wrap text by default, which is particularly beneficial for viewing long command lines without horizontal scrolling.

The change is:

  • ✅ Non-breaking (users can still toggle word wrap via the UI button)
  • ✅ Minimal risk (simple boolean default value change)
  • ✅ Better default behavior (as demonstrated in the screenshots)

LGTM! 🚀

@mrubens mrubens merged commit 1533efc into main Sep 20, 2025
23 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Sep 20, 2025
@mrubens mrubens deleted the wrap_code_blocks_by_default branch September 20, 2025 16:08
@github-project-automation github-project-automation bot moved this from PR [Needs Review] to Done in Roo Code Roadmap Sep 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer PR - Needs Review size:XS This PR changes 0-9 lines, ignoring generated files. UI/UX UI/UX related or focused

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants