jquery-bootstrap-scripting icon indicating copy to clipboard operation
jquery-bootstrap-scripting copied to clipboard
verified publisher icon nikku

Add response,status,xhr,form,dialog params to dialog2.ajax-complete. Fix extra spaces in lines.

Open felipe-prenholato opened this issue 13 years ago • 4 comments

Hello, with that patch I made dialog2.ajax-complete event usefull. With that params I can have much more control over ajax process.

felipe-prenholato avatar Aug 06 '12 04:08 felipe-prenholato

Yes. Could you please update your pull request? To remove these additional changes?

nikku avatar Aug 20 '12 20:08 nikku

I'm out of time, but I'll fix this week, probably in weekend.

felipe-prenholato avatar Aug 20 '12 21:08 felipe-prenholato

Lines 45 to 47 in commit.

https://github.com/chronossc/jquery-bootstrap-scripting/commit/cbd1bfb37823d3e523327aaf0b3ee9aa79019707#L0R45

felipe-prenholato avatar Aug 20 '12 21:08 felipe-prenholato

Ahn, @Nikku, maybe is a nice opportunity to think what events should send response, or loaded content, etc, and also change this ones.

Also, a note about sending dialog to event, I did it because I can't close dialog in other way.

If we need to access dialog from any event how we do it without send dialog as argument to binded function?

felipe-prenholato avatar Aug 20 '12 21:08 felipe-prenholato