Skip to content

Conversation

HarleyQeen
Copy link

Invalid CSS: rgb() was used with 4 arguments (rgb(50, 50, 50, 1) and rgb(205, 205, 205, 1)). In comma-separated syntax, rgb() accepts only 3 arguments—alpha must be specified via rgba() or the modern space/slash syntax (rgb(50 50 50 / 1)).

Why:
Browsers ignore invalid rgb(… , … , … , a) declarations, so any utility relying on base.black/base.blackInvert would silently fall back to previous/ inherited color.

How I tested:
Ran next build and verified Tailwind compiled without errors.

Confirmed generated CSS includes expected color values for utilities that reference base.black and base.blackInvert.

Notes:
No visual diffs included since this is a correctness fix; behavior is unchanged except that the colors now apply as intended.

Invalid CSS: rgb() was used with 4 arguments (rgb(50, 50, 50, 1) and rgb(205, 205, 205, 1)). In comma-separated syntax, rgb() accepts only 3 arguments—alpha must be specified via rgba() or the modern space/slash syntax (rgb(50 50 50 / 1)).

Why:
Browsers ignore invalid rgb(… , … , … , a) declarations, so any utility relying on base.black/base.blackInvert would silently fall back to previous/ inherited color.
@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

Copy link

vercel bot commented Aug 18, 2025

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

A member of the Team first needs to authorize it.

@HarleyQeen
Copy link
Author

@cb-heimdall rebased & squashed to a single commit. Could you approve the workflows and Vercel deployment pls?

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