BoostIO / BoostNote-Legacy

This repository is outdated and new Boost Note app is available! We've launched a new Boost Note app which supports real-time collaborative writing. https://github.com/BoostIO/BoostNote-App
Other
17.07k stars 1.47k forks source link

Entire application lags when typing with lots of LaTeX #3582

Open jensmcatanho opened 4 years ago

jensmcatanho commented 4 years ago

Current behavior

I'm currently experiencing lag when typing on notes that have a lot of lines, including LaTeX and/or diagrams. Here's a video of this behavior and the note used on the video.

Might be related with issue #2541

Expected behavior

The text should not take a few seconds to render after I type.

Environment

Flexo013 commented 4 years ago

Can confirm that lots of LaTeX increases computation/rendering time immensely. Typing in this specific note takes about 3-4 GHz to render while typing.

We probably need some improvements to rendering LaTeX. Such as caching those parts and/or doing smart analysis of where one is typing to not re-render those parts.

Another part that would help here is to detach the rendering of the editor from the preview. That way you can still see your edits, and we could display something when rendering is taking more than a tenth of a second (or something).

Ckbhatia commented 4 years ago

@Flexo013 Please add a different label for the platform: Linux, Browser, Android and IOS.

Flexo013 commented 4 years ago

This version of Boostnote (Legacy) does not have a browser or mobile app version. It only runs on Linux, MacOS, and Windows.

I have reproduced the behavior that you (Mac) reported on my Windows and Linux systems. So the issue of lag for these notes is platform independent.

huzibizi commented 4 years ago

the pastebin link to the code no longer exists, so i can't check to see if the problem still occurs with the latest version of boostnote for windows

Flexo013 commented 4 years ago

@jensmcatanho Could you provide another copy of the note?

jensmcatanho commented 4 years ago

Sorry for the delay. I'm not sure if this was the original note but I was able to reproduce the problem with it.