Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Dialog: Possible to tab outside modal dialog. Partial fix for #7862 - dialog: modal accessibility #684

Closed
wants to merge 1 commit into from

2 participants

@larowlan

Whilst working on #7862, modal accessibility (as part of push to get modal forms into Drupal core) noticed you could tab outside of modal dialogs.

This commit fixes that.

@scottgonzalez

Thanks, landed in 5a95285.

@larowlan
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jul 1, 2012
  1. @larowlan

    Dialog: Possible to tab outside modal dialog. Partial fix for #7862 -…

    larowlan authored
    … dialog: modal accessibility
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  ui/jquery.ui.dialog.js
View
2  ui/jquery.ui.dialog.js
@@ -315,7 +315,7 @@ $.widget("ui.dialog", {
return;
}
- var tabbables = $( ":tabbable", this ),
+ var tabbables = $( ":tabbable", this.uiDialog ),
first = tabbables.filter( ":first" ),
last = tabbables.filter( ":last" );
Something went wrong with that request. Please try again.