Closed
Bug 1388652
Opened 7 years ago
Closed 7 years ago
stylo: list-style-type is not animatable yet
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: hiro, Assigned: hiro)
References
Details
Attachments
(2 files)
The test cases are filtered out by isSupported() check somehow, so the failures have not bee n appeared.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=a48a3eb5fa2070979923cb92427d781a94236df1
Assignee | ||
Comment 1•7 years ago
|
||
Daisuke, could you please check where there are other properties that are filtered out?
Flags: needinfo?(dakatsuka)
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8895275 [details]
Bug 1388652 - Drop AnimatedProperty for stylo.
https://reviewboard.mozilla.org/r/166452/#review171598
Well, I guess. Adding the extra argument isn't a huge deal either, and maybe it'd be preferrable... The less conditionally compiled stuff the better, but anyway it's fine :).
Attachment #8895275 -
Flags: review?(emilio+bugs) → review+
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8895276 [details]
Bug 1388652 - Make list-style-type animatable
https://reviewboard.mozilla.org/r/166454/#review171600
Attachment #8895276 -
Flags: review?(dakatsuka) → review+
Comment 6•7 years ago
|
||
(In reply to Hiroyuki Ikezoe (:hiro) from comment #1)
> Daisuke, could you please check where there are other properties that are
> filtered out?
OHH,, Okay!
Assignee | ||
Comment 7•7 years ago
|
||
Assignee: nobody → hikezoe
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Comment 8•7 years ago
|
||
(In reply to Hiroyuki Ikezoe (:hiro) from comment #1)
> Daisuke, could you please check where there are other properties that are
> filtered out?
Currently, all filtered properties are logical properties. Since they are not animatable yet.
However, font-variation-settings[1] property is not register in the list, also does not support the animation.
(Gecko defines as eStyleAnimType_Custom)
Also, flex-box is not register as well, and does not implement for Servo yet[2].
I'll file a bug for font-variation-settings at least, since looks there is not such the bug.
[1] https://drafts.csswg.org/css-fonts-4/#font-variation-settings-def
[2] https://searchfox.org/mozilla-central/source/servo/components/style/properties/longhand/position.mako.rs#177
Flags: needinfo?(dakatsuka)
Assignee | ||
Comment 9•7 years ago
|
||
(In reply to Daisuke Akatsuka (:daisuke) from comment #8)
> (In reply to Hiroyuki Ikezoe (:hiro) from comment #1)
> > Daisuke, could you please check where there are other properties that are
> > filtered out?
>
> Currently, all filtered properties are logical properties. Since they are
> not animatable yet.
>
> However, font-variation-settings[1] property is not register in the list,
> also does not support the animation.
> (Gecko defines as eStyleAnimType_Custom)
> Also, flex-box is not register as well, and does not implement for Servo
> yet[2].
I think you mean flex-basis.
https://searchfox.org/mozilla-central/source/servo/components/style/properties/longhand/position.mako.rs#165
You need to log in
before you can comment on or make changes to this bug.
Description
•