On Mon, 24 Mar 2025 20:47:17 GMT, Ziad El Midaoui <zelmida...@openjdk.org>
wrote:
>> Andy Goryachev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> arabic
>
> The tests run successfully with the fix however there are a few things that
> do not work as expected I guess :
> 1- Pasting text from notes app (MacOS) that uses numbered list in some cases
> shows 0. instead of the item number example:
> - in note app do something like this
>
> Test
> 1. one
> 2. two
> 3. three
> Test
>
> - Copy and paste it is rendred like this
>
> Test
> 0. one
> 0. two
> 0. three
> Test
>
> 2- Tested with text written in outlook and pasting it and it do not apply
> formatting neither for RTL or LTR text
> 3- another thing the arabic diacritics text is not rendered correctly (this
> may not be related to this issue specifically) you can try with this text
> السَّلَامُ عَلَيْكُمْ
@Ziad-Mid :
interesting, thank you!
1. The list formatting was not explicitly supported, so in a sense it works as
designed (all numbered items are represented by `{\listtext 0. }`.
This will likely be a separate enhancement.
2. RTL/LTR attributes are not yet supported by the demo due to many bugs in RTL.
3. Arabic diacritics rendered with the System font look bad, I agree. Maybe
@prrace can weigh on it, looks like changing the font to one of the Arabic ones
does improve the rendering, although the result appears not as good compared to
Chrome:
Can you try setting different fonts?
السَّلَامُ عَلَيْكُمْ =
\u0627\u0644\u0633\u0651\u064e\u0644\u064e\u0627\u0645\u064f
\u0639\u064e\u0644\u064e\u064a\u0652\u0643\u064f\u0645\u0652
-------------
PR Comment: https://git.openjdk.org/jfx/pull/1735#issuecomment-2749400073
