Skip to content

Conversation

@juliusknorr
Copy link
Contributor

Otherwise the trap is kept active while the component is being destroyed e.g. on unshare of a link

Signed-off-by: Julius Härtl <jus@bitgrid.net>
@raimund-schluessler
Copy link
Contributor

Let's see if this also solves #3021.

@juliusknorr juliusknorr added the bug Something isn't working label Aug 12, 2022
@juliusknorr juliusknorr added this to the 6.0.0 milestone Aug 12, 2022
@raimund-schluessler
Copy link
Contributor

Let's see if this also solves #3021.

Unfortunately not, see the example I added to #3021 where the input is still not focused if the focus-trap is dismissed afterDestroy (whereas it gets focused with focus-trap disabled completely in #3021).

@raimund-schluessler raimund-schluessler merged commit a0a877a into master Aug 12, 2022
@raimund-schluessler raimund-schluessler deleted the bugfix/noid/popover-focus-trap branch August 12, 2022 11:59
@skjnldsv skjnldsv mentioned this pull request Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants