please do not use base64

[DanielRuf]

In my opinion it makes no sense to create a huge JSON file, a mess of base64 encoded image files.

The approach from JSMovie is much faster and better
See also #3

The images can be also loaded parallely

[roman01la]

+1 to not to use base64, it's slow on mobile and drains battery because of heavy encoding process.
Please, check this research.

[vagnervjs]

Thanks for sharing this research @roman01la. It's realy true, and I agree. I'll implement this changes on Frame Player

[vagnervjs]

Hey @DanielRuf and @roman01la , I made the change of JSON/base64-URI to images files URL's. Can you guys see it and and leave your opinions?

The main code is: https://github.com/vagnervjs/frame-player/blob/img-tag/src/js/frameplayer.js
There is a branch just for this change: https://github.com/vagnervjs/frame-player/tree/img-tag

And the on-line player using this version is at: http://vagnersantana.com/frame/

[roman01la]

Looks good for me. Just wonder if still going to use CSS Filters? It's not just much heavier stuff in terms of battery life but also it makes a huge impact on the rendering performance on the web page. Should we move this to another issue?

[DanielRuf]

works much better, but this makes no sense at the moment:
https://github.com/vagnervjs/frame-player/blob/img-tag/src/js/frameplayer.js#L259

we should move this to the configuration in the js file and use the parameter set by the function

@roman01la did you take a look at the demo? they are much better than canvas filters in case of performance, or what do you think?

http://caniuse.com/#search=canvas

https://github.com/meltingice/CamanJS/
https://github.com/jseidelin/pixastic
https://github.com/mezzoblue/PaintbrushJS

[vagnervjs]

Thanks for the reply @roman01la

I thought in point of css filters too, just create this issue #6 about this

[vagnervjs]

Yes @DanielRuf, this constant no make senses, It's just for test at this moment, I'll create the new parameter

[DanielRuf]

@vagnervjs ok, I already thought this

[vagnervjs]

Add the new parameter frames: https://github.com/vagnervjs/frame-player/blob/img-tag/src/js/frameplayer.js#L261