Skip to content
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

fix: accessible colors and set up more neutrals #93

Merged
merged 5 commits into from
Jul 1, 2024

Conversation

josh-chamberlain
Copy link
Contributor

@josh-chamberlain josh-chamberlain commented Jun 28, 2024

Fixes

fixes #92

Description

I've been playing with these colors and being indecisive, so I figured it was time to just submit a small PR that fixes the accessibility and play with things later.

  • defines 50–950 values for brand gold and wine where 500 is the original hex value
    • gold's default is 600, putting it >3 accessibility-wise
    • wine's default is 500, the original hex value
    • 50 and 950 are not pure black or white, just close
  • adds goldneutral and wineneutral to the playground, but nothing is using them yet
    • 50 and 950 are not pure black or white, just close
    • fully swapping neutral with goldneutral or wineneutral in the config often looks very nice, but can be weird
  • moves blue out of the brand section out of pure pedantry (it's not a brand color)
Screen Shot 2024-06-28 at 11 54 06 AM

@josh-chamberlain
Copy link
Contributor Author

@joshuagraber it may be bad form to add all those neutral variants we aren't using yet, LMK and I will stash them somewhere. I do want to incorporate this soon, but I want that exercise to stop slowing down the gold accessibility thing

@joshuagraber
Copy link
Collaborator

joshuagraber commented Jun 28, 2024

@josh-chamberlain I don't think it's bad form. This looks good, and it'll be nice to have the full range of options available.

Do you think we can merge this to beta? Or do you need it more urgently in main?

@joshuagraber joshuagraber changed the title fix accessible colors and set up more neutrals fix: accessible colors and set up more neutrals Jun 28, 2024
@josh-chamberlain josh-chamberlain changed the base branch from main to beta June 28, 2024 16:55
Copy link

gitguardian bot commented Jun 28, 2024

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
9480360 Triggered Generic Password 5db616b src/components/Form/form.spec.ts View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Copy link
Collaborator

@joshuagraber joshuagraber left a comment

Choose a reason for hiding this comment

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

LGTM

@josh-chamberlain josh-chamberlain merged commit cd3fefe into beta Jul 1, 2024
5 checks passed
@josh-chamberlain josh-chamberlain deleted the fix/accessible-colors branch July 1, 2024 19:59
Copy link

github-actions bot commented Jul 1, 2024

🎉 This PR is included in version 3.0.0-beta.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

inaccessible gold colors
2 participants