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

fix(dialog): only restore focus with keyboard interaction #9923

Merged
merged 1 commit into from
Nov 17, 2016

Conversation

devversion
Copy link
Member

  • Currently the dialog always restores focus to the origin element upon close.
  • The focus should be only restored if the dialog was opened with keyboard interaction - Same behavior as with the sidenav closing (See here)

Fixes #7963.

@devversion devversion added the needs: review This PR is waiting on review from the team label Oct 26, 2016
@devversion devversion force-pushed the fix/focus-dialog-restore branch from df99f6e to 3cb318f Compare October 26, 2016 15:50
@ThomasBurleson ThomasBurleson added needs: presubmit and removed needs: review This PR is waiting on review from the team labels Oct 27, 2016
@ThomasBurleson ThomasBurleson modified the milestones: 1.1.3, 1.1.2 Oct 27, 2016
* Currently the dialog always restores focus to the origin element upon close.

* The focus should be only restored if the dialog was opened with keyboard interaction - Same behavior as with the sidenav closing.

Fixes angular#7963.
@devversion devversion force-pushed the fix/focus-dialog-restore branch from 3cb318f to b8889b6 Compare October 30, 2016 17:15
@kara kara added pr: merge ready This PR is ready for a caretaker to review and removed needs: presubmit labels Nov 17, 2016
@kara kara merged commit c851204 into angular:master Nov 17, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

a11y: dialog is always restoring focus.
3 participants