Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

Commit

Permalink
fix(dialog): moved bottom focus trap to be a sibling
Browse files Browse the repository at this point in the history
- bottomFocusTrap was appended to the dialog, in FF it seems like it didn't trapped the focus, ,moving it to be a sibling solves it.

fixes #7407

  Closes #7408
  • Loading branch information
EladBezalel authored and ThomasBurleson committed Mar 7, 2016
1 parent 3a322dd commit 1e2d380
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/components/dialog/dialog.js
Original file line number Diff line number Diff line change
Expand Up @@ -938,7 +938,7 @@ function MdDialogProvider($$interimElementProvider) {
// The top focus trap inserted immeidately before the md-dialog element (as a sibling).
// The bottom focus trap is inserted at the very end of the md-dialog element (as a child).
element[0].parentNode.insertBefore(topFocusTrap, element[0]);
element.append(bottomFocusTrap);
element.after(bottomFocusTrap);
}

/**
Expand Down

0 comments on commit 1e2d380

Please sign in to comment.