Skip to content

Conversation

@AllanOXDi
Copy link
Member

@AllanOXDi AllanOXDi commented Nov 28, 2025

Summary

This PR adds dialog-polyfill to provide consistent behavior across all browsers.

Closes #13716

References

#13716

Reviewer guidance

Verify if :

  • Polyfill is registered only when native APIs are missing.
  • Lightbox opens and closes using consistent methods (showModal, close).
  • Backdrop and background-inert behaviour are correctly handled.
  • The escape key and backdrop click dismiss the Lightbox.
  • Focus is trapped inside the dialog while open.
  • Focus returns to an appropriate element after closing
  • Logic doesn’t break native in supported browsers.
  • Confirm no regressions in modern browsers
  • Polyfill path is correctly exercised for unsupported browsers.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 28, 2025

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[HTML5 Article Renderer - Lightbox] Implement polyfill for <dialog> to ensure browser compatibility

1 participant