Markdown Lists - Task List, Tab behavior

[JYone3A]

Describe the feature

It would be nice if task lists were supported (https://github.github.com/gfm/#task-list-items-extension-), i.e. when pressing Enter while in a task list, the next line should start with * [ ] .

Another nice behavior would be if I press Tab at the beginning of an empty list item (i.e. with the caret right after * or 3. or * [ ] ), the whole line should be indented, including the list item itself. This should probably only happen if the previous line also starts with a list item definition.

Use Case

Quicker creation of task lists and sub-lists.

Proposed Solution

https://github.com/ajaxorg/ace/blob/master/src/mode/markdown.js#L33

The match regular expression could look like this:

var match = /^(\s*)(?:([-+*])|(\d+)\.)(\s+)(\[(?:x| )\]\s+)?/.exec(line);

// ...

return match[1] + marker + match[4] + (match[5] || '');

Not sure if there should always be an empty [ ] inserted, the code above would keep the checkbox state of the previous item.

---

No idea how to implement the Tab behavior. I don't see anything in the ace indent function that would allow customizing this from within the markdown mode?

Anyway, it should take the above regular expression into account, check if the current and the previous line start with this regexp, and if the current caret is positioned right after that. If yes, the whole line should be indented.

Other Information

No response

Acknowledgements

• I may be able to implement this feature request
• This feature might incur a breaking change

ACE version used

1.14.0

[JYone3A]

I did kind of hack the Tab indentation into ACE at runtime, by overwriting the editor.indent function. I know this is wrong (and ignores the mode, so that's no generic solution), but maybe the code helps someone who also wants this right now.

const session  = editor.getSession();
const matchMdList = /^\s*(-|\*) (\[( |x)\] )?$/;

editor.oldIndent = editor.indent;

editor.indent = function() {
    var range = this.getSelectionRange();
    
    if (range.start.row !== range.end.row || range.start.column !== range.end.column || range.start.column === 0) {
        return this.oldIndent();
    }
    
    range.start.column = 0;
    
    if (!matchMdList.test(session.getTextRange(range))) {
        return this.oldIndent();
    }
    
    var rows = this.$getSelectedRows(); // not sure if I could also use range.start.row, but this is how the original indent does it
    session.indentRows(rows.first, rows.last, "\t");
}

[github-actions]

This issue has not received any attention in 1 year. If you want to keep this issue open, please leave a comment below and auto-close will be canceled.

[ghost]

Unsubscribe Remove Get Outlook for iOS<https://aka.ms/o0ukef>

…

________________________________ From: github-actions[bot] ***@***.***> Sent: Wednesday, March 20, 2024 12:04:42 PM To: ajaxorg/ace ***@***.***> Cc: Subscribed ***@***.***> Subject: Re: [ajaxorg/ace] Markdown Lists - Task List, Tab behavior (Issue #5056) This issue has not received any attention in 1 year. If you want to keep this issue open, please leave a comment below and auto-close will be canceled. — Reply to this email directly, view it on GitHub<#5056 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AFQLBHJLYBMAWAEBWYAWA2LYZGXRVAVCNFSM6AAAAAAUXYHGHWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBZHEZTKMBQGM>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>