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

Hability to hide message for field #47

Open
francorisso opened this issue Jul 30, 2014 · 3 comments
Open

Hability to hide message for field #47

francorisso opened this issue Jul 30, 2014 · 3 comments

Comments

@francorisso
Copy link

Sometimes you don't want to put a span next to the form field, and just want to highlight the field, I've added on my fork of this library an option to fields called show_message, true by default, that you can set to false if you don't want it.

@wraithgar
Copy link
Contributor

Make a PR, please! If you're not comfortable writing tests I will do that too (but it will mean a little longer before I can get to it). That way you don't have to maintain your own fork to keep current whenever this repo changes.

@francorisso
Copy link
Author

Hey! I've done the pull request, sorry, I'm a newbie here, what do you use
to write tests?
Thanks!

On Wed, Jul 30, 2014 at 3:10 PM, Michael Garvin [email protected]
wrote:

Make a PR, please! If you're not comfortable writing tests I will do that
too (but it will mean a little longer before I can get to it). That way you
don't have to maintain your own fork to keep current whenever this repo
changes.


Reply to this email directly or view it on GitHub
#47 (comment)
.

Franco Risso.

@wraithgar
Copy link
Contributor

Check out https://github.com/HenrikJoreteg/Happy.js/blob/master/test/tests.js if you wanna take a peek

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

No branches or pull requests

2 participants