From 2916159a088f20eb33602d69c97b79792dd5c037 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Scott=20Gonz=C3=A1lez?= Date: Fri, 13 Feb 2009 03:06:43 +0000 Subject: [PATCH] Dialog: Don't set focus to elements in the titlebar on open. Fixes #4126 - focus on dialog open should be constrained to the contents of the dialog. NOTE: This breaks accessibility in Safari, which doesn't allow setting focus to 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. --- ui/ui.dialog.js | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/ui/ui.dialog.js b/ui/ui.dialog.js index f6cd5ac20..4cec0f7f3 100644 --- a/ui/ui.dialog.js +++ b/ui/ui.dialog.js @@ -230,14 +230,12 @@ $.widget("ui.dialog", { } })); - // set focus to the first tabbable element in: - // - content area - // - button pane - // - title bar + // set focus to the first tabbable element in the content area or the first button + // if there are no tabbable elements, set focus on the dialog itself $([]) .add(uiDialog.find('.ui-dialog-content :tabbable:first')) .add(uiDialog.find('.ui-dialog-buttonpane :tabbable:first')) - .add(uiDialog.find('.ui-dialog-titlebar :tabbable:first')) + .add(uiDialog) .filter(':first') .focus(); -- 2.39.5