WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
Bug 24249
execCommand("outdent") on a list containing a table duplicates the content
https://bugs.webkit.org/show_bug.cgi?id=24249
Summary
execCommand("outdent") on a list containing a table duplicates the content
Eric Seidel (no email)
Reported
2009-02-27 13:47:54 PST
execCommand("outdent") on a list containing a table duplicates the content 1. See attached test case. 2. Weep, softly.
Attachments
test case
(497 bytes, text/html)
2009-02-27 13:48 PST
,
Eric Seidel (no email)
no flags
Details
View All
Add attachment
proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1
2009-02-27 13:48:11 PST
Created
attachment 28102
[details]
test case
Eric Seidel (no email)
Comment 2
2009-02-27 15:01:45 PST
Still reproduces in Safari 4 (I filed this bug from Safari 3, eek!)
Ahmad Saleem
Comment 3
2024-05-31 19:20:13 PDT
Fixed in Blink commit -
https://chromium.googlesource.com/chromium/blink/+/0cd4b3864db4fc9c0abfeeee0775c5fcbbca0184
Radar WebKit Bug Importer
Comment 4
2024-05-31 19:20:23 PDT
<
rdar://problem/129093516
>
Ahmad Saleem
Comment 5
2024-05-31 20:07:49 PDT
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
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug