New issue
Advanced search Search tips

Issue 744936 link

Starred by 1 user

Issue metadata

Status: Fixed
Closed: Aug 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Sign in to add a comment

InsertOrderedList command applied to table cells inserts an extra BR before the table

Project Member Reported by, Jul 17 2017

Issue description

Chrome Version:  61.0.3135.4

What steps will reproduce the problem?
(1) Load the attached test case. The contents of the table will be selected on load
(2) Hit the "List" button multiple times. 

What is the expected result?

The cell content become a list item after the first click. List items become regular text after a new click. Subsequent clicks will produce the same results.

What happens instead?

On every second click, the table will move down.

376 bytes View Download
Status: Started (was: Untriaged)

Comment 2 by, Aug 1 2017

Online version of sample:
Project Member

Comment 3 by, Aug 10 2017

The following revision refers to this bug:

commit 1e5ec55dd115f8663389231c82928066184def96
Author: Javier Fernandez <>
Date: Thu Aug 10 20:21:49 2017

Don't add BR when pruning empty blocks if it's the start of a paragraph

The function MoveParagraph is used for several operations and editing
commands, like 'insertOrderedList'. When moving paragraphs we check out
whether we should add a 'br' element to avoid undesired block collapse.

However, we shouldn't do this when destination is the start of a

Bug:  744936 
Change-Id: I366b903b5f3737bd6c84e9fc8c91a4894ca680af
Commit-Queue: Javier Fernandez <>
Reviewed-by: Yoshifumi Inoue <>
Cr-Commit-Position: refs/heads/master@{#493504}

Status: Fixed (was: Started)
This issue should be FIXED now.

Sign in to add a comment