joekrill / silverbullet-treeview

SilverBullet TreeView plug
MIT License
39 stars 11 forks source link

Fixes state issue & Uses more accurate events #14

Closed MrMugame closed 5 months ago

MrMugame commented 5 months ago

(1)

I removed all trailing spaces I could find, because it got annoying to work with otherwise

(2)

I looked into the sortable-tree library, because the way you fixed it, one sometimes got the tree twice, which was annoying. One case where that would happen is:

This is a bug in that library I made a PR there fixing it and added the version from my fork to this repository. I also removed the old error recovery mechanism, because it's now obsolete.

(3)

I fixed the registered event handlers to better capture all necessary events. I'm still not quite sure there but I think this now makes the tree feel more responsive.

joekrill commented 5 months ago

Thanks for this @MrMugame!