dynamic :hover changes don't work inside :not()

RESOLVED FIXED

Status

()

Core
CSS Parsing and Computation
RESOLVED FIXED
15 years ago
15 years ago

People

(Reporter: dbaron, Assigned: dbaron)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

15 years ago
The test at
http://www.hixie.ch/tests/evil/css/selectors/tests/xhtml/full/flat/css3-modsel-63.xml
fails because dynamic :hover changes don't work inside :not().

Patch coming.
(Assignee)

Comment 1

15 years ago
Created attachment 126342 [details] [diff] [review]
patch (diff -uw)
(Assignee)

Comment 2

15 years ago
It's worth noting that this patch isn't quite optimal for two reasons.  One, we
could add something to one of these lists twice, which means we'll check it
twice, but I don't think that's worth worrying about.  Two, we don't really need
to check negations past the first in the current implementation.

(I guess I should add the above as a comment.)
Comment on attachment 126342 [details] [diff] [review]
patch (diff -uw)

Add that comment you pasted on irc, and looks great.
Attachment #126342 - Flags: superreview+
Attachment #126342 - Flags: review+
(Assignee)

Comment 4

15 years ago
Fix checked in to trunk, 2003-06-23 22:34 -0700.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.