Skip to content

Add validation notification combinators #2

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add validation notification combinators #2

wants to merge 5 commits into from

Conversation

@juselius
Copy link

@juselius juselius commented Mar 31, 2016

Add combinators to show formlet validation:

  • WithValidationIcon adds an error icon and an error tooltip to the formlet
  • WithValidationMessage flas an error by color and adds an error message below the formlet
  • ValidateOnChange delays validation until the onchange event of an input is fired

Jonas Juselius added 5 commits March 31, 2016 09:12
Add WithValidationIcon, with or without tooltips.
2320650
Change ConvertBy to MapSeqCachedBy to quench deprecarion warning.
9e942b9
Validation tooltip on top.
fb45618
Add WithValidationMessage combinator.
0844f92 
Have the option to select between error icon + tooltip, or errorFormlet and
errorPanel.
Add ValidateOnChange combinator to delay validation errors.
cab8fac 
The ValidateOnChange combinator delays validation until the OnChange event of
the input element fires. Form validation with reactive validation status
indicators can become annoying when the validation triggers too early. For
example, a user starts typing an email address, and the validator immediately
shows an error until the validator clears.
@juselius juselius changed the title Add WithValidationIcon and WithValidationMessage combinators. Add validation notification combinators Apr 6, 2016
@CLAassistant
Copy link

CLAassistant commented May 25, 2020

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@juselius @CLAassistant