[WiP] Skip level lists#20088
Conversation
…ixes todo and multi-level lists).
…d/remove some features.
…t-skip-level-lists
…ls-in-multi-level-lists
…odel Allow skip level lists in the model (commands and postfixers).
…ists Downcast skip-level lists to valid HTML
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
…ls-in-multi-level-lists
Upcast skip level lists.
…level-lists Fix skip levels in multi level and todo lists.
…ed-lists Extend autoformat for lists to accept any number.
…nto cc/9765-changing-list-start-after-skip-level-not-visible
…hould merge to previous list item.
…fter-skip-level-not-visible Improve detecting which items should be reconverted when using skip-level lists.
…r-skip-level Tab before list item at indent 0 after a skip-level should indent list item.
…m-after-skip-level-error Backspace before a nested list item should not fail when there is no previous item to merge with (skip level list).
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 5160363. Configure here.
| { additionalAttributes: { listStart: parseInt( match[ 1 ] ) } } : | ||
| undefined | ||
| ); | ||
| } ); |
There was a problem hiding this comment.
Undo-on-backspace triggered when autoformat callback declines
Low Severity
When the numbered-list autoformat callback returns false (because the selection is already inside a numbered list), blockAutoformatEditing still calls requestUndoOnBackspace() unconditionally. Previously, the command-string approach exited before enqueueChange when command.value === true, avoiding this side effect. Now, typing something like "5. " inside an existing numbered list item correctly preserves the text, but the next backspace press may trigger an undo of the entire recent typing batch instead of just deleting the space character.
Reviewed by Cursor Bugbot for commit 5160363. Configure here.
…el-lists Update metadata for skip-level lists.
Add skip level list changelogs.
…-level-lists-with-ghs Improve detecting skip-level lists.
2 participants
🚀 Summary
A brief summary of what this PR changes.
📌 Related issues
💡 Additional information
Optional: Notes on decisions, edge cases, or anything helpful for reviewers.
🧾 Checklists
Use the following checklists to ensure important areas were not overlooked.
This does not apply to feature-branch merges.
If an item is not relevant to this type of change, simply leave it unchecked.
Author checklist
Reviewer checklist
t()(if any).