Small issue on TODO list and return

[Shuunen]

Hi,

Here an example of the behaviour of Abricotine when hitting "Enter" at the end of a TODO line, it does not create another todo on the new line.

Then as a second issue, when selecting multiple lines and toggling TODO style, it mess up with the star character.

[brrd]

1st issue: we need a custom rewrite of CodeMirror's continuelist.js plugin that would support todolists.

2nd issue is probably a bug somewhere in Abricotine's cm-extend-markdown.js

[brrd]

From @jacobmischka (#161)

Interestingly, this works out of the box if you use - for bullets intead of *.

I didn't notice this. It's actually already supported in CodeMirror: https://github.com/codemirror/CodeMirror/blob/master/addon/edit/continuelist.js#L14-L15

Actually the * [ ] syntax is not clearly mantionned in Github flavored markdown even if Github supports it. I think we should use the recommended - [ ] syntax in Abricotine. And by the way I'll PR the support for * [ ] in CodeMirror.

[jacobmischka]

I believe CodeMirror actually does support it, unless I'm terrible at reading regexes. Both listRE and emptyListRE seem to include the boxes with *|-. I couldn't figure out why the * didn't work, though.

Edit: Whoops I didn't see that you linked to that same place in your comment too. Can you see why * isn't working by looking at those regexes?

Edit: Oh, there aren't any parens around the *|- in either of them. Is that a bug?

Final edit: Oh lol, no that * is a modifier, not a character, nevermind. Man I hate regexes.

[brrd]

@jacobmischka This may answer your question: codemirror/codemirror5@9c9a4cb 😉

[jacobmischka]

Ha yeah, the coincidental *|- confused me. I just woke up let's blame that 😪