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

Bug fixes and one additional feature. #39

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

bwilson
Copy link

@bwilson bwilson commented Mar 25, 2014

I fixed several issues:

  1. The letters constraint would not accept upper-case letters.
  2. The generated JavaScript included some extraneous commas that broke IE.
  3. This plugin was incompatible with the Grails Melody plugin.
  4. The remote validation call has not working properly for me. Making it synchronous fixed it.

I added one feature:

  1. You can now specify the element that a qtip error message will be attached to, just by giving the target element an id which is the same as the input element with the word "Target" appended. This is useful in cases where the qtip would otherwise obscure an important part of the form.

I have been running this version for several weeks in production. It works. Please feel free to contact me if you have any questions. Thanks for reading!

bwilson and others added 9 commits February 12, 2014 12:47
…tibility. Made remote validation call synchronous.
…e to. The alternate element must have an id that is [form element id] + "Target".
…e in validation messages.

It was previously assuming that the field id was the same as the
property name, but ids should be unique and it is possible to have two
forms with the same property name on a page, so this assumption is not
valid.
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.

1 participant