Skip to content

Conversation

@gsaslis
Copy link

@gsaslis gsaslis commented Dec 7, 2012

In some cases, it is useful to be able to validate hidden fields in a form.
This is particularly true in the case of jquery-UI tabs, if you have a single form split over multiple tabs.
By adding an option about whether we want to validate hidden fields in a form, we can override the default behaviour when we need to, while at the same time maintaining the original behaviour by default.

form.
This is particularly true in the case of jquery-UI tabs, if you have a
single form split over multiple tabs.
By adding an option about whether we want to validate hidden fields in a
form, we can override the default behaviour when we need to, while at
the same time maintaining the original behaviour by default.
@posabsolute
Copy link
Owner

Hi,

First thank you for contributing, we generally ask to update the doc with new feature, also providing a test case is appreciate.

thank you

posabsolute added a commit that referenced this pull request Dec 12, 2012
@posabsolute posabsolute merged commit b6a803f into posabsolute:master Dec 12, 2012
@cecco974
Copy link

Great job ... was just what I was looking for...
this great plugin has been improved even more but i've got a problem with the position of the formError div.
When the field is in a hidden tab the popup alert seems to ignore the real coordinates of the fields and it is displayed with wrong coordinates.
The _calculatePosition function uses the field parameter to obtain the position of the field but the popup is displayed
with fieldTop and fieldLeft as 0 (so it is aligned with the container tab at top+5, left-60.

@justinconn
Copy link

@cecco974 Having the same issue here.... Multi-tabbed form using validateNonVisibleFields. Validation seems to work, but when the offending tab is made visible, the validation prompt is aligned at the very top of the form. When you click anywhere inside the tab, the validation prompt scrolls to the proper position.

Did you find a way to get around this?

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.

4 participants