Page MenuHomePhabricator

Bug 1808410 - Part 1: Move animation-related values from box.rs/ui.rs to animation.rs.
ClosedPublic

Authored by boris on Mar 28 2023, 11:18 PM.
Referenced Files
Unknown Object (File)
Jul 11 2024, 2:15 AM
Unknown Object (File)
Jun 2 2024, 4:59 AM
Unknown Object (File)
Mar 29 2023, 12:29 AM
Subscribers

Details

Summary

Although we store animation and transition style values in StyleUIReset and
define their properties in longhands/ui.mako.rs, but we may move them in
the future if this style struct becomes too large. So let's move the
definition of their values to an independent module, animation, so we
don't have to worry about this again.

This patch doesn't change any other things. Only move code.

Diff Detail

Repository
rMOZILLACENTRAL mozilla-central
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

phab-bot changed the visibility from "Custom Policy" to "Public (No Login Required)".
phab-bot changed the edit policy from "Custom Policy" to "Restricted Project (Project)".
phab-bot removed a project: secure-revision.

Fix clang-format warnings

May be worth while at it to have separate animations / transitions modules?

This revision is now accepted and ready to land.Apr 20 2023, 12:03 PM

May be worth while at it to have separate animations / transitions modules?

Yes. I decide to move them into animation module.

boris retitled this revision from Bug 1808410 - Part 1: Move animation-related values from box.rs to ui.rs. to Bug 1808410 - Part 1: Move animation-related values from box.rs/ui.rs to animation.rs..Apr 20 2023, 11:07 PM
boris edited the summary of this revision. (Show Details)