New issue
Advanced search Search tips
Starred by 1 user
Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment
Not possible to remove the 'li' element inside the table cell
Project Member Reported by jfernan...@igalia.com, Jun 9 2017 Back to list
Chrome Version: (60.0.3095.5)
OS: (Linux)

What steps will reproduce the problem?
(1) Load the attached test case
(2) Place the cursor at any of the li items
(3) hit backspace

What is the expected result?

Both list items can be deleted.

What happens instead?

None of these list items can be deleted by hitting backspace.

 
backspaceOutOfLIInCell.html
362 bytes View Download
This looks like a clear interoperability issue, as both Firefox and IE/Edge allow deleting the list item element inside the table cell.
Components: Blink>Editing>Content
Comment 3 by yosin@chromium.org, Jun 12 2017
Cc: -yosin@chromium.org
Components: -Blink>Editing>Content Blink>Editing>Command
Owner: jfernan...@igalia.com
Status: Started
In review http://crrev.com/c/530224
Project Member Comment 4 by bugdroid1@chromium.org, Jun 12 2017
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cdc284f986daab444f0a96557c9a59e49457c715

commit cdc284f986daab444f0a96557c9a59e49457c715
Author: Javier Fernandez <jfernandez@igalia.com>
Date: Mon Jun 12 16:18:38 2017

Break out list items when they are at the start of a editable table cell

We need to add a new case for breaking out empy list items when they are
at the start of an editable area. Since list items can be also inside
table cells, we need to consider this kind of elements as well.

Bug:  731621 
Change-Id: If2cfc2bd926dc30004e92b18712d8167db7241fe
Reviewed-on: https://chromium-review.googlesource.com/530224
Commit-Queue: Javier Fernandez <jfernandez@igalia.com>
Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#478648}
[add] https://crrev.com/cdc284f986daab444f0a96557c9a59e49457c715/third_party/WebKit/LayoutTests/editing/deleting/delete-list-items-in-table-cell.html
[add] https://crrev.com/cdc284f986daab444f0a96557c9a59e49457c715/third_party/WebKit/LayoutTests/external/wpt/editing/data/delete-list-items-in-table-cells.js
[add] https://crrev.com/cdc284f986daab444f0a96557c9a59e49457c715/third_party/WebKit/LayoutTests/external/wpt/editing/run/delete-list-items-in-table-cell.html
[modify] https://crrev.com/cdc284f986daab444f0a96557c9a59e49457c715/third_party/WebKit/Source/core/editing/commands/TypingCommand.cpp

Status: Fixed
This issue should be FIXED now.
Sign in to add a comment