Tween: Full version [#seek] method isn't working

[dalisoft]

Issue

Issue name

Tween: Full version [#seek] method isn't working

Issue description

Tween: Full version [#seek] method isn't working

Related

This issue is related to

• [ ] LICENSE

• [ ] README

• [ ] ABOUT COPYRIGHT VIOLATION (be careful before checking this...!)

• [ ] package.json

• [ ] demos/examples

• [ ] core.ts

• [x] Tween.ts

• [ ] lite.ts

• [ ] Easing.ts

• [ ] Interpolation.ts

• [x] Timeline.js

• or provide something else (from es6-tween files...)

Example

https://codepen.io/dalisoft/pen/boBNaV (remove commented code)

What you excepted?

Seek the tween

NOTE

• Please add read code, docs, files, issues, PR list for avoid duplicating existing question, bug or something else...

[dalisoft]

I can't figure out why my fix ways not working, help would be nice. @trusktr Maybe we merge this repo to tween.js for more help from community?

[trusktr]

Maybe we merge this repo to tween.js for more help from community?

Yeah, that'd be great because the community is using the main tween.js repo, and focus is there. I would suggest to pick your favorite ideas from here and start adding them there.

The creatUi stuff is really cool and useful! I think it'd be nice to have a separate repo just for that, as a standalone package (rather than bundling in the same tween.js package).

[dalisoft]

Thank you for best words. I not understand (english) good, but it's mean we start merge?

[dalisoft]

Or we make github repo for createUi? Its already on npm. I am working on compatibility for tween.js

[dalisoft]

@trusktr Now Tween.js also compatible with createUi, here link

But you should add

TWEEN.Tween.prototype.play = TWEEN.Tween.prototype.start;

to make play button visible, as it's dynamic by method checking. Make sure call this before createUi instance

Maybe you use Lite version?

Also don't forget to check Three.js demo. (see directly material.color tweening) how it handles

[dalisoft]

About [#seek] method, it should work partially, always better having partly working than nothing.

[trusktr]

it's mean we start merge?

Yeah, if you can open some new pull requests with changes to Tween.js, then I can review them.

we make github repo for createUi?

A new repo would be great. I made on here: https://github.com/tweenjs/tween-ui. How do you feel about that name? If you can open pull requests to start merging there, I can start reviewing them. A couple notes:

• perhaps tweenUi would be a better name for the function.
• It should reference only tween.js (TWEEN) so that it is specific for the Tween.js project. (thoughts @mikebolt ?)
• TWEEN.Tween.prototype.play = TWEEN.Tween.prototype.start; - this shouldn't be required. But if you think so, can you open a PR on Tween.js so we can discuss it?

Also don't forget to check Three.js demo. (see directly material.color tweening) how it handles

Which demo? Got a link?

[trusktr]

Basically, let's make the tween-ui work with the current Tween.js, not considering other libraries in the code. And let's focus on merging your ideas one by one into Tween.js. Then that way many people can enjoy your new features! 👍

[dalisoft]

tweenjs.github.io/es6-tween then click Getting Started then scroll to bottom and check Simple demos page

[dalisoft]

You have access to create repo on tweenjs? I do not have access

[dalisoft]

I think one thing. After these changes to tween.js es6-tween repo will be dead as users can get these things from tween.js?

[trusktr]

Yes I do. Sole is busy. The best I can do without her is make a new repo and I can help you merge your code by reviewing PRs.

[dalisoft]

I will back in computer after week then i commit to tween ui

[trusktr]

Bring your knowledge to Three.js, then eventually when/if @sole is around town she can put you on the team that has access. We can start with PRs for now, to get your ideas in, and as I said I am happy to help review so you can merge. :)

[dalisoft]

@trusktr I got full time working (14hour per day), so sorry i have not time to it. @sole if add me, i will do later pr or you can do it yourself

[trusktr]

After these changes to tween.js es6-tween repo will be dead as users can get these things from tween.js?

Forgot to answer that. Yes, users will enjoy your changes in tween.js, and es6-tween will become obsolete.

[dalisoft]

then my plugin architecture and lite (faster) becomes unused? The createui is compatible with tween.js. I want to keep es6 tween live and make it popular. Also i think i should move repo to my profile as i can make code coverage, improve code quality and much more by other CI services

[trusktr]

then my plugin architecture and lite (faster) becomes unused?

Unless you would like to prose PRs to implement something like it in Tween.js

I want to keep es6 tween live and make it popular. Also i think i should move repo to my profile as i can make code coverage, improve code quality and much more by other CI services

True, you could do that.

The createui is compatible with tween.js.

Interested to put it in the http://github.com/tweens/tween-ui repo?

[trusktr]

I meant https://github.com/tweenjs/tween-ui

[dalisoft]

I will do that but it should be compatible with es6-tween too. I havent time nw. But you free to imlement that

[dalisoft]

I will do but later

[dalisoft]

Until now, even, no one of users didn't ask about issue, so if someone need that, please use GSAP/TweenMax (works more than one dev), i'm alone can't fix this