Skip to content

Conversation

@YangScott
Copy link

  • Fixed a typo in the PolicyBanner button className where transition-all and active:bg-[#06318E] were concatenated into an invalid Tailwind class.
  • Removed duplicate text-md utility.

Previously the button used:

  • transition-allactive:bg-[#06318E] — Tailwind couldn't parse this class so the transition and active background styles were never applied.
  • text-md was present twice in the class string.

Now the button uses:

  • text-md transition-all active:bg-[#06318E] ...

This restores the expected transition/active styles and cleans up the class list without any functional changes to the layout

@cb-heimdall
Copy link
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@vercel
Copy link

vercel bot commented Dec 4, 2025

@YangScott is attempting to deploy a commit to the Coinbase Team on Vercel.

A member of the Team first needs to authorize it.

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.

2 participants