The latest updates on your projects. Learn more about Vercel for Git ↗︎

size-limit report 📦

Were you testing this with #6917 also applied? There's a fix in there that I believe is related to this issue

https://github.com/facebook/lexical/pull/6917/files#diff-ebf55b0878052c62d91299c29e05b3f3ef9026a05bc7881a7f0d7bb37ed2d0c9L514-R520

As far as rationale goes, the ListItemNode is basically supposed to support everything that a ParagraphNode does. Unlike a TableCellNode, it is not a shadow root that is expected to have children that are ParagraphNode, so to support all of the things that ParagraphNode supports then it should either be a ParagraphNode or separately implement all of the things that ParagraphNode implements, but the problem is that you can't just copy and paste that because the extensibility is not present in selections and the reconciler to make that work, there are $isParagraphNode special cases that can't be extended outside of the lexical package.

If we really don't want nodes like this to subclass ParagraphNode then we should move the features that ParagraphNode has into some new common base class, e.g. LexicalNode -> ElementNode -> BlockNode -> (ParagraphNode | ListItemNode) so that this sort of behavior can be inherited and baked into the lower levels without introducing dependencies from lexical to @lexical/list or elsewhere.

Were you testing this with #6917 also applied? There's a fix in there that I believe is related to this issue

https://github.com/facebook/lexical/pull/6917/files#diff-ebf55b0878052c62d91299c29e05b3f3ef9026a05bc7881a7f0d7bb37ed2d0c9L514-R520

thanks for pointing out #6917. the sync diff didnt contain that fix. in that case ill put this revert on hold and test with that patch

i tested with PR 6917 applied but it did not fix the issue sadly

ill proceed with this revert first to unblock the diff syncs

If we really don't want nodes like this to subclass ParagraphNode then we should move the features that ParagraphNode has into some new common base class, e.g. LexicalNode -> ElementNode -> BlockNode -> (ParagraphNode | ListItemNode) so that this sort of behavior can be inherited and baked into the lower levels without introducing dependencies from lexical to @lexical/list or elsewhere.

this might work better to prevent breaking changes from $isParagraphNode(...)