feat(modal): added support for autofocus fields on open

[nl5887]

No description provided.

[marcalj]

Please merge this!!! :)

[klamping]

👍

[wilg]

Does this close #2802?

Looks like it has a conflict, also. @nl5887 Are you around to update this or should I open a new pull?

[cvrebert]

X-Ref: twbs/bootstrap#12525

[wilg]

Is there a way to use the shown.bs.modal event in angular-ui? I couldn't figure it out from the docs.

[nl5887]

Yes, I'm around. Will look at it today or tomorrow.

[nl5887]

Hmm, I think we can use the shown.bs.modal. Then angular-way will be to implement it in the directive.

[nl5887]

If I interpret the jQuery Lite (https://docs.angularjs.org/api/ng/function/angular.element) correctly, it should be able to intercept the "shown.bs.modal" event. I'll implement this and submit to the pull request.

[pkozlowski-opensource]

@nl5887 since we are not basing those directives on Bootstrap's CSS you will not see the shown.bs.modal fired....

[nl5887]

Right... Misread it. Would you like me to fire the events on open and close? This should work with jqLite only.

[pkozlowski-opensource]

@nl5887 honestly I don't know what is the best solution for this issue at the moment. I'm not sure if an event would help here since you would have hard time figuring out when to send such an event, IMO.

[nl5887]

And what if I just fix the merge conflicts as it is was first? That should work and probably sufficient to fix the focus issues.

[wesleycho]

I merged #2945, which should also address the original issue - thanks for the effort though!