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

Comments

Expand chained timing functions#1

Open
steveblock wants to merge 3 commits intow3c:masterfrom
steveblock:timingFunctions
Open

Expand chained timing functions#1
steveblock wants to merge 3 commits intow3c:masterfrom
steveblock:timingFunctions

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