#9200 closed bug (notabug)

Opened April 02, 2013 11:51AM UTC

Closed April 02, 2013 12:14PM UTC

Modal Dialog incompatible with jquery-validate

Reported by: ohcibi Owned by: ohcibi
Priority: minor Milestone: none
Component: ui.dialog Version: 1.10.2
Keywords: Cc:
Blocked by: Blocking:
Description

Demo: http://jsfiddle.net/YTwFY/2/

I've included jQuery-ui 1.10.2 manually in this fiddle as jsfiddle doesn't support it yet.

Steps to reproduce:

1. Enter some text into the first input field (name="name")

2. Hit the enter key on your keyboard

Expected:

  • Error is shown as pinCode is still empty.
  • Form doesn't get submitted
  • Dialog gets opened

What goes wrong:

  • Error is not shown
  • Form is submitted

What goes right:

  • Dialog gets opened

If the first input field is empty or the focus is on the second field or the submit button is clicked, everything is working correctly.

Workaround:

Delay the opening of the dialog by 1 ms:

invalidHandler: function() {
//...
  setTimeout(function() { 
    $('#dialog-confirm').dialog({
      //...
    });
  }, 1);
//..
}
Attachments (0)
Change History (3)

Changed April 02, 2013 12:05PM UTC by scottgonzalez comment:1

component: ui.core → ui.dialog
owner: → ohcibi
status: new → pending

I get a 404 for that fiddle.

Can you provide a reduced test case that doesn't include the validation plugin? As long as that's included, it's not clear where or what the problem is.

Changed April 02, 2013 12:11PM UTC by ohcibi comment:2

status: pending → new

I forgot to add my username into the link, after I edited. This one will work: http://jsfiddle.net/ohcibi/YTwFY/2/ (can't edit the issue directly).

To provide an example without the validation plugin is a tough one as this is what the problem is about. The validation isn't triggered correctly if the dialog is opened inside the invalidHandler of the validation plugin and the above conditions met. Maybe this is rather a jquery-validate bug?

Changed April 02, 2013 12:14PM UTC by scottgonzalez comment:3

resolution: → notabug
status: new → closed

We're not going to dig through all of the validation code to figure out where the incompatibility is. Perhaps Jörn will know off the top of his head. My guess is that he'll miss this discussion unless you file an issue against the validation plugin since he's on vacation right now.