FEE-823 Make BRAT repository URLs copyable on installation docs page#1122
FEE-823 Make BRAT repository URLs copyable on installation docs page#1122jsmorabito wants to merge 3 commits into
Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Wrap both repo URLs in fenced code blocks so Nextra renders a copy button - Enable defaultShowCopyCode site-wide in nextra config - Update step text to "Enter the repository URL below and choose Latest version" Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Deployment failed with the following error: View Documentation: https://vercel.com/docs/accounts/team-members-and-roles |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
jsmorabito
changed the title
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
| <footer className="mt-12 border-t border-neutral-light/10 bg-neutral-dark px-6 py-6"> | ||
| <div className="mx-auto flex max-w-6xl items-center justify-between"> | ||
| <p className="text-secondary"> | ||
| <p className="text-orange-500"> |
There was a problem hiding this comment.
🟡 Hardcoded text-orange-500 bypasses the design system color tokens
The footer copyright text was changed from text-secondary (#5F57C0) to text-orange-500 (#f97316), a raw Tailwind utility color. The shared tailwind config (packages/tailwind-config/tailwind.config.ts) defines design tokens (primary: #FF8C4B, secondary: #5F57C0), and the entire codebase consistently uses these tokens (e.g., text-primary, text-secondary). This is the only usage of text-orange-500 in the website app, violating the AGENTS.md rule to "Maintain visual consistency with the host application's design system." If the intent was to use an orange color, text-primary should be used instead.
| <p className="text-orange-500"> | |
| <p className="text-primary"> |
Was this helpful? React with 👍 or 👎 to provide feedback.
| ``` | ||
| https://github.com/blacksmithgu/datacore | ||
| ``` |
There was a problem hiding this comment.
🚩 Markdown list continuity broken by unindented code blocks
The refactored installation instructions place the code blocks (lines 26-28 and 43-45) at the root indentation level rather than indented under list item 4. In CommonMark, this breaks the ordered list into two separate lists (items 1-4, then a new list starting at 5). Visually the numbers still render as 5 and 6 because CommonMark respects the starting number, so the user-facing impact is minimal. However, the list semantics are broken, and some renderers may add extra spacing between the two lists. Indenting the code blocks by 4 spaces (or 3 to match the list prefix width) would keep them as part of list item 4 and maintain a single continuous list.
Was this helpful? React with 👍 or 👎 to provide feedback.
1 participant
Summary
defaultShowCopyCode: truein the Nextra config (applies site-wide to all docs code blocks)Test plan
/docs/obsidian/welcome/installationand hover over both repository URL code blocks — copy button should appear🤖 Generated with Claude Code