Skip to content
This repository has been archived by the owner on Jul 30, 2018. It is now read-only.

Focus trap / inert mixin #730

Open
smhigley opened this issue Oct 26, 2017 · 0 comments
Open

Focus trap / inert mixin #730

smhigley opened this issue Oct 26, 2017 · 0 comments
Assignees

Comments

@smhigley
Copy link
Contributor

Enhancement

From the widgets focus management issue (dojo/widgets#309 (comment)):

Either trap focus within a node, or remove it from the focus order/AOM

Both these functions rely on inert, which I think we should use and polyfill. We can also either include blockingElements and its polyfill, or create our own implementation. I'm a little dubious about blockingElements actually becoming spec, at least in the foreseeable future. On the other hand, using the polyfill would mean we don't need to traverse the DOM ourselves to apply inert (this is essentially what the polyfill does).

I believe this would also make sense as a mixin that adds inert and blocking to the properties interface.

I'm not sure what this should be called, since its use isn't limited to focus traps. Perhaps FocusOrderMixin?

@dylans dylans added this to the 2017.11 milestone Oct 27, 2017
@kitsonk kitsonk added the beta5 label Nov 6, 2017
@kitsonk kitsonk modified the milestones: 2017.11, beta.5 Nov 6, 2017
@smhigley smhigley self-assigned this Nov 29, 2017
@agubler agubler removed the beta5 label Jan 4, 2018
@agubler agubler removed this from the beta.5 milestone Jan 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants