[Github-comments] [geany-plugins] Addons bookmark refresh fix (#284)

[Enrico Tröger]

Thanks for the feedback.

I changed the document referencing to IDs as suggested.

Refreshing the treeview with about 5000 markers set takes about 40-50ms with de- and reattching the model. Without detaching, refreshing takes about 60-70ms, on my machine.
So it is actually not that big difference and I doubt users will click that many markers. I cheated a bit by programatically setting them on startup :).

So, as the difference is actually not really big, I removed commit 0d130be42b3e7968c597c85a1782b1e34d69d5a8. However, I could not notice any flickering, with or without the commit.

---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/284#issuecomment-141519831
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20150918/6bdf7f99/attachment.html>