facebook
Feature: Lists should be toggleable
Description
If you dispatch a list command like INSERT_UNORDERED_LIST_COMMAND it will not toggle a list back to normal. This is different from formatting a text with bold, italic etc.
So I want to recreate the keyboard shortcuts from word. Using Ctrl + Shift + L can toggle a list. This should be the default behavior as users a used to it.
It could also be solved by adding a seperate toggle command.
I think text alignment it is the same. Also it is the same for headlines (the dropdown in the playground). If you select the same style again, it should be reverted.
You could certainly do this on your own by registering the command at a higher priority, detecting if the selection is on a list, and then removing the list. Since lists can be nested I think either approach is probably valid, and it would be harder to override the toggle logic if that was the default.
One suggestion was just adding a toggle command so its there out of the box.
I cant create a nested list inside the playground and also not in word using the keyboard shortcut so I think having the same behaviour would be the best.
Headlins cannot be nested at all so that should definitely toggle the mode.
Looks like you're right that the command does not allow you to create nested lists, but the implementation does allow it and that's how multiple levels work. For example if you look at this document you can see several list nodes nested in listitem nodes
Yes I dont say the are wrong or should not be possbile but using a keyboard short cut or the command should toggle the current list I guess
I've investigated this issue and can confirm that list toggling is working as expected. The keyboard shortcut Cmd + Option + 5 (or Ctrl + Alt + 5 on Windows/Linux) correctly toggles numbered lists. This is implemented in the codebase and verified by test cases.
The functionality is properly implemented in:
- The keyboard shortcuts are defined in
ShortcutsPlugin/shortcuts.ts - The list toggling is handled in
ShortcutsPlugin/index.tsx - There are test cases in
List.spec.mjsthat verify the toggling behavior
The issue appears to be resolved as the current implementation does support toggling lists back to their original state.
Please let me know if I am missing anything?
