Skip to content
This repository was archived by the owner on Jun 7, 2018. It is now read-only.

Expand chained timing functions #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

steveblock
Copy link
Contributor

No description provided.

@birtles
Copy link
Contributor

birtles commented Aug 22, 2013

Pushed a changeset based on this:
https://dvcs.w3.org/hg/FXTF/rev/9ea455a47bf4

We agreed to only allow specifying the x positions of a chained timing function for now so that's all I've added for now. Likewise I haven't added smoothing.

@steveblock
Copy link
Contributor Author

Looks good Brian! I really like 'distribute' and 'align'.

My initial reaction was that it's a shame to require authors to include '0'
at the start of 'easingTimes: it seems unnecessary and a little ugly that
the lengths of 'easing' and 'easingTimes' don't match. But I guess that if
we plan to relax the restrictions on timing functions to operate outside
the range [0,1], we'll want to relax the restrictions on 'easingTimes'
similarly, and we'll then need both limits to be explicitly specified.

Steve

@birtles
Copy link
Contributor

birtles commented Aug 22, 2013

Yeah, I wondered about that. Originally I specced it so you wouldn't fill in 0 or 1 but it seemed a bit odd to me to just say easingTimes: 0.75. I hadn't really thought about forwards compatibility but that's a good point.

@steveblock
Copy link
Contributor Author

I've updated this branch to reflect Brian's updates to the spec to add timing functions chains. These commits add the ability to specify output times in the timing function position list, and add smoothing. They're not for immediate inclusion in the spec but are uploaded to document the latest proposal on these topics.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants