Merge pull request #5514 from rstudio/bugfix/modal-alert-not-reading-…

…messages prevent focus bounce in modal dialogs (was confusing screen readers)
rstudio · Oct 7, 2019 · ca0f78683644fc8c8cf575f9d29e7b8414d40c1e · ca0f786
2 parents 5b19513 + 96fc4ae
commit ca0f78683644fc8c8cf575f9d29e7b8414d40c1e
Unified Split

Showing with 0 additions and 7 deletions.

+0 −7 src/gwt/src/org/rstudio/core/client/widget/ModalDialogBase.java
diff --git a/src/gwt/src/org/rstudio/core/client/widget/ModalDialogBase.java b/src/gwt/src/org/rstudio/core/client/widget/ModalDialogBase.java
@@ -209,13 +209,6 @@ protected void onDialogShown()
    {
       refreshFocusableElements();
       focusInitialControl();
-
-      // try hard to make sure focus ends up in dialog
-      Element focused = DomUtils.getActiveElement();
-      if (focused == null || !DomUtils.contains(getElement(), focused))
-      {
-        focusFirstControl();
-      }
    }
 
    protected void addOkButton(ThemedButton okButton, String elementId)