Labels
topic: popoverThe popover attribute and friends
Comments
Yep, I think this is the proper fix. Chromium indeed doesn't look at the attribute value, but instead the popover mode. |
I'd love to submit a fix for this (and my other popover issues) but I can't contribute without publishing my full legal name, which I don't feel comfortable doing. (I believe it would also be legally dubious to give a to someone to submit on my behalf) |
josepharhar added a commit to josepharhar/html that referenced this issue Apr 8, 2025
This PR prevents an assertion from being hit in hide all popovers until by changing the conditions in which hide all popovers until gets called by the hide popover algorithm to use the opened in popover mode instead of looking at the current value of the popover attribute. Fixes whatwg#10996
6 tasks
Thanks for figuring this out! Here's a PR: #11197 |
Sign up for free to join this conversation on . Already have an account? Sign in to comment
What is the issue with the HTML Standard?
Here's the problematic scenario:
When the popover is shown, its opened in popover mode is set to "hint".
Upon changing the popover attribute:
If element's popover attribute is in the auto state or the hint state, then: Run hide all popovers until given element
(step 7.1).If document's showing hint popover list contains endpoint: Assert: endpoint's popover attribute is in the hint state
(step 6.1)The text was updated successfully, but these errors were encountered: