Skip to content

Update marketing navbar#6805

Open
carlosabadia wants to merge 1 commit into
mainfrom
carlos/update-navbar
Open

Update marketing navbar#6805
carlosabadia wants to merge 1 commit into
mainfrom
carlos/update-navbar

Conversation

@carlosabadia

Copy link
Copy Markdown
Contributor

No description provided.

@carlosabadia
carlosabadia requested review from a team and Alek99 as code owners July 22, 2026 12:07
@carlosabadia carlosabadia added the skip-changelog For doc/internal changes label Jul 22, 2026
@greptile-apps

greptile-apps Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR refreshes the marketing navbar and adds bannerless navigation support. The main changes are:

  • Redesigns the Products, Resources, and Solutions menus.
  • Adds XY links and active-route handling to the docs navbar.
  • Hides the hosting banner on the 404 page.
  • Adds shared gutter, caption, and dashed-rule styles.
  • Updates the mobile drawer for bannerless pages.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
docs/app/reflex_docs/views/docs_navbar.py Adds XY navigation and active-route matching.
packages/reflex-site-shared/src/reflex_site_shared/pages/page404.py Uses the new bannerless navbar option on the 404 page.
packages/reflex-site-shared/src/reflex_site_shared/styles/assets/tailwind-theme.css Adds shared spacing, caption, and dashed-rule styles.
packages/reflex-site-shared/src/reflex_site_shared/templates/webpage.py Adds a backward-compatible banner visibility option to the page template.
packages/reflex-site-shared/src/reflex_site_shared/views/marketing_navbar.py Redesigns the desktop menus and adds memoized bannerless header rendering.
packages/reflex-site-shared/src/reflex_site_shared/views/sidebar/init.py Adjusts mobile drawer positioning for pages without the hosting banner.

Reviews (1): Last reviewed commit: "Update marketing navbar" | Re-trigger Greptile

@codspeed-hq

codspeed-hq Bot commented Jul 22, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 26 untouched benchmarks
⏩ 8 skipped benchmarks1


Comparing carlos/update-navbar (6d877b6) with main (9854a3a)

Open in CodSpeed

Footnotes

  1. 8 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6d877b6350

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread docs/app/reflex_docs/views/docs_navbar.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog For doc/internal changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant