Skip to content

fix(web): file preview wrap buttons no longer turn off word wrap everywhere - #26432

Merged
t3dotgg merged 1 commit into
mainfrom
t3/preview-wrap-stays-local
Oct 28, 2026
Merged

t3dotgg merged 1 commit into
mainfrom
t3/preview-wrap-stays-local

Conversation

@t3dotgg

@t3dotgg t3dotgg commented Oct 28, 2026

Copy link
Copy Markdown
Member

Word wrap was off in my install even though it defaults to on. That made chat tables open cut off to one line. The cause: the wrap button in the file preview and the attachment preview wrote the global "Word wrap" setting. One click there turned off wrapping for every code block, table, diff, and file preview.

Now those two buttons change only the view they sit in, like the diff panel and PR code tab buttons already do. Settings > Appearance > Word wrap is the only place that changes the default.

Before click (wrapped by default) After "Disable word wrap" in this file Setting stays on
File preview wrapped Same file unwrapped Word wrap setting still on

Installs that already got flipped keep their stored value. Turn Word wrap back on once in Settings > Appearance.

Proof: a new browser test (fails on main, where the global setting flips to false), vp run check:changed, and a manual pass in the dev web app (screenshots above).

Reviewed with sol-loop: 1 round with GPT-6.1 Sol. Sol verdict: approved

Created with Claude Opus 5.5 in Claude Code.

🤖 Generated with Claude Code

…ywhere

The wrap button in the file and attachment previews wrote the global
Word wrap setting, so one click there turned wrapping off for chat code
blocks, tables, diffs, and every file preview. Those buttons now keep a
per-view override, like the diff panel and PR code tab already do.
Settings > Appearance > Word wrap is the only place that changes the
default.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M 30-99 changed lines (additions + deletions). labels Oct 28, 2026
@macroscopeapp

macroscopeapp Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at b9b6dbf

Macroscope's review found this PR approvable — Preview wrap buttons now affect only their preview rather than changing word wrap throughout the app. Previews still initially follow the existing setting, keeping this a contained presentation change.

You can add or adjust custom eligibility rules. Learn more.

@t3dotgg
t3dotgg merged commit 48a2923 into main Oct 28, 2026
30 of 31 checks passed
@t3dotgg
t3dotgg deleted the t3/preview-wrap-stays-local branch October 28, 2026 11:22
Pipyakas pushed a commit to Pipyakas/t3code that referenced this pull request Oct 28, 2026
…ywhere (pingdotgg#26432)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
(cherry picked from commit 48a2923)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30-99 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant