Update jstreer.js

[ismirsehregal]

Again using changed.jstree event instead of select_node.jstree and deselect_node.jstree to fix #34

The fix for #16 is not affected

[stla]

Hi @ismirsehregal

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.

[ismirsehregal]

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 select_node.jstree and deselect_node.jstree events instead of changed.jstree, we could add listeners for select_all.jstree / deselect_all.jstree as an alternative solution. Although, I haven't noticed any issue using changed.jstree and it seems to be the recommended way as it covers all cases.

[ismirsehregal]

Hi Stéphane, have you had the chance to test this? Cheers

[stla]

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.

[ismirsehregal]

Okay - thanks for the update! Please let me know if there is anything I can do to help. Cheers

[stla]

Yes you can ;-) Tell me whether there's a nice IDE for R that works with Windows 7. I really miss RStudio.

[ismirsehregal]

Uh 🙈... that's bad.

Maybe you can downgrade:
https://forum.posit.co/t/what-is-the-latest-rstudio-version-for-windows-7/152152

..or once I installed RStudio Server on a Raspberry Pi 😅

[ismirsehregal]

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