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

Remove/update m-form-field modifiers #1973

Merged
merged 1 commit into from
May 7, 2024
Merged

Conversation

anselmbradford
Copy link
Member

cfgov PR cfpb/consumerfinance.gov#8384

Removals

  • Unstyled m-form-field__select, m-form-field__success, m-form-field__warning, m-form-field__error

Changes

  • Update m-form-field--checkbox--success to m-form-field--checkbox-success.
  • Update m-form-field--checkbox--warning to m-form-field--checkbox-warning.
  • Update m-form-field--checkbox--error to m-form-field--checkbox-error.

Testing

  1. PR checks should pass.

Copy link

netlify bot commented May 3, 2024

Thanks for the improvements! Browse a preview of your changes using the link below.

Name Link
🔨 Latest commit 4c196d6
🔍 Latest deploy log https://app.netlify.com/sites/cfpb-design-system/deploys/663556e1c81c01000820daa4
😎 Deploy Preview https://deploy-preview-1973--cfpb-design-system.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@anselmbradford anselmbradford merged commit 764e5d3 into main May 7, 2024
8 checks passed
@anselmbradford anselmbradford deleted the ans_remove_formfield branch May 7, 2024 12:08
@anselmbradford anselmbradford added the lerna-changelog/enhancement lerna label. DO NOT MODIFY label May 7, 2024
chosak added a commit that referenced this pull request May 20, 2024
Fix text input notification colors broken in #1973.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lerna-changelog/enhancement lerna label. DO NOT MODIFY
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant