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)

defect
Not set
normal

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
Daisuke, could you please check where there are other properties that are filtered out?
Flags: needinfo?(dakatsuka)
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+
Attachment #8895276 - Flags: review?(dakatsuka) → review+
(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: nobody → hikezoe
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
(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)
(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.

Attachment

General

Created:
Updated:
Size: