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

Make togglePopover return a boolean #40409

Merged
merged 1 commit into from
Jun 20, 2023

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jun 6, 2023

This patch makes togglePopover return true if it opens the popover or
false if it closes it.

HTML spec issue: whatwg/html#9043

Change-Id: I64a1f543cb380fca01866d7b08edc8bb0c3136e9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4595402
Commit-Queue: Joey Arhar <[email protected]>
Reviewed-by: Mason Freed <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1159748}

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 patch makes togglePopover return true if it opens the popover or
false if it closes it.

HTML spec issue: whatwg/html#9043

Change-Id: I64a1f543cb380fca01866d7b08edc8bb0c3136e9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4595402
Commit-Queue: Joey Arhar <[email protected]>
Reviewed-by: Mason Freed <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1159748}
@chromium-wpt-export-bot chromium-wpt-export-bot merged commit e3da78c into master Jun 20, 2023
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-4595402 branch June 20, 2023 00:22
annevk pushed a commit to whatwg/html that referenced this pull request Jun 20, 2023
rubberyuzu pushed a commit to rubberyuzu/html that referenced this pull request Jul 20, 2023
rubberyuzu pushed a commit to rubberyuzu/html that referenced this pull request Jul 21, 2023
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.

4 participants