RFR: 8304831: TextFlow.hitTest.insertionIndex incorrect with surrogate pairs

[Karthik P K]

On Fri, 21 Apr 2023 16:29:46 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

> could you please check with the Monkey Tester? I could be doing something wrong.

I checked above scenario in Monkey tester and I'm also seeing same issue. But in a simple test app which I created separately, it looks to be working. I will do some more testing and see what is the exact issue.

> tests/system/src/test/java/test/robot/javafx/scene/TextFlowSurrogatePairInsertionIndexTest.java line 109:
> 
>> 107:     }
>> 108: 
>> 109:     @Test
> 
> would it be possible to add another test, where the cursor is moved across an emoji character one pixel at a time, making sure that insertionIndex monotonically increases?
> 
> as a variant, move across a text string with multiple emojis?

Sure. I will try to add these test cases.

-------------

PR Comment: https://git.openjdk.org/jfx/pull/1091#issuecomment-1522830539
PR Review Comment: https://git.openjdk.org/jfx/pull/1091#discussion_r1177383167