-
Notifications
You must be signed in to change notification settings - Fork 838
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EuiFocusTrap causes relayout cycles #4382
Comments
Marking as needing triage because the extra DOM elements are expected when enabling the focus trap. This may end up being a change in EUI (e.g. exposing a prop) or providing implementation guidance. |
👋 Hey there. This issue hasn't had any activity for 180 days. We'll automatically close it if that trend continues for another week. If you feel this issue is still valid and needs attention please let us know with a comment. |
❌ We're automatically closing this issue due to lack of activity. Please comment if you feel this was done in error. |
As discovered in elastic/kibana#85430,
if EuiFocusTrap is used, additional HTML elements are introduced and changed in some cases, triggering a layout cycle which can cause problems in other components:
cc @chandlerprall
The text was updated successfully, but these errors were encountered: