Skip to content

Conversation

@ComputelessComputer
Copy link
Collaborator

No description provided.

@ComputelessComputer ComputelessComputer merged commit 19be004 into main Nov 4, 2025
2 checks passed
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 4, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Layout refactoring across blog and route files: container max-widths changed from responsive 6xl to fixed 1400px; plain img tag replaced with Image component in blog article view; TableOfContents and RightSidebar converted to fixed widths; header/footer/mobile navigation max-widths updated consistently.

Changes

Cohort / File(s) Summary
Blog view layout updates
apps/web/src/routes/_view/blog/$slug.tsx, apps/web/src/routes/_view/blog/index.tsx
Replaced img with Image component; adjusted max-width from 6xl to 1400px; converted grid layout to flex; set TableOfContents and RightSidebar to fixed widths (w-64 and w-80); updated Back to blog link styling with font-serif and Icon component; border color updated from neutral-100 to neutral-200.
Root route layout
apps/web/src/routes/_view/route.tsx
Changed three container max-width utilities from max-w-6xl to max-w-\[1400px\] across header, mobile navigation, and footer.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

  • Primary concern: Image component usage in blog article view—verify correct rendering and responsive behavior with new Image wrapper
  • Layout consistency: Confirm 1400px fixed width change aligns across all affected areas (header, footer, blog views)

Possibly related PRs

  • blog beautify #1613: Overlapping modifications to blog view files with similar layout and cover-image/TOC/sidebar changes
  • cleanup #1618: Modifies blog index and article files, updating layout structure and image usage
  • landing wip #1597: Modifies route.tsx container utilities and overall header/footer layout structure

Suggested reviewers

  • yujonglee
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch jj-branch-7

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c6bb60f and 1a7492a.

📒 Files selected for processing (3)
  • apps/web/src/routes/_view/blog/$slug.tsx (6 hunks)
  • apps/web/src/routes/_view/blog/index.tsx (1 hunks)
  • apps/web/src/routes/_view/route.tsx (3 hunks)

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants