Closed Bug 1346408 Opened 5 years ago Closed 5 years ago

stylo: CSS animations on pseudo elements

Categories

(Core :: CSS Parsing and Computation, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: hiro, Assigned: hiro)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

I did totally misunderstand about the pseudo styles on servo.
We had to pass pseudo_style to process_animations.
Note that I'm reworking pseudo-element handling in bug 1331047. Hopefully that will land next week, but probably best not to work on this until that lands.
Depends on: 1331047
Bobby, do you change how pseudo-styles are stored on elements? Otherwise this is probably not really dependent on it.

This is about the pseudo-element styles in bug 1341985 comment 47.
Flags: needinfo?(bobbyholley)
(In reply to Emilio Cobos Álvarez [:emilio] from comment #2)
> Bobby, do you change how pseudo-styles are stored on elements? Otherwise
> this is probably not really dependent on it.
> 
> This is about the pseudo-element styles in bug 1341985 comment 47.

I'm not sure I understand exactly what this bug is about after reading that comment, so I can't tell.

My changes mostly revolve around resolving the correct style for NAC-backed pseudo-elements like ::before and ::after, particularly in cases where the pseudos are nested, and particular during restyle. That may be unrelated to this bug, I just didn't want Hiro to spend a bunch of time debugging weird pseudo-related edge cases when that architecture will soon be overhauled.
Flags: needinfo?(bobbyholley)
(In reply to Bobby Holley (:bholley) (busy with Stylo) from comment #3)
> (In reply to Emilio Cobos Álvarez [:emilio] from comment #2)
> > Bobby, do you change how pseudo-styles are stored on elements? Otherwise
> > this is probably not really dependent on it.
> > 
> > This is about the pseudo-element styles in bug 1341985 comment 47.
> 
> I'm not sure I understand exactly what this bug is about after reading that
> comment, so I can't tell.
> 
> My changes mostly revolve around resolving the correct style for NAC-backed
> pseudo-elements like ::before and ::after, particularly in cases where the
> pseudos are nested, and particular during restyle. That may be unrelated to
> this bug, I just didn't want Hiro to spend a bunch of time debugging weird
> pseudo-related edge cases when that architecture will soon be overhauled.

Yeah, this is unrelated then, this is about choosing the wrong style for inheritance, but fixing this doesn't change how it's computed or anything. So this bug can be worked on then.
(In reply to Emilio Cobos Álvarez [:emilio] from comment #4)
> (In reply to Bobby Holley (:bholley) (busy with Stylo) from comment #3)
> > (In reply to Emilio Cobos Álvarez [:emilio] from comment #2)
> > > Bobby, do you change how pseudo-styles are stored on elements? Otherwise
> > > this is probably not really dependent on it.
> > > 
> > > This is about the pseudo-element styles in bug 1341985 comment 47.
> > 
> > I'm not sure I understand exactly what this bug is about after reading that
> > comment, so I can't tell.
> > 
> > My changes mostly revolve around resolving the correct style for NAC-backed
> > pseudo-elements like ::before and ::after, particularly in cases where the
> > pseudos are nested, and particular during restyle. That may be unrelated to
> > this bug, I just didn't want Hiro to spend a bunch of time debugging weird
> > pseudo-related edge cases when that architecture will soon be overhauled.
> 
> Yeah, this is unrelated then, this is about choosing the wrong style for
> inheritance

Ok, I assume you mean inheritance when computing eagerly cascaded pseudos on the originating element? If so, that should be ok. Bug 1331047 is all about picking the correct style for the elements that implement those eagerly-cascaded pseudos.
Thank you guys! I will post a fix here soon.
Comment on attachment 8846844 [details]
Bug 1346408 - Pass pseudo computed style and its parent style of the pseudo element.

https://reviewboard.mozilla.org/r/119842/#review121848

::: commit-message-72741:1
(Diff revision 1)
> +Bug 1346408 - Pass pseudo computed style and its parent style of the pseudo element. r?heycam

Pass to what?  Let's be a bit clearer in the commit message, maybe something like:

"Use computed styles of the pseudo and its parent when calling Gecko_UpdateAnimations to update CSS Animations on pseudo-elements."
Attachment #8846844 - Flags: review?(cam) → review+
Comment on attachment 8846845 [details]
Bug 1346408 - Update mochitest expectation.

https://reviewboard.mozilla.org/r/119844/#review121852
Attachment #8846845 - Flags: review?(cam) → review+
Attachment #8846844 - Attachment is obsolete: true
Thank you Cameron!
Pushed by hikezoe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/126f0f9a82d2
Update mochitest expectation. r=heycam
Assignee: nobody → hikezoe
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/126f0f9a82d2
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Blocks: 1351145
You need to log in before you can comment on or make changes to this bug.