On Sun, 15 Dec 2024 22:11:30 GMT, Thiago Milczarek Sayao <tsa...@openjdk.org> 
wrote:

>> This replaces obsolete XIM and uses gtk api for IME.
>> Gtk uses [ibus](https://github.com/ibus/ibus)
>> 
>> Gtk3+ uses relative positioning (as Wayland does), so I've added a Relative 
>> positioning on `InputMethodRequest`.
>> 
>> [Screencast from 17-09-2023 
>> 21:59:04.webm](https://github.com/openjdk/jfx/assets/30704286/6c398e39-55a3-4420-86a2-beff07b549d3)
>
> Thiago Milczarek Sayao has updated the pull request incrementally with three 
> additional commits since the last revision:
> 
>  - Better solution for fcitx
>  - Revert "Fixes for fcitx"
>    
>    This reverts commit d8b38ea8f1379f9783d9186e462b0652ae7fd98b.
>  - Revert "Fixes for fcitx"
>    
>    This reverts commit 6cd1d32a51149df2cbe36f0b348a464173422201.

Holiday season went in the way of wrapping this up last year, but I finally got 
a chance to come back to it and finish the review.

Code-review-wise looks good. I also run the tests on my Ubuntu 24.04 and did 
not notice any abnormalities.

@kevinrushforth also wanted to take a look at this after me.

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

Marked as reviewed by lkostyra (Reviewer).

PR Review: https://git.openjdk.org/jfx/pull/1080#pullrequestreview-2555932036

Reply via email to