Repository navigation
iOS tests: Don't double tap next to the cursor - #85481
Merged
Merged
Conversation
The tap put the caret at the start of the word the test then double taps. A tap on the caret opens the edit menu, so the double tap sometimes selected nothing. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
🤖 PR meta 🤖🎉 PropsIf you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. Updated as activity occurs, without notifying anyone named here. Add the ⚡ PerformanceShow the resultsClient side metrics exclude the server response time. front-end-block-theme
front-end-classic-theme
media-processing
media-upload
post-editor
site-editor
🏁 Flaky testsSome tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information. Custom Completer: should allow newlines after multiple option completions in
|
ellatrix
marked this pull request as ready for review
October 23, 2026 15:49
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What?
Fixes the most common failure of the iOS test that selects text across two paragraphs: "The double tap did not select a word".
Why?
The test focuses the second paragraph with a tap that leaves the caret at the start of "Delta", then double taps "Delta" 10 points from that caret. On iOS, a tap on the caret opens the edit menu, so the first touch of the double tap is sometimes taken as a tap on the caret: the menu shows "Paste, Select, Select All" and no word gets selected.
Seen by running the test 20 times on a simulator while recording the screen: 2 iterations failed this way, with the same timing as the 18 that passed. On CI the test fails about 7% of the time, and this is about 60% of those failures.
How?
Testing Instructions
Run the iOS tests a number of times, for example locally with
./test/ios/run.sh -test-iterations 30, and check that the double tap selects "Delta" every time.Use of AI Tools
Written with Claude Code.
🤖 Generated with Claude Code