Skip to content

Align XY docs with the Reflex marketing design - #530

Merged
carlosabadia merged 3 commits into
mainfrom
carlos/xy-docs-marketing-design
Oct 15, 2026
Merged

carlosabadia merged 3 commits into
mainfrom
carlos/xy-docs-marketing-design

Conversation

@carlosabadia

@carlosabadia carlosabadia commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

View guided diff Turn on auto-fix

Summary by CodeRabbit

  • New Features

    • Refreshed the documentation site with coordinated light and dark themes, updated typography, and redesigned navigation, announcement banner, and footers.
    • Updated example tabs and cards with a compact pill-style design, and refined page content and navigation.
    • Added consistent styling for search, feedback controls, hints, and other overlays.
    • Improved mobile layouts, including a sidebar drawer, and adjusted page navigation for narrower screens.
  • Bug Fixes

    • Footer controls now wrap when space is limited.
    • Reduced-motion preferences are respected across documentation animations and transitions.

@coderabbitai

coderabbitai Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 957cd0bd-1d6c-466a-9f3b-52cbccf1258b
📥 Commits

Reviewing files that changed from the base of the PR and between 8df769b and 94995bf.

⛔ Files ignored due to path filters: 1
  • docs/app/reflex.lock/bun.lock is excluded by !**/*.lock
📒 Files selected for processing: 6
  • docs/app/assets/docs-header-theme.css
  • docs/app/assets/docs-overlays-theme.css
  • docs/app/assets/docs-theme.css
  • docs/app/reflex.lock/package.json
  • docs/app/xy_docs/examples.py
  • spec/process/production-readiness.md
🚧 Files skipped from review as they are similar to previous changes: 4
  • docs/app/assets/docs-header-theme.css
  • docs/app/assets/docs-theme.css
  • docs/app/xy_docs/examples.py
  • docs/app/assets/docs-overlays-theme.css

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 1 remain after this review.

📝 Walkthrough

This PR adds three documentation stylesheets and loads them in the XY docs app. It updates navigation, layout, search, feedback, demos, and content components to use shared docs-* hooks. It adds variable font packages, updates layout tests, and revises theme and lockfile guidance.

Changes: XY documentation theme

Layer / File(s) Summary
Theme foundation and dependencies
docs/app/assets/docs-theme.css, docs/app/rxconfig.py, docs/app/reflex.lock/package.json, docs/app/AGENTS.md, docs/app/README.md, spec/process/production-readiness.md
The base stylesheet defines fonts, theme tokens, and documentation layout styles. Frontend dependencies include the variable fonts. Documentation describes the theme ownership boundary, styling requirements, and package update guidance.
Navigation, banner, and overlay styles
docs/app/assets/docs-header-theme.css, docs/app/assets/docs-overlays-theme.css, docs/app/xy_docs/navbar.py, docs/app/xy_docs/xy_docs.py, docs/app/xy_docs/footer.py
The styles cover the docs header, mobile navigation, announcement banner, search, page actions, feedback, hints, and footers. The app loads the stylesheets, applies docs-specific classes to navigation and layout elements, sets light and dark theme colors, and allows footer controls to wrap.
Documentation content and demos
docs/app/assets/docs-theme.css, docs/app/xy_docs/api_reference.py, docs/app/xy_docs/breadcrumb.py, docs/app/xy_docs/examples.py, docs/app/xy_docs/markdown.py, docs/app/tests/test_docs_site.py, docs/app/tests/test_docs_layout.py
Documentation headings, breadcrumbs, API summaries, and demo tabs use the updated styling. Tests check the demo card class and verify article structure and pager links for first, middle, and last routes.

Priority: ➖ Normal

Estimated code review effort: 3 (Moderate) | ~25 minutes

Change: Feature

Suggested reviewers: alek99

Merge Risk: ⚪ Minimal · up to 94995

The reviewed theme and dependency changes show no concrete failure. No actionable merge-blocking risk remains beyond normal checks.

Pre-merge checks: Passed 5
Passed Passed checks: 5 passed
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly summarizes the main change: updating the XY documentation to match the Reflex marketing design.
Docstring Coverage Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 17 functions across 10 files. (5 skipped: …
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@greptile-apps

greptile-apps Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge; no new actionable issues remain.

Summary

Updates the XY docs with marketing-style fonts, light/dark colors, navigation, demo tabs, and overlays.

  • XY docs now match the Reflex marketing site's look.
  • Back and Next links stay in a two-column row.

Reviews (3) · Last reviewed commit: "Fix docs disclosure fade and refresh Pos..." · Reviewed by Greptile

Comment thread docs/app/xy_docs/xy_docs.py

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 19 files

Tip: Get a Slack DM when cubic finishes reviewing your PRs. Connect Slack

View guided diff | Turn on auto-fix | Re-trigger cubic

Comment thread docs/app/assets/docs-theme.css
Comment thread docs/app/assets/docs-overlays-theme.css
Comment thread docs/app/assets/docs-header-theme.css
Comment thread docs/app/xy_docs/examples.py Outdated
Comment thread docs/app/reflex.lock/package.json
Comment thread docs/app/xy_docs/xy_docs.py
@carlosabadia
carlosabadia merged commit 973cca6 into main Oct 15, 2026
12 checks passed
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.

1 participant