Syntax module does not correctly highlight multi-line comments

[micimize]

Reproducable with the example editor at beta.quilljs.com:

We should wrap them in a single <pre/> similar to how lists are wrapped by <ol/>:

[jhchen]

One line blocks are not a problem by itself. There are many benefits to it such as consistent newline creation/deletion, at any position across all supported platforms, which is why the implementation choice was made in the first place. The incorrect highlighting for multiline is the target bug that needs to be fixed, which may or may not be done with a one line per block implementation.

[micimize]

I don't think there's a solution for consistent multi-line highlighting that doesn't run into the same problem / trade-off, just like you get with lists. We could make the code-block merging configurable, or make such merging strategies configurable in general.

One thing I tried was making code-block a wrapper around code-lines, but that was a bit painful, and was really hard to fit to the current highlighting strategy.

[jhchen]

I'm not ready to give up on a solution yet and force all Quill users to make a configured tradeoff. The solution may not be apparent yet but that does not mean it does not exist.

[auralon]

This is exactly the issue that I came across, although I noticed it with the blockquote tag too, which certainly doesn't have the same knock-on effects (when used in conjunction with other tools like highlight.js), but if behaviour is changed for pre tags, then it might as well be altered for blockquote too.

[auralon]

Looking forward to this being released as a beta build.