Core: Deprecate .focus( delay)

[Jörn Zaefferer]

As discussed in Amsterdam, we only use the delayed focus call in dialog. We don't need to modify core methods for just two calls. We should just use _delay() to delay the focus call.

[Jörn Zaefferer]

We have a ton of usage within our unit tests. Updating all those will be a PITA. Can we move the proxied focus method to tests/unit/testsuite.js and leave unit tests as-is? That way we still only have to update dialog.

[Jörn Zaefferer]

My earlier comment was very wrong, I tested the wrong thing. There's two calls in dialog, that's all there is. I've got a patch locally that makes .focus(n) throw an error ("Delayed focus is deprecated, use timeouts and call focus yourself"), along with removing the tests for the proxied method and replacing the usage in dialog. That's probably not good enough for deprecation, and while I remember seeing something about not using $.ui.backCompat anymore, I currently don't know what the right approach for this particular case is.

For rerefence, gist-diff of the patch: ​https://gist.github.com/jzaefferer/0cc41a3c2fd12494d139

[Scott González]

Throwing an error is never valid for deprecation, the functionality needs to still work otherwise it's not deprecated. We're still using $.uiBackCompat for now.

[Jörn Zaefferer]

PR: ​https://github.com/jquery/jquery-ui/pull/1199

[Jörn Zaefferer]

Core: Deprecate .focus( n ), replace in dialog with explicit timeouts

Fixes #9646

Changeset: df6110c0d424ff3306fdd5576011f2dcf4d242d0

[Jörn Zaefferer]

Dialog: Fix shift-tab handling, focus the correct element

Copy-paste error introduced in df6110c0d424ff3306fdd5576011f2dcf4d242d0

Updates the tabbing test to be more specific about which element should have focus, instead of only checking if focus is within the dialog.

Ref #9646 Ref #10103 Closes gh-1264

Changeset: a0b84767a76098cdcc6375dfe28a7fee866bd395