On Wed, 7 Feb 2024 10:43:12 GMT, Karthik P K <k...@openjdk.org> wrote:

>> In the `getHitInfo()` method of PrismTextLayout, RTL node orientation 
>> conditions were not considered, hence hit test values such as character 
>> index and insertion index values were incorrect.
>> 
>> Added checks for RTL orientation of nodes and  fixed the issue in 
>> `getHitInfo()` to calculate correct hit test values.
>> 
>> Added system tests to validate the changes.
>
> Karthik P K has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Fix emoji issue

The issue with emojis and tabs are fixed.
In Monkey tester with RichText option selected, I see issue in the last line 
shown in the screenshot below.
![Screenshot 2024-02-07 at 3 53 02 
PM](https://github.com/openjdk/jfx/assets/26969459/0a900679-ad85-4949-bdec-8e9b7b25a6a5)

Since the emoji itself is not rendered properly I don't think it is an issue 
relevant to this PR.

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

PR Comment: https://git.openjdk.org/jfx/pull/1323#issuecomment-1931769144

Reply via email to