I think this intentional to indicate that these are meant to be callbacks, not regular functions. Not a usual convention and there is the _fn
suffix now, so we can probably clean that up. I'm…
I do think it would make sense to show the
Default
directory as read-only in the list, as you can explicitly select it when installing an add-on. It's a bit confusing without that right now,…
I figured that a generic struct like NamedDirectoryPathEntry
probably isn't a good idea. Should we want more options for one of its use-cases (e.g. a toggle to enable/disable a script directory)…
From further discussions: It's not clear how this would break with the extensions platform project. And it's quite uncertain how long this project will take to release. Meanwhile a whole number…
I was going to make this struct a general utility, since I can see a number of use cases for a name + directory-path struct. Not all may require a unique name, so noted that accordingly.
Will leave this up to you to decide. Can be removed in a separate change.
nullopt
return value instead of {}