feat(diff): add line wrap toggle#1250
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip CodeRabbit can use TruffleHog to scan for secrets in your code with verification capabilities.Add a TruffleHog config file (e.g. trufflehog-config.yml, trufflehog.yml) to your project to customize detectors and scanning behavior. The tool runs only when a config file is present. |
| <Columns2Icon className="size-3" /> | ||
| </Toggle> | ||
| </ToggleGroup> | ||
| <ToggleGroup |
There was a problem hiding this comment.
nit: not a group with just one item. use Toggle without wrapper
There was a problem hiding this comment.
Thanks! I’ve changed it to use Toggle directly.
What Changed
Adds a new toggle button in the diff view header that toggle line wrapping in the diff. This is next to the split/unified toggle group and follows the same styling.
Why
When the diff panel width is small and the lines are long, the changes are out of view making review hard. This was especially bad when using split view.
UI Changes
Line wrapping off

Line wrapping on

Checklist
Note
Add line wrap toggle to DiffPanel
Adds a toggle button to the
DiffPanelheader usingWrapTextIconthat switches betweenscrollandwrapoverflow modes. The selected mode is passed as anoverflowoption to eachFileDiff, controlling whether long lines scroll horizontally or wrap.Macroscope summarized 87fad1e.