Import frames from folder

[le717]

This has a higher priority level than projects because it is the fundamental process behind projects.

Code workflow idea (these are not supposed to be checkboxes. It's more of a brain dump):

• Scan specified folder for frames (as specified by the project file), generate file list of images
• Loop through list, adding each one to frame array, updating curFrame value, and calling updateFrameReel()
• Display some form of loading animation while this process takes place (I made a simple loading dots animation a while back).

[le717]

So I've implemented the core of this issue on branch issue-85 but it does not work because the fs.readdirSync option I'm using is not supported in our old version of Node. :( It's also not the most efficient because it's fully synchronous (it really should be asynchronous). Further, updating the UI is going to be a paaaaaaaiin because of the monster that is updateFrameReel().

[charlielee]

Nice stuff 😊 In 0.9.x I'm hoping to bump us from NW.js 0.26.6 to 0.35. This bumps us from Node.js 9.1 to 11.3 (on a sidenote CONTRIBUTING.md should really state the version of Node included with our current version of NW.js as the required one).

Framereel logic should really be in a separate class, perhaps this would be a good opportunity to make it less of a beast?

[le717]

I thought about going ahead and implementing it because (as you see, it's rather trivial) of the positive feedback to 0.8.0 and talk of possibly using it for THAC. That said, it is part of projects, so maybe it can wait until then.

Framereel logic should really be in a separate class, perhaps this would be a good opportunity to make it less of a beast?

Yes

[charlielee]

@le717 what is the status of the issue-85 branch? 😊

[le717]

The status is nothing has changed since my comment and the code is not on usable state yet.