684487 - IME input box is always placed at bottom-left corner of the screen

Status: RESOLVED WONTFIX

(SeaMonkey :: General, defect)

Description

[KO Myung-Hun]

User Agent: Mozilla/5.0 (OS/2; Warp 4.5; rv:6.0) Gecko/20110821 Firefox/6.0 SeaMonkey/2.3
Build ID: 20110821165600

Steps to reproduce:

Input DBCS characters using IME


Actual results:

The input box of IME is always placed at the bottom-left corner of the screen.


Expected results:

The bottom-left corner of the input box of IME should be placed at the same position as the cursor(caret).

Comment 1

[KO Myung-Hun]

I suspects that WM_QUERYCONVERTPOS message processing was changed at some point.

Alex Taylor said that he could confirm this problem at SeaMonkey 2.1.x as well.

SeaMonkey 2.0.x does not have this problem.

Dave Yeo suspected that WM_QUERYCONVERTPOS parts were removed at 
    
    https://bugzilla.mozilla.org/show_bug.cgi?id=522896#c33

Comment 2

[Alex Taylor]

This has also been reported by some Japanese eCS users.  It occurs consistently on DBCS OS/2 systems.

This makes typing on DBCS systems quite awkward.

Comment 3

[KO Myung-Hun]

Attachment: Patch for IME input box misposition

Comment 4

[Philip Chee]

Hi! Do you intend to flag your patch for review? or is this just a WIP?

Comment 5

[KO Myung-Hun]

Review, please.

Comment 6

[Philip Chee]

Comment on attachment 636227 [details] [diff] [review]
Patch for IME input box misposition

Picking some OS/2 person at random. Please reassign requestee as needed.

Comment 7

[Walter Meinl]

Comment on attachment 636227 [details] [diff] [review]
Patch for IME input box misposition

Dave, you're probably the better choice for reviewing this

Comment 8

[Steve Wendt]

Setting confirmed based on comment 2.

Comment 9

[KO Myung-Hun]

Attachment: Patch against trunk for IME input box misposition [checked-in comment 10]

Comment 10

[Philip Chee]

Comment on attachment 638094 [details] [diff] [review]
Patch against trunk for IME input box misposition [checked-in comment 10]

Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/edf132b767b0

Comment 11

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Does that patch set correct caret position to the pCursorPos on XUL <panel>s? I don't know about the OS/2's IME APIs but it looks like the patch doesn't work when focused widget and caret widget are different. E.g., is it work fine on the editors in Bookmark panel?

Comment 12

[KO Myung-Hun]

I'm not sure what 'editors in Bookmark panel' is.

But, If I understand it correctly, I did the following.

  1. Bookmarks -  Show All Bookmakrs

  2. Select some bookmark

  3. Click 'More'

  4. Edit 'Description:'

This works fine.

  5. Press 'Alt' key

  6. Some menu item is selected.

  7. Input DBCS chars

  8. IME input-box is placed in editor of 'Description:'

Hmm... Is this what you are saying ?

Comment 13

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

No, I meant that the "Bookmak panel" is a XUL <panel> which is opened when "star" is clicked in location bar of Firefox.

Comment 14

[KO Myung-Hun]

When I click 'star', 'Edit This Bookmark' window is opened.

And I can input DBCS chars on 'Name:' and 'Tags:' field. And IME input-box is located as expected.

Comment 15

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

(In reply to KO Myung-Hun from comment #14)
> When I click 'star', 'Edit This Bookmark' window is opened.
> 
> And I can input DBCS chars on 'Name:' and 'Tags:' field. And IME input-box
> is located as expected.

Okay, then, my worry is wrong. Sorry. And thank you for your confirmation and implementing the IME handlers!

Comment 16

[KO Myung-Hun]

(In reply to Masayuki Nakano (:masayuki) (Mozilla Japan) from comment #15)
> (In reply to KO Myung-Hun from comment #14)
> > When I click 'star', 'Edit This Bookmark' window is opened.
> > 
> > And I can input DBCS chars on 'Name:' and 'Tags:' field. And IME input-box
> > is located as expected.
> 
> Okay, then, my worry is wrong. Sorry. And thank you for your confirmation
> and implementing the IME handlers!

Then, would you mind explaining the reason why you worried about that ?

Frankly, I don't understand the logic of Mozilla IME-releated stuffs completely. So I found this code by trial-and-error from the sources of other platforms.

I would appreciate your explanation. ^^

Comment 17

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Hmm, I was confused by older Gecko's code.

Even if an editor in XUL <panel> has focus in Gecko, actual (native) focused widget is its parent window.

Gecko used to return XUL <panel> related x/y for the caret position. Therefore, widget needed to convert the position to screen or focused widget.

But current query events returns the position relative to the root widget.

So, I just misunderstood current design. I designed it, though. :-(

Note that if IME API needs refers different widget from the focused widget, we still need to reconvert the position. This face also confused me.

http://mxr.mozilla.org/mozilla-central/source/widget/cocoa/TextInputHandler.mm#2565
http://mxr.mozilla.org/mozilla-central/source/widget/gtk2/nsGtkIMModule.cpp#1407
http://mxr.mozilla.org/mozilla-central/source/widget/windows/nsIMM32Handler.cpp#2003

Comment 18

[Frank-Rainer Grahl (:frg)]

Sorry OS/2 is no longer an offcially supported platform. Closing.