Skip to content

Improve raw patch rendering and YouTube embed layout#130

Merged
BunsDev merged 2 commits intomainfrom
okcode/fix-youtube-player-layout
Apr 1, 2026
Merged

Improve raw patch rendering and YouTube embed layout#130
BunsDev merged 2 commits intomainfrom
okcode/fix-youtube-player-layout

Conversation

@BunsDev
Copy link
Copy Markdown
Member

@BunsDev BunsDev commented Apr 1, 2026

Summary

  • Extracted a reusable RawPatchViewer for fallback/raw diff content and reused it in both PR review and diff panel views.
  • Added file-aware raw patch navigation, collapsible sections, line highlighting, and copy-to-clipboard support for raw patch text.
  • Fixed the YouTube player embed layout by increasing the iframe height and allowing same-origin so the player renders correctly.
  • Bumped the persisted YouTube player storage version and changed the default selected slot to avoid stale state.

Testing

  • Not run (bun fmt, bun lint, bun typecheck not run in this branch review context).
  • Manual check recommended: open a PR or diff that falls back to raw patch rendering and verify file navigation/collapse behavior.
  • Manual check recommended: open the YouTube player drawer and confirm the embed displays at the expected height without layout issues.

BunsDev added 2 commits March 31, 2026 20:36
- Increase iframe height for a usable player
- Allow same-origin access so the embed renders correctly
- Reset persisted player state for the new layout
- Share raw patch rendering between diff and PR review views
- Add file navigation, collapse controls, and copy support for fallback patches
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 1, 2026

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

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Apr 1, 2026 2:05am

@github-actions github-actions Bot added size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Apr 1, 2026
@BunsDev BunsDev merged commit 4d77af4 into main Apr 1, 2026
8 of 9 checks passed
@BunsDev BunsDev deleted the okcode/fix-youtube-player-layout branch April 1, 2026 08:57
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