Code collapsing issue - db<>fiddle

TopAnswers db<>fiddle

Top Answer

Jack Douglas

Updating CodeMirror to the latest version has fixed this issue.

I [raised it](https://github.com/codemirror/dev/issues/1008) on the CodeMirror issue tracker and the maintainer pointed out it was working fine in their sandbox - next time I will try that first!

comments

transcript

Jack Douglas

I could fix it by simply dropping the use of the MySQL and MariaSQL language modules but there would be other downsides to that

Jack Douglas replying to Akina

Thanks, I've raised this as an issue on the CodeMirror bug tracker: https://github.com/codemirror/dev/issues/1008

Akina replying to Jack Douglas

Yes, your fiddle works as expected. The problem is observed only for MySQL and MariaDB. While testing do not forget to add a space after double dash - it is compulsory for these DBMSs. https://dbfiddle.uk/Xuhq-FM0

Jack Douglas replying to Akina

can you add a db<>fiddle link to your post please, this actually works as expected for me: https://dbfiddle.uk/6AQKw34N

Akina replying to Jack Douglas

I think that code collapsing must be stored - it is very useful, the queries may be long enough. Also I think that you're right, and a comment should be treated as separate codeblock/statement. Moreover, I do not see the reason for the comment MUST be collapseable (but it may, of course). I mean separate inline comments (those comments which starts from the beginning of line and are marked with starting double dash). In SQL code the comments are short in most cases. Of course, multiline comments must be collapseable, and they should be treated as separate statements too, and inline comments in the middle of the query should be treated as a part of this query. IMHO.

Jack Douglas

if not, what would you prefer, treat the comment as a separate block? Show the first two lines? I don't know what's possible with CodeMirror, but it isn't clear to me what the ideal behaviour would be

Jack Douglas

@Akina,  is the code collapsing worth having at all? It would be easy to remove it altogether.

1 Answer