Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(812)

Side by Side Diff: LayoutTests/editing/deleting/merge-lists.html

Issue 112633003: Handle special case of merging lists in mergeParagraphs() (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Attempt to address the review comment Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | LayoutTests/editing/deleting/merge-lists-expected.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 <!DOCTYPE>
2 <html>
3 <body>
4 <p id="description">Placing cursor at the end of first list and executing forwar d delete should merge the second list with the first one.</p>
5 <div contenteditable>
6 <ol><li>one</li><li>two</li><li id="li">three</li></ol>
7 <ol><li>four</li><li>five</li><li>six</li></ol>
8 </div>
9 <script src="../../resources/dump-as-markup.js"></script>
10 <script>
11 var li = document.getElementById("li");
12 var selection = window.getSelection();
13 selection.collapse(li, li.childNodes[0].length);
14 document.execCommand("forwardDelete");
15 Markup.description(document.getElementById('description').textContent);
16 Markup.dump(document.querySelector('div'));
17 </script>
18 </body>
19 </html>
20
OLDNEW
« no previous file with comments | « no previous file | LayoutTests/editing/deleting/merge-lists-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698