@damsfx ooo, I like this. And in Snowboard too! 😍

Couple of quick comments/questions:

• How performant is it with a whole heap of items (like, say, 100s of items)?
• Could the animation be a touch faster? Might be just on the border of being a bit too slow to feel "snappy".

@bennothommo

And in Snowboard too! 😍

Definitely yes!
The more I practice, the more I love it! It's good to be back in a vanilla javascript environment.

Despite this, a function written in jQuery is still called to update the scroll bar. :o(

How performant is it with a whole heap of items (like, say, 100s of items)?

The tree structure used for my tests and visible in the animation above contains 550 nodes spread over 4 levels of depth.
I can always do a test with more nodes.

Could the animation be a touch faster? Might be just on the border of being a bit too slow to feel "snappy".

For sure.

@bennothommo

Latest update :

• Use promise to adjust srcollbar after expand/collaspe node(s)
• CloseAll() close only opened nodes instead of all nodes
• Reduce animation timmings
• Use CSS animation end to trigger scrollbar update

@damsfx don't be too concerned if you need to fall back on jQuery to do something in the Backend. A lot of the Backend still uses it and the goal of Snowboard was to remove the dependency of jQuery for the CMS side in particular.

@bennothommo are you happy with this as it is?

@damsfx is there a related docs PR?

@damsfx is there a related docs PR?

Not yet, but I can add a pending PR for this.

@damsfx is there a related docs PR?

@LukeTowers Done : wintercms/docs#243

@bennothommo I know your time is precious, but if you have 5 mins to take a look and give me a little feedback if you see any improvements to be made.
Thank you for your time.

@damsfx sorry for the delay. Looks great, and the timing of the animation is much better.

Only issue I can see now is that it appears that some plugins and themes were added in your most recent commit:
90fbd40

Could you make sure these are not included in your branch? Once that's done, @LukeTowers should be able to merge if he's happy with it.

pending removal of plugins and themes in recent commit

😱🙄 Oh, I was really distracted!
Thanks for reminding me.

Done !