Bug 24249 – execCommand("outdent") on a list containing a table duplicates the content

• Component: HTML Editing
• Status: NEW

[Eric Seidel (no email)]

execCommand("outdent") on a list containing a table duplicates the content

1. See attached test case.
2.  Weep, softly.

[Eric Seidel (no email)]

Created attachment 28102 [details]
test case

[Eric Seidel (no email)]

Still reproduces in Safari 4 (I filed this bug from Safari 3, eek!)

[Ahmad Saleem]

Fixed in Blink commit - https://chromium.googlesource.com/chromium/blink/+/0cd4b3864db4fc9c0abfeeee0775c5fcbbca0184

[Radar WebKit Bug Importer]

<rdar://problem/129093516>

[Ahmad Saleem]

This compiles and make us match Chrome Canary 127 and Firefox Nightly 128:

// We should clone all the children of the list item for indenting purposes. However, in case the current
    // selection does not encompass all its children, we need to explicitally handle the same. The original
    // list item too would require proper deletion in that case.
    if (end.anchorNode() == selectedListItem.get() || end.anchorNode()->isDescendantOf(selectedListItem->lastChild()))
        moveParagraphWithClones(start, end, newList.get(), selectedListItem.get());
    else {
        moveParagraphWithClones(start, positionAfterNode(selectedListItem->lastChild()), newList.get(), selectedListItem.get());
        removeNode(*selectedListItem);
    }

Here - https://searchfox.org/wubkat/rev/427a310bda2e15e3014e59ff411c56c9da485c4c/Source/WebCore/editing/IndentOutdentCommand.cpp#84

[Ahmad Saleem]

Pull request: https://github.com/WebKit/WebKit/pull/30111