Skip to content

Conversation

@ComputelessComputer
Copy link
Collaborator

  • Simplified layout and styling for documentation pages
  • Consistent signature image sizing
  • Minor cleanup and chore improvements

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 17, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This PR comprises layout and typography refinements across documentation and blog views. Changes include removal of extraneous CSS utilities, restructuring of grid-based layouts to flex-based layouts, typography scale reductions, sidebar width adjustments, and UI component removals for simplified responsive behavior.

Changes

Cohort / File(s) Summary
Documentation Landing Page
apps/web/content/docs/index.mdx
Removed extraneous exclamation marks from image className values and removed inline width style attributes.
Blog Article View Layout
apps/web/src/routes/_view/blog/$slug.tsx
Removed MobileHeader component, expanded main content padding, adjusted heading typography scales, modified CoverImage container padding, updated TOC link spacing, revamped RightSidebar with sticky offset changes and replaced header styling with a new composite header and navigation link.
Documentation Routes & Components
apps/web/src/routes/_view/docs/-components.tsx, apps/web/src/routes/_view/docs/route.tsx
Restructured layouts from CSS grid to flex-based layouts with fixed sidebar widths (changed from grid-cols-1 lg:grid-cols-12 to flex, LeftSidebar from lg:col-span-3 to w-64 flex-shrink-0). Removed Icon import, MobileCTA component, and reduced typography scales for article headings (h1/h2/h4 reduced by 1-2 sizes).
Product Bot UI
apps/web/src/routes/_view/product/bot.tsx
Updated border radius from rounded-[24px] to rounded-3xl on draggable icon container.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

  • Layout restructuring: Verify grid-to-flex conversions maintain responsive behavior across breakpoints (md, lg, xl).
  • Typography changes: Confirm heading hierarchy reductions across docs and blog views don't negatively impact readability or visual hierarchy.
  • Component removals: Ensure MobileHeader and MobileCTA removal doesn't break mobile-specific functionality or layouts.
  • Sidebar styling: Review fixed width assignments (w-64) and flex-shrink-0 behavior to ensure proper responsiveness and overflow handling.

Possibly related PRs

  • cleanup #1618 — Modifies the same docs index file with overlapping edits to image className and style attributes.
  • ui fixes #1623 — Modifies blog article view with similar grid-to-flex layout restructuring and sidebar width adjustments.
  • fix landing #1661 — Modifies docs layout components and routing files with aligned typography and layout changes.

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 web/docs-page

📜 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 11455f2 and ce9b7db.

⛔ Files ignored due to path filters (1)
  • apps/web/content/docs/README.md is excluded by !**/README*
📒 Files selected for processing (5)
  • apps/web/content/docs/index.mdx (1 hunks)
  • apps/web/src/routes/_view/blog/$slug.tsx (7 hunks)
  • apps/web/src/routes/_view/docs/-components.tsx (4 hunks)
  • apps/web/src/routes/_view/docs/route.tsx (2 hunks)
  • apps/web/src/routes/_view/product/bot.tsx (1 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.

@ComputelessComputer ComputelessComputer merged commit 9c6f2d6 into main Nov 17, 2025
3 of 4 checks passed
@ComputelessComputer ComputelessComputer deleted the web/docs-page branch November 17, 2025 04:01
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