Replies: 3 comments 4 replies
-
I am certainly open to making major changes to the API in v2.0. It is something I have been mulling about myself. The current iteration of Having an alternative mechanism to fetch I would like more details on the As an FYI, there is a branch, |
Beta Was this translation helpful? Give feedback.
-
The reasoning for this choice is that The code below shows the the example using the new models. I am not sure I like this. I think that maybe The
|
Beta Was this translation helpful? Give feedback.
-
I tried yesterday to make I've been debating using the When using the Using the Currently my
I hope to have a working example soon. |
Beta Was this translation helpful? Give feedback.
-
@jakezatecky
I have an idea for a breaking change to
CheckboxTree
. I'll be working on this full time over the next couple of weeks. I'm not sure if you will be interested so I thought I'd describe it here and see what you think. This change is driven by my needs for a tree component to manage map layers in my application.The problem I am having is that the
node
returned as a function parameter toonCheck
is not completely valid. Thechecked
array is not useful to me and I do not use it in my application. I do realize that others may use this array. In my application a check change will turn on/off layers in a map. When I get the changed node, I must go down to the leafs and make the appropriate layer visibility changes. Sometimes the parent is necessary in this processing also (radio groups, another enhancement I'll discuss later).The tree configuration is more easily managed by a human in object form thus the
nodes
prop used inCheckboxtree
. I am proposing to flatten the tree before passing it toCheckboxTree
as aflatTree
prop. This plus aroot
prop will be the props that control the component instead ofnodes
,checked
andexpanded
. I propose adding thechecked
andexpanded
properties to thenodeShape
as shown below.With these changes, the example from your
README.md
would look like this:Please let me know if you are interested in these changes and I'll can discuss it with you more.
Beta Was this translation helpful? Give feedback.
All reactions