confirm on close example not working with jqueryui
confirm on close example not working with jqueryui
Test case without jqueryui: http://live.datatables.net/widoxiyu/1/edit
Test case with jqueryui: http://live.datatables.net/fezipuji/1
I copied the code from https://editor.datatables.net/examples/api/confirmClose. I'm seeing the error in a child row editor, but not sure that is relevant.
Note I couldn't get the styling working maybe because editor.jqueryui.min.css failed to load, but that is irrelevant.
If I edit a child row, change a field, then try to close the edit modal I see the confirmation message. If I cancel the confirmation message, the edit modal disappears even though I cancelled.
This seems to be related to jqueryui.
This question has an accepted answers - jump to answer
Answers
Hi, just bumping this -- maybe it got lost over the weekend.
I've updated the styling: http://live.datatables.net/jafenuni/1/edit . The download builder will help with that sort of thing.
I'm not getting that. Maybe I broke it when fixing the styling? Unfortunately I'm short of time this evening to debug it further at the moment I'm afraid.
Allan
http://live.datatables.net/jafenuni/1 does indeed do the behavior I mentioned.
Follow these steps
Perhaps a clue? If you do this more than once, the confirmation message is redisplayed the number of times you do this. But maybe that's a red herring. Actually that's probably happening because the
'postCreate postEdit close'
handler isn't executing. Which may or may not be a clue.Seems like jqueryui is getting in before Editor has a chance to handle the confirmation.
I was also able to reproduce the problem in a simpler test case without a child row: http://live.datatables.net/qizenidu/1
Please can you provide steps on how to reproduce on that last example,
Colin
I had to clone it in order to see live preview. Not sure why this happens with my bins but they time out.
http://live.datatables.net/tocixequ/1
Thanks for the clarification. It is an error in our jQuery UI integration for Editor. We let jQuery UI handle the close button, but actually we need to override that.
I've put the updated jQuery UI integration code into your example here: http://live.datatables.net/mugoyagi/1/edit . It will also be in the next Editor release (1.9.5).
Regards,
Allan
perfect, thanks