35356 - [IME] Changing mode to direct input moves the caret to the next line

Status: VERIFIED FIXED

(Core :: Internationalization, defect, P3)

Description

[Teruko Kobayashi]

When you editting JA characters by using Hiragana mode, the caret will move
to the beginning of the next line after you change the direct input mode.

Steps of reproduce
1. Open Composer
2. Turn on IME by Shift+~ (default is Hiragana mode)
3. Type "あああ" in hiragana and enter to commit
4. Turn off IME 
5. Type "aaa" in Engish

Notice that the first "a" is display at the beginning of
next line.  After you type 3 "aaa", they are displayed next to
the Japanese characters.

Tested 2000041009 Win32 build.

Comment 1

[Teruko Kobayashi]

Reassign to Joe.

Comment 2

[Kathleen :Brade]

Is this Windows-specific or cross-platform?

Comment 3

[Joe Francis]

doesn't look like an editor problem.  When I dump the content after the first 
"a", I see that the japanese characters and the "a" are all in the same text 
node, consecutively.  Somehow this is getting rendered (on windows only) on two 
seperate lines.  

Troy, can you find this bug a home?  windows-specific rendering woes.

Comment 4

[troy]

Reassigning to Erik

Comment 5

[Erik van der Poel]

Frank, this seems more like your area, I believe.

Comment 6

[Joe Francis]

I recently changed some code which may hide this bug.  I still think there is a 
rendering problem, but I'm not sure if it will show up after today, given my 
checkins.

Comment 7

[Frank Tang]

I will try to try to render a static html there to see what happen. If the 
static html page in the browser show the problem, then we probably have bug in 
layout, otherwise, it could be some timing issue....(just guessing, don't take 
it too seriously)

Comment 8

[Joe Francis]

I had a suspicion earlier that the ime text and english text were initially in 
two seperate text nodes, and at the end of the processing of typing "a", that 
those two textnodes were combined.  My theory was that somehow this combining of 
text nodes was neccessary to drive a layout problem that failed to properly 
render the text.  It's probably a dumb theory.

Since then, I have checked in code which changes the sequence of events in the 
editor a bit.  Even if my theory we right, you minght not be able to see the 
problem any longer.

Comment 9

[Shanjian Li]

I saved the page and loaded it to browser, the same problem can be seen
there too. I checked the page, "あああ"and "a" are in the same line.
That probably suggests a layout problem.

Comment 10

[Shanjian Li]

Attachment: A proposed fix

Comment 11

[Phil Peterson]

Montse or Teruko, could you retest this since Joe says it might be fixed?

Comment 12

[ji]

Retested with 2000050412 comm build. It's not reproducible anymore.

Comment 13

[msanz]

removing nsbeta2, frank, you can mark it fixed

Comment 14

[Shanjian Li]

I haven't check in the fix yet, how can it be fixed?

Comment 15

[msanz]

read jfrancis comment

Comment 16

[Joe Francis]

I dont think my comment is relevant any longer.  If the problem is suddenly 
fixed, someting else is going on.

Comment 17

[ji]

Please ignore the comments I put out yesterday. I thought this is a Linux bug.
Sorry for the confusing test result. I will check it with Windows build.

Comment 18

[ji]

Checked with WIN32 2000050513 commercial build. 
The problem described in the original report is still there. It's not fixed yet.

Comment 19

[ji]

Added myself to CC list.

Comment 20

[Frank Tang]

*** Bug 38365 has been marked as a duplicate of this bug. ***

Comment 21

[Frank Tang]

this is not a IME issue only. The line break in layout are broken. put nsbeta2. 
So far, shanjian proposed a fix bug troy do not agree with it. We need to make 
shanjian explain his fix to troy .

Comment 22

[Frank Tang]

shanjian. I reassign this to you. Check the fix in. r=ftang,erik

Comment 23

[leger]

Putting on [nsbeta2+] radar for beta2 fix.

Comment 24

[Shanjian Li]

fix checked in.

Comment 25

[Teruko Kobayashi]

I verified this in 2000-05-12-10 Win32 build.