-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update jstreer.js #35
base: master
Are you sure you want to change the base?
Conversation
update shiny input on changed.jstree event
Did you identify the reason for which I commented out the line? Because I don't want to restore this line if I don't know this reason. |
Hi Stéphane, it seems you haven't actually commented it out. The (commented) line was added with this commit (triggered by this issue) (see the diff - nothing was deleted. Maybe it was just for testing and you haven't noticed any difference?). However, if you rather want to keep using the |
Hi Stéphane, have you had the chance to test this? Cheers |
My main laptop is dead. Now I only have an old one (with Windows 7) and I don't like to work on this one. But I will. |
Okay - thanks for the update! Please let me know if there is anything I can do to help. Cheers |
Yes you can ;-) Tell me whether there's a nice IDE for R that works with Windows 7. I really miss RStudio. |
Uh 🙈... that's bad. Maybe you can downgrade: ..or once I installed RStudio Server on a Raspberry Pi 😅 |
Hi Stéphane, I just wanted to leave a frindly reminder, that it would be really helpful if this is merged (or the alternative approach mentioned above is implemented). Cheers |
Again using changed.jstree event instead of
select_node.jstree
anddeselect_node.jstree
to fix #34The fix for #16 is not affected