Question: ProseMirror dispatch decorations about mlmorph HOT 4 CLOSED

Have you consider using getChangedRanges to execute the plugin only on the changed nodes?

No. I was not aware of this. Thanks for the link.

There are many optimization ideas we can apply for spellcheck or similar usecases. For example, keeping the dictionary of words and the corresponding API result(spellcheck result) and checking the word in that one before an API request, so that we can fast lookups. That would be independent of transactions and text changes. Imagine a paragraph is cut(Say Transaction-12) and then pasted somewhere else as Transaction-42. In such case, a content diff from previous transactions wont save API cost, but the cached results would help. Am I right?

For my usecase, I was not interested in optimizing this editor usage since my purpose is just demo the spellcheck API. Improving the spellcheck and underlying morphology anayser is where I would spend time. But definitely the spellchecker won't be useful if there is no good text editor :-)

from mlmorph.

Thanks for the question. Yes, I did this intentionally, but not for any performance optimization(I have not checked if there is a problem with performance by doing like this). This was just a lazy attempt to avoid waiting for all promises(checknode is async) to finish, aggregate all problems. I thought if I just dispatch the transaction, once the node is checked is more easier and may be faster response to user. By the way, nodes are not necessarily lines, they are probably paragraphs, or inline markup elements.

from mlmorph.

That's interesting, thank you very much.

Have you consider using getChangedRanges to execute the plugin only on the changed nodes?

from mlmorph.

Thanks so much for your insights! Very helpful and much appreciated!

from mlmorph.