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

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

Open chronossc opened this issue 12 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.

Aug 06 '12 04:08 chronossc

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

Aug 20 '12 20:08 nikku

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

Aug 20 '12 21:08 chronossc

Lines 45 to 47 in commit.

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

Aug 20 '12 21:08 chronossc

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?

Aug 20 '12 21:08 chronossc