docs: style Ask AI buttons with composio orange#2708
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Claude finished @Sushmithamallesh's task in 2m 7s —— View job PR Review
Looks good. Two small, focused style changes — here's what I checked:
The new classes use Orange (
Simpler change: text color updated from One thing to note: The mobile icon button ( The Approved. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| type="button" | ||
| onClick={toggleDecimalWidget} | ||
| className="inline-flex items-center gap-2 rounded-lg border bg-fd-secondary/50 p-1.5 ps-2.5 text-sm text-fd-muted-foreground transition-colors hover:bg-fd-accent hover:text-fd-accent-foreground shrink-0" | ||
| className="inline-flex items-center gap-2 rounded-lg border border-[var(--composio-orange)]/20 bg-[var(--composio-orange)]/5 p-1.5 ps-2.5 text-sm text-[var(--composio-orange)] transition-colors hover:bg-[var(--composio-orange)]/10 shrink-0" |
There was a problem hiding this comment.
Opacity modifiers on CSS variable arbitrary values ignored
High Severity
In Tailwind CSS v4, the opacity slash modifier (/20, /5, /10, /80) does not work with the [var(--...)] bracket syntax. Classes like bg-[var(--composio-orange)]/5 will either be ignored entirely or apply full-opacity orange — making the button an opaque orange rectangle instead of a subtle 5% tint. The correct v4 syntax for CSS variables with opacity is bg-(--composio-orange)/5 using parentheses without var().


Summary
Test plan
🤖 Generated with Claude Code