Skip to content

[NOT TO BE MERGED] - GAUD-9398 - Add demo page demonstrating the issue#6563

Draft
svanherk wants to merge 1 commit intomainfrom
GAUD-9398-dialog-does-not-focus-trap-if-opened-by-inner-iframe
Draft

[NOT TO BE MERGED] - GAUD-9398 - Add demo page demonstrating the issue#6563
svanherk wants to merge 1 commit intomainfrom
GAUD-9398-dialog-does-not-focus-trap-if-opened-by-inner-iframe

Conversation

@svanherk
Copy link
Contributor

@svanherk svanherk commented Feb 5, 2026

I doubt we'd want to merge this demo as is, or at all. But this demonstrates the issue the quizzing team is seeing. If an inner iframe calls the parent frame to open its dialog (normal, fullscreen, or confirm), this messes with the focus trap logic and you can start tabbing into the inner iframe.

This seemingly doesn't happen if the opener is in the same iframe or if there is a focusable element before the iframe.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

Thanks for the PR! 🎉

We've deployed an automatic preview for this PR - you can see your changes here:

URL https://live.d2l.dev/prs/BrightspaceUI/core/pr-6563/

Note

The build needs to finish before your changes are deployed.
Changes to the PR will automatically update the instance.

@svanherk
Copy link
Contributor Author

svanherk commented Feb 5, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant