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

Nested dialogs can have focus problems due to Bootstrap-modal #9524

Open
peterflynn opened this issue Oct 10, 2014 · 0 comments
Open

Nested dialogs can have focus problems due to Bootstrap-modal #9524

peterflynn opened this issue Oct 10, 2014 · 0 comments

Comments

@peterflynn
Copy link
Member

It turns out that Bootstrap-modal doesn't really support nested dialogs. Two enforceFocus() listeners will be registered at once in that case, and they'll fight over the focus. Normally, we avoid this because the root elements of our dialogs aren't focusable (no tabIndex) -- which breaks enforceFocus() entirely. But if you do make the root element focusable (as in PR #8856), then nested dialogs can't receive keyboard focus (as in #9196, which was caused by that PR).

To avoid from hitting this again, we should either:
a) Patch bootstrap-modal to disable or fix enforceFocus(). (Disabling seems fine -- since it normally doesn't work at all, we aren't really relying on it anyway).
b) Try to move Dialogs away from bootstrap-modal entirely. At this point it seems like we've already reimplemented most of what it provides, since it didn't do what we wanted... so it might not be too hard to remove what's left of our dependency on it.

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

2 participants