Skip to content

Set invoker for popover command attributes #11248

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

josepharhar
Copy link
Contributor

@josepharhar josepharhar commented Apr 22, 2025

This PR fixes parameters to the show popover algorithm and includes the invoker parameter in the command attribute steps.

(See WHATWG Working Mode: Changes for more details.)


/form-elements.html ( diff )

This PR fixes parameters to the show popover algorithm and includes the
invoker parameter.
@josepharhar
Copy link
Contributor Author

@keithamus

Copy link
Contributor

@keithamus keithamus left a comment

Choose a reason for hiding this comment

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

This LGTM 👍

chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Apr 23, 2025
We set the popover invoker when using command attributes to invoke a
popover, but there are no tests for this and the spec is missing it.

Spec PR: whatwg/html#11248

Change-Id: I36b60cc1925376c6ecf5c7f9e72d7b955cae71cd
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Apr 23, 2025
We set the popover invoker when using command attributes to invoke a
popover, but there are no tests for this and the spec is missing it.

Spec PR: whatwg/html#11248

Change-Id: I36b60cc1925376c6ecf5c7f9e72d7b955cae71cd
aarongable pushed a commit to chromium/chromium that referenced this pull request Apr 23, 2025
We set the popover invoker when using command attributes to invoke a
popover, but there are no tests for this and the spec is missing it.

Spec PR: whatwg/html#11248

Change-Id: I36b60cc1925376c6ecf5c7f9e72d7b955cae71cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6481415
Reviewed-by: Mason Freed <masonf@chromium.org>
Commit-Queue: Joey Arhar <jarhar@chromium.org>
Reviewed-by: Keith Cirkel <chromium@keithcirkel.co.uk>
Cr-Commit-Position: refs/heads/main@{#1450757}
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Apr 23, 2025
We set the popover invoker when using command attributes to invoke a
popover, but there are no tests for this and the spec is missing it.

Spec PR: whatwg/html#11248

Change-Id: I36b60cc1925376c6ecf5c7f9e72d7b955cae71cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6481415
Reviewed-by: Mason Freed <masonf@chromium.org>
Commit-Queue: Joey Arhar <jarhar@chromium.org>
Reviewed-by: Keith Cirkel <chromium@keithcirkel.co.uk>
Cr-Commit-Position: refs/heads/main@{#1450757}
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Apr 23, 2025
We set the popover invoker when using command attributes to invoke a
popover, but there are no tests for this and the spec is missing it.

Spec PR: whatwg/html#11248

Change-Id: I36b60cc1925376c6ecf5c7f9e72d7b955cae71cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6481415
Reviewed-by: Mason Freed <masonf@chromium.org>
Commit-Queue: Joey Arhar <jarhar@chromium.org>
Reviewed-by: Keith Cirkel <chromium@keithcirkel.co.uk>
Cr-Commit-Position: refs/heads/main@{#1450757}
@annevk
Copy link
Member

annevk commented Apr 24, 2025

It would be good to state where this bug was introduced. I suspect this was simply inaccurate due to 81fa58f not taking into account other changes while it was developed? Because in that case we can just treat this as a follow-up fix to #9841 without much template fanfare.

moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Apr 25, 2025
…ttributes, a=testonly

Automatic update from web-platform-tests
Add WPT for popover invoker on command attributes

We set the popover invoker when using command attributes to invoke a
popover, but there are no tests for this and the spec is missing it.

Spec PR: whatwg/html#11248

Change-Id: I36b60cc1925376c6ecf5c7f9e72d7b955cae71cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6481415
Reviewed-by: Mason Freed <masonf@chromium.org>
Commit-Queue: Joey Arhar <jarhar@chromium.org>
Reviewed-by: Keith Cirkel <chromium@keithcirkel.co.uk>
Cr-Commit-Position: refs/heads/main@{#1450757}

--

wpt-commits: 21a49888f9625da31db6e6cc0f611a69033329ea
wpt-pr: 52129
sadym-chromium pushed a commit to web-platform-tests/wpt that referenced this pull request Apr 28, 2025
We set the popover invoker when using command attributes to invoke a
popover, but there are no tests for this and the spec is missing it.

Spec PR: whatwg/html#11248

Change-Id: I36b60cc1925376c6ecf5c7f9e72d7b955cae71cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6481415
Reviewed-by: Mason Freed <masonf@chromium.org>
Commit-Queue: Joey Arhar <jarhar@chromium.org>
Reviewed-by: Keith Cirkel <chromium@keithcirkel.co.uk>
Cr-Commit-Position: refs/heads/main@{#1450757}
i3roly pushed a commit to i3roly/firefox-dynasty that referenced this pull request Apr 29, 2025
…ttributes, a=testonly

Automatic update from web-platform-tests
Add WPT for popover invoker on command attributes

We set the popover invoker when using command attributes to invoke a
popover, but there are no tests for this and the spec is missing it.

Spec PR: whatwg/html#11248

Change-Id: I36b60cc1925376c6ecf5c7f9e72d7b955cae71cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6481415
Reviewed-by: Mason Freed <masonf@chromium.org>
Commit-Queue: Joey Arhar <jarhar@chromium.org>
Reviewed-by: Keith Cirkel <chromium@keithcirkel.co.uk>
Cr-Commit-Position: refs/heads/main@{#1450757}

--

wpt-commits: 21a49888f9625da31db6e6cc0f611a69033329ea
wpt-pr: 52129
shtrom pushed a commit to mozilla-firefox/infra-testing that referenced this pull request Apr 29, 2025
…ttributes, a=testonly

Automatic update from web-platform-tests
Add WPT for popover invoker on command attributes

We set the popover invoker when using command attributes to invoke a
popover, but there are no tests for this and the spec is missing it.

Spec PR: whatwg/html#11248

Change-Id: I36b60cc1925376c6ecf5c7f9e72d7b955cae71cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6481415
Reviewed-by: Mason Freed <masonf@chromium.org>
Commit-Queue: Joey Arhar <jarhar@chromium.org>
Reviewed-by: Keith Cirkel <chromium@keithcirkel.co.uk>
Cr-Commit-Position: refs/heads/main@{#1450757}

--

wpt-commits: 21a49888f9625da31db6e6cc0f611a69033329ea
wpt-pr: 52129
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants