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

Improve hover tooltip cache invalidation #368

Open
2 tasks
krassowski opened this issue Sep 18, 2020 · 0 comments
Open
2 tasks

Improve hover tooltip cache invalidation #368

krassowski opened this issue Sep 18, 2020 · 0 comments

Comments

@krassowski
Copy link
Member

The tooltip cache introduced in #363 has many advantages, but the cache invalidation is not perfect.

  • the cache can get outdated if a file from which something is imported changes; currently we do not have any way to invalidate this. We could schedule a refresh of a cached response to run in the background while we show the most recent response we have in the meantime.
  • the cache does not invalidate on undo triggered by ctrl + z. It could be because this might be managed by the lab and not by the CodeMIrror. It would need to listen to the model changes, but model is not exposed to the features at the moment!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant