Skip to content

fix(web): render diff content directly to fix empty diffs panel#231

Merged
BunsDev merged 1 commit into
mainfrom
okcode/empty-diffs-view
Apr 4, 2026
Merged

fix(web): render diff content directly to fix empty diffs panel#231
BunsDev merged 1 commit into
mainfrom
okcode/empty-diffs-view

Conversation

@BunsDev

@BunsDev BunsDev commented Apr 4, 2026

Copy link
Copy Markdown
Member

Summary

  • Replaces the broken DiffFileSection wrapper component with direct FileDiff rendering in plain <div> elements, matching the proven t3code implementation
  • The previous <section> wrapper used overflow-hidden which clipped the FileDiff shadow DOM content, causing the diff panel to appear completely empty
  • Replaces the <Select> dropdown for turn selection with a scrollable chip strip with navigation arrows for better UX

Test plan

  • Open the diffs panel — file diff content should now render correctly instead of showing an empty view
  • Verify the turn chip strip scrolls horizontally and highlights the selected turn
  • Click a file header ([data-title]) to open it in the code viewer
  • Toggle between stacked/split diff modes
  • Toggle word wrap on/off

🤖 Generated with Claude Code

Replace the broken DiffFileSection wrapper (which used overflow-hidden
on a section element, preventing FileDiff from rendering its content)
with direct FileDiff rendering in plain divs matching the proven t3code
implementation. Also replaces the Select dropdown with a scrollable turn
chip strip for better UX.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented Apr 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Apr 4, 2026 6:43pm
v0-compute-the-platform-to-build Ready Ready Preview, Open in v0 Apr 4, 2026 6:43pm

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L labels Apr 4, 2026
@BunsDev BunsDev merged commit 78a6c58 into main Apr 4, 2026
9 of 10 checks passed
@BunsDev BunsDev deleted the okcode/empty-diffs-view branch April 6, 2026 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 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