Childless Nodes

[ksaxberg]

Let me know if there is any additional information you need, either in this PR or about these changes!

We're looking to utilize this awesome package, but we need an additional feature. The filesystem theme is great, but you can preview dropping a file into another file. This drop is rejected, which is nice, but we shouldn't really allow users to preview a file being underneath another file. The custom hover implementation is awesome, and adding a little more control over the hover state would allow us to use the package to the fullest extent.

Did some digging to try and figure out where this change would be sensible. Having a parameter to the SortableTree would be nice, and it needs to be called over in the dndManager. So I added the canNodeHaveChildren property function, which defaults to true.

Preview of Example

Here's a quick gif preview of using the new story with "people" nodes as not being able to have children, but other nodes being able to.

[coveralls]

Coverage decreased (-0.7%) to 74.649% when pulling 01147fb on ksaxberg:childless-nodes into ff980d8 on frontend-collective:master.

[wuweiweiwu]

LGTM! Will merge after documentation is added to the README @ksaxberg

[ksaxberg]

@wuweiweiwu Added a quick blurb to the README about the new prop:

Function to determine whether a node can have children, useful for preventing hover preview when you have a canDrop condition. Default is set to a function that returns true. Functions should be of type (node): bool.

Thanks for pointing out that I missed adding it to the README!

[wuweiweiwu]

Awesome! thanks again @ksaxberg

[wuweiweiwu]

Will update the thread when it released

[grydstedt]

@wuweiweiwu how often do you do releases? Thanks

[wuweiweiwu]

@grydstedt @ksaxberg released in 2.3.0!!!

[ksaxberg]

Thanks!