Bug 248709 - Handle special case of merging lists in mergeParagraphs()
Summary: Handle special case of merging lists in mergeParagraphs()
Status: REOPENED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on: 250700
Blocks:
  Show dependency tree / graph
 
Reported: 2022-12-03 06:50 PST by Ahmad Saleem
Modified: 2023-01-17 00:36 PST (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2022-12-03 06:50:18 PST 
Hi Team,

While going through Blink's commit, came across this one:

Blink Commit [1] - https://src.chromium.org/viewvc/blink?revision=165218&view=revision

and in above, Safari is different from compared to Firefox and Chrome, which match each other:

JSFiddle - https://jsfiddle.net/1ofserwL/

Although above commit will introduce regression, which is fixed in this:

Blink Commit [2] - https://src.chromium.org/viewvc/blink?view=revision&revision=172941

Just raising to merge both and align with other browsers in behavior.

Thanks!
Comment 1 EWS 2022-12-09 15:46:08 PST 
Committed 257650@main (31be36f212e4): <https://commits.webkit.org/257650@main>

Reviewed commits have been landed. Closing PR #7349 and removing active labels.
Comment 2 Radar WebKit Bug Importer 2022-12-09 15:47:30 PST 
<rdar://problem/103197455>
Comment 3 WebKit Commit Bot 2023-01-17 00:27:14 PST 
Re-opened since this is blocked by bug 250700
Comment 4 Ryosuke Niwa 2023-01-17 00:36:15 PST 
The patch got rolled out because it caused
Comment 5 Ryosuke Niwa 2023-01-17 00:36:38 PST 
(In reply to Ryosuke Niwa from comment #4)
> The patch got rolled out because it caused

regressions in WPT. See the bug 250561.