Skip to content

Refactor: Replace static hex values with design tokens in test files #14912

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 1 commit into
base: main
Choose a base branch
from

Conversation

theceo1
Copy link

@theceo1 theceo1 commented Apr 26, 2025

Description

Replaced static hex color values in test files with appropriate design tokens from @metamask/design-tokens to ensure consistent color usage and alignment with the design system.

Related issues

Closes #14299 (or the relevant issue number, if applicable)

Manual testing steps

  1. Run yarn jest app/components/UI/Ramp/components/PaymentMethodSelector.test.tsx
  2. Confirm all tests pass for the modified file.

Screenshots/Recordings

N/A

Pre-merge author checklist

  • All hex values in test files replaced with design tokens
  • All tests passing for changed files
  • No eslint warnings for @metamask/design-tokens/color-no-hex rule

Pre-merge reviewer checklist

  • Code follows project conventions
  • All tests pass
  • No regressions introduced

@theceo1 theceo1 requested a review from a team as a code owner April 26, 2025 09:59
Copy link
Contributor

github-actions bot commented Apr 26, 2025

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot metamaskbot added external-contributor INVALID-PR-TEMPLATE PR's body doesn't match template labels Apr 26, 2025
@theceo1
Copy link
Author

theceo1 commented Apr 26, 2025

I have read the CLA Document and I hereby sign the CLA

@metamaskbot metamaskbot removed the INVALID-PR-TEMPLATE PR's body doesn't match template label Apr 26, 2025
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.

Remove static hex values in favor of design tokens
2 participants