Skip to content

feat: change theme#3067

Open
cpenned wants to merge 2 commits intocanaryfrom
feat/change-theme
Open

feat: change theme#3067
cpenned wants to merge 2 commits intocanaryfrom
feat/change-theme

Conversation

@cpenned
Copy link
Contributor

@cpenned cpenned commented Mar 16, 2026


Summary by cubic

Switch the docs to the willow theme and migrate nav in apps/docs/docs.json from anchors/anchor (incl. navigation.global) to tabs/tab, adding tabs for Documentation, Components, Templates, and Editor.

Written for commit bb21ba5. Summary will update on new commits.

@changeset-bot
Copy link

changeset-bot bot commented Mar 16, 2026

⚠️ No Changeset found

Latest commit: bb21ba5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link
Contributor

vercel bot commented Mar 16, 2026

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

Project Deployment Actions Updated (UTC)
react-email Ready Ready Preview, Comment Mar 17, 2026 3:04am
react-email-demo Ready Ready Preview, Comment Mar 17, 2026 3:04am

Request Review

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 1 file

Confidence score: 3/5

  • There is a concrete regression risk in apps/docs/docs.json: changing navigation.global from anchors to tabs appears to conflict with Mintlify’s documented schema, which could cause header/global links to stop rendering for users.
  • Given the medium severity (6/10) and high confidence (9/10), this is more than a cosmetic concern and introduces noticeable docs navigation impact if merged as-is.
  • Pay close attention to apps/docs/docs.json - verify navigation.global matches Mintlify schema (anchors) so global header links continue to render.
Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="apps/docs/docs.json">

<violation number="1" location="apps/docs/docs.json:15">
P2: `navigation.global` should stay on `anchors`; switching it to `tabs` no longer matches Mintlify's documented schema for global links, so these header links may stop rendering.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="apps/docs/docs.json">

<violation number="1" location="apps/docs/docs.json:140">
P2: The new “Editor” tab is a dead link: `href: "#"` keeps users on the current page instead of taking them to an editor.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

},
{
"tab": "Editor",
"href": "#",
Copy link
Contributor

Choose a reason for hiding this comment

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

P2: The new “Editor” tab is a dead link: href: "#" keeps users on the current page instead of taking them to an editor.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At apps/docs/docs.json, line 140:

<comment>The new “Editor” tab is a dead link: `href: "#"` keeps users on the current page instead of taking them to an editor.</comment>

<file context>
@@ -124,27 +124,23 @@
+      },
+      {
+        "tab": "Editor",
+        "href": "#",
+        "icon": "pencil"
       }
</file context>

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