Page MenuHomePhabricator

Bug 1514086 - The 'all' property is not animatable.
ClosedPublic

Authored by emilio on Dec 14 2018, 10:26 PM.

Details

Summary

Since it allows to animate display, which is not good.

This is a regression from:

https://hg.mozilla.org/mozilla-central/rev/6884ba750aa3

Actually I wonder if the logic shouldn't be the other way around, i.e., a
shorthand is animatable if all the longhands are, not if just one.

In any case this rolls back to the previous behavior, should we do that, it
should be another bug.

Diff Detail

Repository
rMOZILLACENTRAL mozilla-central
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

emilio created this revision.Dec 14 2018, 10:26 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 14 2018, 10:26 PM
phab-bot changed the visibility from "Custom Policy" to "Public (No Login Required)".Dec 14 2018, 10:26 PM
phab-bot changed the edit policy from "Custom Policy" to "Restricted Project (Project)".
phab-bot removed a project: Restricted Project.
emilio requested review of this revision.Dec 14 2018, 10:26 PM
hiro accepted this revision.Dec 15 2018, 10:48 PM
This revision is now accepted and ready to land.Dec 15 2018, 10:48 PM
This revision was automatically updated to reflect the committed changes.