focus on dialog open should be constrained to the contents of the dialog

[jonspalmer]

The open function of dialog attempts to find the first tabbable element and give it the focus via:

this.uiDialog.find(':tabbable:first').focus();

however if the dialog has a close icon then it will be first tabbable element and it receives the focus. Instead the find should be done on this.element not thus.uiDialog so that the focus is applied to the contents and not the header.

might be some overlap with #3768

[Jörn Zaefferer]

I consider this critical. The focused close button is annoying me all the time, as it doesn't have a seperate hover-style and looks broken.

[Scott González]

This has already been fixed in r1658 for dialogs that contain tabbable elements in the content area or have buttons.

[Scott González]

Fixed in r2068.

NOTE: This breaks accessibility in Safari, which doesn't allow setting focus on elements that are not natively tabbable. However, our current stance is that people who require accessibility will not be using Safari specifically for this type of reason.