-
Notifications
You must be signed in to change notification settings - Fork 333
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Multi selection support for treebox #691
Comments
Hi again! After debugging and modifying the code, I am able to add support for multiselect. I added Toggle and Replace Select (ctrl and shift key modifiers respectively). To sum up:
Existing set_selected function was modified to manage selected node vector:
Finally to know if a node is selected before rendering it, modified node attribute for selected field: And that's pretty much of it. Want to know what do you think guys? @cnjinhao @qPCR4vir. |
Hi, @edgarbernal5 , good job! |
Thanks!!! I created the PR #693 |
How do we use this new feature? In the client side, set the flag to true I just added two new methods in treebox:
and this:
|
Look forward to your PR. |
Nice!! The PR is created #694. |
Hi! Just wondering if we can add support for multi selection as already is with listbox, don't know if you have plans to do it soon, and if not can you give me an insight of it, where can I start from?
Thanks in advance!
The text was updated successfully, but these errors were encountered: