Skip to content
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

Added the basic <popup> element, with runtime enabled flag #27291

Merged
merged 1 commit into from Jan 23, 2021

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jan 22, 2021

This CL implements the basics of the new <popup> element,
including the HTMLPopupElement prototype, the UA stylesheet
CSS including open/hidden and position:fixed, promotion of
content to the top-layer, and a Runtime Enabled flag to
turn it on. Some basic WPT tests are included.

See the explainer here:
https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Popup/explainer.md

Bug: 1168738
Change-Id: If9f8279c01a7055da49c51670804400d621deb95
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2644061
Reviewed-by: Ionel Popescu <iopopesc@microsoft.com>
Commit-Queue: Mason Freed <masonfreed@chromium.org>
Auto-Submit: Mason Freed <masonfreed@chromium.org>
Cr-Commit-Position: refs/heads/master@{#846409}

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

This CL implements the basics of the new <popup> element,
including the HTMLPopupElement prototype, the UA stylesheet
CSS including open/hidden and position:fixed, promotion of
content to the top-layer, and a Runtime Enabled flag to
turn it on. Some basic WPT tests are included.

See the explainer here:
  https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Popup/explainer.md

Bug: 1168738
Change-Id: If9f8279c01a7055da49c51670804400d621deb95
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2644061
Reviewed-by: Ionel Popescu <iopopesc@microsoft.com>
Commit-Queue: Mason Freed <masonfreed@chromium.org>
Auto-Submit: Mason Freed <masonfreed@chromium.org>
Cr-Commit-Position: refs/heads/master@{#846409}
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.

None yet

3 participants