callumbwhyte / meganav

A flexible, draggable link picker for constructing site navigation menus in Umbraco
MIT License
35 stars 34 forks source link

Populate from tree - Select node #56

Open eden-jh opened 2 years ago

eden-jh commented 2 years ago

Let me know if you'd like me to split this up into separate issues. Somewhat related to #55

Problem

The Populate from tree option in the property actions only populates from direct children of the current content node (the one containing that property). It always adds all children of that node. This creates a problem if:

Proposed solution

callumbwhyte commented 2 years ago

Hey @eden-jh,

I've been thinking about this one a bit and agree the current "populate from tree" implementation isn't great...

As you point out, the most obvious flaws are:

I think the best and most "Umbraco way" of doing this is to present the user with a multiple content picker dialog, allowing them to choose which nodes (at any level) should be included. This gets around the need for configuration options etc too.

I feel sacrificing a tiny bit of the autonomy for better control is a good trade off. What do you think?

I've got a prototype working in the feature/populate-tree branch - will do some testing and get some opinions before looking at a release.

Cheers, Callum

eden-jh commented 2 years ago

@callumbwhyte I think that could work, though the usability of the multiple-select will be important (like the ability to select all children of a specific node without manually clicking on each one).

I can't seem to find a feature/populate-tree branch. Am I looking in the wrong place, or has it not been pushed to Github yet?