Skip to content

input-method module: use xim for GTK Fcitx#15254

Closed
sorpaas wants to merge 1 commit intoNixOS:masterfrom
sorpaas:master
Closed

input-method module: use xim for GTK Fcitx#15254
sorpaas wants to merge 1 commit intoNixOS:masterfrom
sorpaas:master

Conversation

@sorpaas
Copy link
Member

@sorpaas sorpaas commented May 6, 2016

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • OS X
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

Fcitx has a xim interface, which works better for GTK applications. On my KDE desktop, the fcitx IM module doesn't work for GTK applications. Only xim works.

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @ericsagnes and @ttuegel to be potential reviewers

@ericsagnes
Copy link
Contributor

This is related to GTK_IM_MODULE cache problems. (see #14019)
#14568 should fix this problem and is close to merge.

IMHO we shouldn't set xim as default as it has issues.
For the record, Fcitx main developer explained the details of xim problems in this comment :

... XIM is extremely outdated (90s), buggy (synchronous call everywhere, can easily freeze app if input method server quits), unmaintained (many TODOs in source which is never implemented for years), and lacks of important features comparing with what an native im module can provides, e.g. surrounding text support which is commonly used in Japanese input method.

@sorpaas
Copy link
Member Author

sorpaas commented May 6, 2016

Thanks! Closing this pull request.

@sorpaas sorpaas closed this May 6, 2016
@Janik-Haag Janik-Haag added the 12.first-time contribution This PR is the author's first one; please be gentle! label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

12.first-time contribution This PR is the author's first one; please be gentle!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants