24249 2009-02-27 13:47:54 -0800 execCommand("outdent") on a list containing a table duplicates the content 2025-03-09 07:39:02 -0700 1 1 1 Unclassified WebKit HTML Editing 528+ (Nightly build) All All NEW BrowserCompat, InRadar P2 Normal --- 1 eric ahmad.saleem792 ahmad.saleem792 jparent justin.garcia ojan rniwa webkit-bug-importer oldest_to_newest 111541 0 eric 2009-02-27 13:47:54 -0800 execCommand("outdent") on a list containing a table duplicates the content 1. See attached test case. 2. Weep, softly. 111542 1 28102 eric 2009-02-27 13:48:11 -0800 Created attachment 28102 test case 111557 2 eric 2009-02-27 15:01:45 -0800 Still reproduces in Safari 4 (I filed this bug from Safari 3, eek!) 2039212 3 ahmad.saleem792 2024-05-31 19:20:13 -0700 Fixed in Blink commit - https://chromium.googlesource.com/chromium/blink/+/0cd4b3864db4fc9c0abfeeee0775c5fcbbca0184 2039213 4 webkit-bug-importer 2024-05-31 19:20:23 -0700 <rdar://problem/129093516> 2039218 5 ahmad.saleem792 2024-05-31 20:07:49 -0700 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 2101746 6 ahmad.saleem792 2025-03-09 07:39:02 -0700 Pull request: https://github.com/WebKit/WebKit/pull/30111 28102 2009-02-27 13:48:11 -0800 2009-02-27 13:48:11 -0800 test case outdent.html text/html 497 eric PGRpdj5leGVjQ29tbWFuZCgib3V0ZGVudCIpIG9uIGEgbGlzdCBpdGVtIGNvbnRhaW5pbmcgYSB0 YWJsZSBkdXBsaWNhdGVzIHRoZSBsaXN0IGl0ZW0gY29udGVudCwgYnV0IGRvZXMgbm90IGNvcnJl Y3RseSBvdXRkZW50IHRoZSBsaXN0ITwvZGl2Pgo8ZGl2PllvdSBzaG91bGQgb25seSBzZWUgImEi IGZvbGxvd2VkIGJ5IGEgc2luZ2xlIGNlbGwgdGFibGUgYmVsb3c8L2Rpdj4KCjxkaXYgY29udGVu dEVkaXRhYmxlPjxvbD48bGkgaWQ9Imxpc3RpdGVtIj5hPHRhYmxlIGJvcmRlcj0iMSIgY2VsbHBh ZGRpbmc9IjIiIGNlbGxzcGFjaW5nPSIyIj48dGJvZHk+PHRyPjx0ZD48YnI+PC90ZD48L3RyPjwv dGJvZHk+PC90YWJsZT48L2xpPjwvb2w+PC9kaXY+CjxzY3JpcHQ+CndpbmRvdy5nZXRTZWxlY3Rp b24oKS5jb2xsYXBzZShkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgibGlzdGl0ZW0iKSwgMCk7CmRv Y3VtZW50LmV4ZWNDb21tYW5kKCJvdXRkZW50Iik7Cjwvc2NyaXB0Pgo=