[webkit-changes] [WebKit/WebKit] 03433c: [Writing Tools] During the rewrite animation, some...
Richard Robinson
noreply at github.com
Thu Sep 5 16:09:22 PDT 2024
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 03433cf8ea044ee49d35adf47aa8dd9df8988490
https://github.com/WebKit/WebKit/commit/03433cf8ea044ee49d35adf47aa8dd9df8988490
Author: Richard Robinson <richard_robinson2 at apple.com>
Date: 2024-09-05 (Thu, 05 Sep 2024)
Changed paths:
M Source/WebCore/page/writing-tools/WritingToolsController.mm
M Tools/TestWebKitAPI/Tests/WebKitCocoa/WritingTools.mm
Log Message:
-----------
[Writing Tools] During the rewrite animation, some text temporarily is hidden and the animation looks broken
https://bugs.webkit.org/show_bug.cgi?id=279231
rdar://134991317
Reviewed by Aditya Keerthi.
Effectively revert 282750 at main, which ended up inadvertently adding transparent document markers to incorrect ranges.
Note that the bug that 282750 at main fixed still remains fixed, since 283109 at main inadvertently also fixes it by clearing
document markers after the animation finishes.
* Source/WebCore/page/writing-tools/WritingToolsController.mm:
(WebCore::WritingToolsController::compositionSessionDidReceiveTextWithReplacementRangeAsync):
* Tools/TestWebKitAPI/Tests/WebKitCocoa/WritingTools.mm:
(TEST(WritingTools, CompositionShowOriginalHasNoTransparentMarkers)):
Canonical link: https://commits.webkit.org/283238@main
To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications
More information about the webkit-changes
mailing list