[Differential] [Changed Subscribers] D1638: Integrate QtVirtualKeyboard into KWin/Wayland

[xuetianweng (Xuetian Weng)]

xuetianweng added inline comments.

INLINE COMMENTS

> main_wayland.cpp:449
>      qunsetenv("QT_DEVICE_PIXEL_RATIO");
> -    qunsetenv("QT_IM_MODULE");
> +    qputenv("QT_IM_MODULE", "qtvirtualkeyboard");
>      qputenv("QSG_RENDER_LOOP", "basic");

Don't do this. Otherwise qtvirtualkeyboard will be the only usable input method for Qt.

IMHO for wayland there should be  a wayland specific im module that talk to input method via wl_text.

And X11 app should use a different QT_IM_MODULE.

FYI: https://fcitx-im.org/wiki/Input_method_related_environment_variables

REPOSITORY
  rKWIN KWin

BRANCH
  virtual-keyboard

REVISION DETAIL
  https://phabricator.kde.org/D1638

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, Plasma, sebas
Cc: xuetianweng, sebas, mart, broulik, plasma-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160526/c8ebbaa5/attachment.html>