Skip to content
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

Tiddlymap makes lazy loading not work #300

Open
guillefix opened this issue Nov 9, 2018 · 5 comments
Open

Tiddlymap makes lazy loading not work #300

guillefix opened this issue Nov 9, 2018 · 5 comments
Labels
bug Something that should work is not working

Comments

@guillefix
Copy link

I tried installing tiddlywiki as mentioned here because I had issues with static building before, but now lazy loading does not work. I tried twice, and also removing all the plugins except tiddlymap and vis, and it did the same thing. Node.js sends all the tiddlers even with $:/core/save/lazy-all, and the wiki takes ages to finish loading.
Removing the tiddlymap plugin makes it work again.
This happened with Tiddlywiki 5.1.16 and 5.1.17

@felixhayashi felixhayashi added the bug Something that should work is not working label Jan 29, 2020
@felixhayashi
Copy link
Owner

Hi @guillefix thanks for reporting. I am sorry for this but I don't think this will be fixed since I do not use static building and Tiddlymap also relies on accessing tiddler contents even if they are not loaded yet, so this may conflict with the concept of lazy loading or results in the whole wiki being loaded one by one 😞

@danielo515
Copy link

This is indeed a big problem. However, if the tiddlers that are "key" are a fixed set they can be embedded on the index.html or pre-loaded before tiddlymap
Which are the exact limitations?

@felixhayashi
Copy link
Owner

Hi @danielo515 , hope you are doing well! 🙂
Since I am not using lazy loading, I can only make assumptions: I think it is the general mechanism of TiddlyMap that tries to access tiddler contents using js (via TiddlyWiki's provided methods such as $tw.wiki.getTiddler). The problem is that those methods are not written in an async style which means that TiddlyMap assumes access is direct and instant when in fact the tiddlers are not ready and loaded lazy. This may confuse TiddlyMap when it tries to visualize a map etc. 🤷‍♂️ but to be honest, I am lost here.

@danielo515
Copy link

Do you use the text field? That's the only field that is not available on lazy loaded tiddlers @felixhayashi

@felixhayashi
Copy link
Owner

Good point, I may be using it indirectly by calling getTiddlerLinks on nodes in the map, also if a node contains an image stored in another tiddler I try to access the text field... other situations may exist. 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something that should work is not working
Projects
None yet
Development

No branches or pull requests

3 participants