Skip to content

Adding tag, gray900 #571

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Adding tag, gray900 #571

wants to merge 2 commits into from

Conversation

emilmilanov
Copy link
Contributor

image

Copy link

changeset-bot bot commented May 5, 2025

🦋 Changeset detected

Latest commit: b950c20

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@cypress-design/constants-tag Minor

Not sure what this means? Click here to learn what changesets are.

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

Copy link

vercel bot commented May 5, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
cypress-design ✅ Ready (Inspect) Visit Preview May 5, 2025 2:52pm

Copy link

cypress bot commented May 5, 2025

cypress-design    Run #2846

Run Properties:  status check passed Passed #2846  •  git commit 858b971fff ℹ️: Merge b950c2073574d30e6b027b4c77d7cc881705081f into 694ff2102a8c5c60ddf5f87a9a70...
Project cypress-design
Branch Review tag-gray-900
Run status status check passed Passed #2846
Run duration 03m 34s
Commit git commit 858b971fff ℹ️: Merge b950c2073574d30e6b027b4c77d7cc881705081f into 694ff2102a8c5c60ddf5f87a9a70...
Committer Emil Milanov
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 236
View all changes introduced in this branch ↗︎
UI Coverage  8.39%
  Untested elements 268  
  Tested elements 26  
Accessibility  99.43%
  Failed rules  0 critical   2 serious   0 moderate   1 minor
  Failed elements 23  

@@ -17,6 +17,10 @@ export const CssColor = {
default: 'bg-gray-50 text-gray-700 border-gray-100',
dark: 'bg-gray-700 text-white border-gray-700',
},
gray900: {
Copy link
Contributor

Choose a reason for hiding this comment

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

@emilmilanov I would recommend calling this one gray-dark maybe just to align closer to how we call our Button components.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree.

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