-
Notifications
You must be signed in to change notification settings - Fork 98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
timeline love #4
Comments
Some ideas:
|
Hi, I have not pushed:
Will review tomorrow and push On Wed, Jul 30, 2014 at 7:45 PM, Forrest Oliphant
|
Sweet, thanks Marcin! |
Have you looked at https://github.com/idflood/timeline.js/commits/dev ? Seems like it has quite a lot of ui / integration changes. |
Hmm looks like good stuff there. Will you merge his stuff? @idflood is your dev branch stable? |
@idflood pull looks ok. I'll cherry pick from his other upgrades too. |
The dev branch is stable I think but the commits could have been better. Typically the "enable parameters and add some color options" add 2 features. The parameters one is important I think but the color option is not necessary. So yes, it may be a good idea to cherry pick what you need : ) And don't hesitate to ask if you have any question regarding some commits. |
@forresto Hi, started updating stuff and broke looping in the process. I'll continue tomorrow.. |
Ok, I fixed the looping but there is more work to be done with callbacks. Next will be UI customization based on @idflood work. |
Thinking about giving this some love to integrate it with https://github.com/noflo/noflo-ui . I'll pretty much just use the GUI and then make a new component in https://github.com/noflo/noflo-tween that actually drives the tweens.
If there is anything you have been planning or putting off, I'll try to contribute changes.
The text was updated successfully, but these errors were encountered: