Closed Bug 1384020 Opened 8 years ago Closed 8 years ago

Stylo: CSS targeting nested element on hover state doesn't work (quirksmode)

Categories

(Core :: CSS Parsing and Computation, defect)

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla56
Tracking Flags:
Tracking Status
firefox56 --- fixed

People

(Reporter: rohitpaulk, Assigned: emilio)

References

(Blocks 1 open bug,
URL
)

Details

(Keywords: nightly-community)

Attachments

(2 files, 1 obsolete file)

test page to reproduce the bug
528 bytes, text/html
Details
Bug 1384020: Test.
59 bytes, text/x-review-board-request
canova
: review+
Details
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:56.0) Gecko/20100101 Firefox/56.0 Build ID: 20170724100311 Steps to reproduce: - Using nightly. Version: 56.0a1 (2017-07-24) (64-bit). - I've turned `layout.css.servo.enabled` on. When that setting is turned off, the bug is not reproducible. Here's a test page: ``` <body> <h2>Here's the bug</h2> <div class="container"> <span class="remove-this-and-it-works"> <span class="only-on-hover">Only visible on hover</span> </span> <span> Text should apper to the left when you hover on this. </span> </div> <style> .container .only-on-hover { visibility: hidden; color: red; } .container:hover .only-on-hover { visibility: visible; } </style> </body> ``` - I hovered on the text, triggering the :hover state on `.container` Actual results: - The `.only-on-hover` element's visibility did not change Expected results: - The `.only-on-hover` element's visibility should've changed to `visible`.
Product: Firefox → Core
Summary: CSS targeting nested element on hover state doesn't work with stylo enabled → CSS targeting nested element on hover state doesn't work with servo enabled
Component: Untriaged → CSS Parsing and Computation
Summary: CSS targeting nested element on hover state doesn't work with servo enabled → Stylo: CSS targeting nested element on hover state doesn't work
Thank you! :) Confirmed in Nightly 56 x64 20170725144053 @ Debian Testing.
Blocks: stylo-site-issues
URL: https://bugzilla.mozilla.org/attachme...
Status: UNCONFIRMED → NEW
Has STR: --- → yes
Ever confirmed: true
Keywords: nightly-community
Version: 56 Branch → Trunk
Huh, this is an interesting one... If I change it to be other kind of selector like a class, which should trigger a very similar invalidation pass, it does work. And furthermore, I've verified that the invalidation pass works as intended, that is, finds the "only-on-hover" element, and marks it as dirty...
Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:56.0) Gecko/20100101 Firefox/56.0 Confirming. This bug is only in quirksmode for me, works in standards mode (with <!doctype html>). (Changing summary)
Summary: Stylo: CSS targeting nested element on hover state doesn't work → Stylo: CSS targeting nested element on hover state doesn't work (quirksmode)
Gah, I know why this is... Nazim, this is due to the :hover quirk.
Assignee: nobody → emilio+bugs
Attachment #8890097 - Flags: review?(canaltinova)
Comment on attachment 8890097 [details] selectors: Fix note_next_sequence. https://reviewboard.mozilla.org/r/161172/#review166514 Looks good. Thanks for fixing this!
Attachment #8890097 - Flags: review?(canaltinova) → review+
Attachment #8890097 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/215c7f7ff026
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Verified fixed in Nightly 56 x64 20170726100241 @ Debian testing. (Stylo: true (enabled by user)) The hidden text in attachment 8889774 [details] is shown on mouseover even with Stylo now.
Status: RESOLVED → VERIFIED
Yay! That was quick :)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: