You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think Blink's behavior (which if I understand correctly is "only apply the quirk if the only thing in the selector is :hover or :active and it's not nested) is simpler, and I think we should change the spec to match that, if my understanding is correct.
@lilles / @andruud: Could you confirm my read of Blink's source is correct? @annevk: Would WebKit be fine with that change? If so I'd be happy to send a PR.
cc @whatwg/css, I assume making stuff less quirky is generally good if we can get away with it...
The text was updated successfully, but these errors were encountered:
Here's a test-case. One quirk here is that at least Blink and Gecko parse *:hover and :hover to exactly the same thing. So ignoring that * special-case, yes, I think the chromium behavior is as I described above (for simple selectors at least). Not sure what the is_sub_selector in chromium conveys, it seems a mix of "being used for pseudo-elements" and "being in a nested pseudo-class / slotted / etc thing".
What is the issue with the Quirks Mode Standard?
https://quirks.spec.whatwg.org/#the-active-and-hover-quirk has a list of conditions that doesn't keep up with new CSS features.
It seems implementations do subtly different things that end up in subtle bugs like https://bugzilla.mozilla.org/show_bug.cgi?id=1856285:
I think Blink's behavior (which if I understand correctly is "only apply the quirk if the only thing in the selector is
:hover
or:active
and it's not nested) is simpler, and I think we should change the spec to match that, if my understanding is correct.@lilles / @andruud: Could you confirm my read of Blink's source is correct?
@annevk: Would WebKit be fine with that change? If so I'd be happy to send a PR.
cc @whatwg/css, I assume making stuff less quirky is generally good if we can get away with it...
The text was updated successfully, but these errors were encountered: