Bug 173186 - Delete second list item inside a single table cell should not delete the whole table
Summary: Delete second list item inside a single table cell should not delete the whol...
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-06-09 15:03 PDT by Javier Fernandez
Modified: 2022-11-17 11:49 PST (History)
4 users (show)

See Also:


Attachments
Test case to reproduce the issue (251 bytes, text/html)
2017-06-09 15:03 PDT, Javier Fernandez
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Javier Fernandez 2017-06-09 15:03:12 PDT
Created attachment 312495 [details]
Test case to reproduce the issue

What steps will reproduce the problem?
(1) Load the attached test case
(2) Place the cursor at the list item 2.
(3) Hit backspace to the list item is removed

What is the expected result?

The list item 2 is removed but the list still holds the item 1.

What happens instead?

The whole table is deleted.
Comment 1 Javier Fernandez 2017-06-09 15:04:58 PDT
Maybe related to bug #173117 ?
Comment 2 Javier Fernandez 2018-02-26 08:14:32 PST
The Blink bug (731913) has been already fixed, so perhaps we try a similar approach with this bug.
Comment 4 Ahmad Saleem 2022-11-16 13:39:49 PST
(In reply to Ahmad Saleem from comment #3)
> Blink Commit -
> https://chromium.googlesource.com/chromium/src.git/+/
> 5961ff06240e30a18405608f06d8eea4730c7b8f
> 
> Webkit GitHub Source -
> https://github.com/WebKit/WebKit/blob/
> 8a344c3387b2bfe58dc8c8a94b1683187effef2e/Source/WebCore/editing/
> DeleteSelectionCommand.cpp#L112
> 
> !IsListItem(position.xxx
> 
> Still haven't figured out xxx yet. Thanks!

Although before that I am not able to reproduce this bug and only 2. bullet get destroyed and 1. remains. So I think the test case attached is working fine in Safari 16.1.

Do we need anything else or we can close this? Thanks!
Comment 5 Ryosuke Niwa 2022-11-17 11:49:31 PST
No longer reproduces.