[webkit-changes] [WebKit/WebKit] 41ea64: Add a nbsp at the the end of the text when the nex...
Ahmad Saleem
noreply at github.com
Fri Dec 9 06:40:17 PST 2022
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 41ea6427c93797b6c7aceb36d05329618ddd87b9
https://github.com/WebKit/WebKit/commit/41ea6427c93797b6c7aceb36d05329618ddd87b9
Author: Ahmad Saleem <ahmad.saleem792+github at gmail.com>
Date: 2022-12-09 (Fri, 09 Dec 2022)
Changed paths:
A LayoutTests/editing/inserting/insert-space-expected.txt
A LayoutTests/editing/inserting/insert-space.html
A LayoutTests/resources/assert-selection.js
M Source/WebCore/editing/CompositeEditCommand.cpp
Log Message:
-----------
Add a nbsp at the the end of the text when the next text has a leading space
Add a nbsp at the the end of the text when the next text has a leading space
https://bugs.webkit.org/show_bug.cgi?id=248723
Reviewed by Ryosuke Niwa.
This patch is to fix three regressions stemming from 257136 at main.
1) Breaking chat window or login forms while trying to press 'space' key.
2) Add nbsp instead of plain space between text nodes in case next text node has a leading plain space.
3) Not accounting for all kind of whitespaces including "Enter"
These are merge of following Blink commits:
> https://chromium.googlesource.com/chromium/src.git/+/4dab74137593abb0888e415294aeb80da27362e3
> https://chromium.googlesource.com/chromium/src.git/+/9ebff223d5d639167c3efb01a5fa912b3aee05a9
> https://chromium.googlesource.com/chromium/src.git/+/5b9a9a6f8c9c646ac542a9e1a2f18c59140849e6
* Source/WebCore/editing/CompositeEditingCommand.cpp:
(CompositeEditCommand::rebalanceWhitespaceOnTextSubtring):
(1) Add logic to account for leading space (including "Enter" key)
(2) accept "Space" Key
(3) Refactor code by introducing const bool "nextSiblingIsTextNodeWithoutLeadingSpace"
* LayoutTests/resources/assert-selection.js: Added Test Script with Chromium Copyright header
* LayoutTests/editing/inserting/insert-space.html: Added Test Case
* LayoutTests/editing/inserting/insert-space-expected.txt: Added Test Case Expectation
Canonical link: https://commits.webkit.org/257622@main
More information about the webkit-changes
mailing list