Get rid of the wrapper() method

[hanspagel]

I’d like to get rid of the wrapper() method. It’s used - for example - to move the content of list items in a paragraph.

I’m afraid we’d need to add broad support for the content attribute from ProseMirror to achieve that though.

[Coding-Kiwi]

I'm not sure but I think the wrapper causes some issues for me. I'm currently trying to migrate from html to tiptap doc and when I try to import html with nested lists stuff breaks.

Given a nested list:

<ul>
  <li>
    <p>This is a test</p>
    <ul>
     <li>Subitem</li>
   </ul>
  </li>
</ul>

The wrapper adds a paragraph because the childnodelength is no longer 1:

<ul>
  <li>
    <p> <-- added
    <p>This is a test</p>
    <ul>
      <li>Subitem</li>
    </ul>
    </p> <-- added
  </li>
</ul>

When I convert this to a doc json and load it into the frontend tiptap it works but as soon as I edit the list, all subitems are merged into a single paragraph and I get Invalid content for node paragraph errors

I disabled the wrapper and now I have no problems

class CustomExtensionextends Extension {
    public static $name = 'custom';

    public function addExtensions() {
        return [
            new CustomListItem (),
        ];
    }
}

class CustomListItem extends ListItem {
    public static function wrapper($DOMNode) {
        return null;
    }
}

new Editor([
            "extensions" => [
                new CustomExtension(),
                new StarterKit([
                    "listItem" => false
                ])
            ]
]);