Qt
  1. Qt
  2. QTBUG-49955

QWindowsInputContext doesn't handle input method correctly with Japanese IME.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: P2: Important P2: Important
    • Resolution: Done
    • Affects Version/s: 5.6.0 Alpha
    • Fix Version/s: 5.6.0 RC
    • Labels:
      None
    • Environment:

      Windows

    • Commits:
      c09412fcea92864c4240bced258a72ab5f73870d

      Description

      QWindowsInputContext doesn't handle input method correctly with Japanese IME HIRAGANA mode especially input in the process of KANJI conversioning.
      It makes buggy pre-edit string.

      Please show attatched files.

      It is reproduced on most versions of Windows and Qt.
      I test on the 'Google Earth', 'Virtual Box', 'SQL Browser' and 'Qt Creator' with Windows10 and more.

      Steps to reproduce:
      1. In the 'Region and Language' select 'Change keyboards..' Add Japanese Microsoft IME Keyboard. (or run on Japanese edition of Windows)
      2. Run Qt Creator and open file as edit mode.
      3. From the Language Bar change the input language to Japanese. Also change the Input mode to Hiragana (it has the pre-edit state).
      4. Focus on editor and press 'a' key three times. (inputed "あああ"in Japanese)
      5. Press Space key once.
      6. Press 'a' key once. (inputed new "あ")
      7. Press BackSpace key three times.

      a) I expect a result string "あ" but it shows "あ あ".
      b) Cannot delete the last "あ" with the Delete or BackSpace key.
      c) The text cursor looks as if it is anchored behind the last "あ".(Only Qt Creator)

      1. GoogleEarth.gif
        39 kB
      2. MySample.gif
        4 kB
      3. QtCreator.gif
        65 kB
      No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

        Hide
        YAMAMOTO Atsushi added a comment -
        Show
        YAMAMOTO Atsushi added a comment - I pushed the patch: https://codereview.qt-project.org/#/c/143845/
        Hide
        Liang Qi added a comment -

        YAMAMOTO Atsushi, thanks for the work. Patch Set 1 did fix the issue about using backspace to delete the preedit strings.

        I could see there is another bug here also, after step 5(Press Space key once), we got "あああ", but only the first two are selected, neither selected all nor selected nothing, is there any possibility to fix that?

        Show
        Liang Qi added a comment - YAMAMOTO Atsushi , thanks for the work. Patch Set 1 did fix the issue about using backspace to delete the preedit strings. I could see there is another bug here also, after step 5(Press Space key once), we got "あああ", but only the first two are selected, neither selected all nor selected nothing, is there any possibility to fix that?
        Hide
        Liang Qi added a comment -

        BTW, on Windows 10, the pinyin and wubi IME from MS don't have pre-edit state mode, can't test there by now. I could have a try with win 7 later today.

        Show
        Liang Qi added a comment - BTW, on Windows 10, the pinyin and wubi IME from MS don't have pre-edit state mode, can't test there by now. I could have a try with win 7 later today.
        Hide
        Liang Qi added a comment -

        msvc2010 2013 doesn't work well on my win7. I tried mingw32-492 with 5.6 branch, it works fine to fix the issue.
        And this time, after step 5, I didn't reproduce the issue like on my win10, so I guess there is some difference cross different versions of windows. Then no need to work on that now.

        Show
        Liang Qi added a comment - msvc2010 2013 doesn't work well on my win7. I tried mingw32-492 with 5.6 branch, it works fine to fix the issue. And this time, after step 5, I didn't reproduce the issue like on my win10, so I guess there is some difference cross different versions of windows. Then no need to work on that now.

          People

          • Assignee:
            YAMAMOTO Atsushi
            Reporter:
            YAMAMOTO Atsushi
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes