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

Form error field class #117

Merged
merged 3 commits into from
Oct 30, 2021
Merged

Conversation

exegeteio
Copy link
Contributor

Description of Feature or Issue

closes #107

Added an initializer to customize the CSS class added by ActiolnView to a field when in an error state. The class being added is 'is-danger'.

Checklist

  • [ x] Added/changed specs for the changes made
  • [ x] Is the linting build successful?
  • [ x] Is the test build successful?

User-Facing Changes

Should see a error highlight when a field is not validated. At the moment, that highlight is red per pre-existing styles.

Copy link
Owner

@ChaelCodes ChaelCodes left a comment

Choose a reason for hiding this comment

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

I love it!! I've been waiting for this change. I'm so glad we're adding it! The default is-danger is pretty subtle in the dark mode though, right?

Thanks for this! By the way, your fork is pretty out of date. We've got some new styling around login, event attendees, and default event owners.

@ChaelCodes ChaelCodes merged commit fb0414f into ChaelCodes:main Oct 30, 2021
@ChaelCodes ChaelCodes added this to the Hacktoberfest 2021 milestone Nov 1, 2021
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