Closed Bug 5912 Opened 26 years ago Closed 26 years ago

Editor doesn't correctly refresh Japanese input

Categories

(Core :: DOM: Editor, defect, P3)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: tague, Assigned: kinmoz)

Details

The editor doesn't correctly refresh Japanese input on Win32. This problem is new and seemed to start late last week. If you are in input method mode and start typing, the displayed text will be one character/transaction behind the text in the text buffer. If you start typing, you will see the insertion point advance, but it won't display the character you just typed until you type a second character.
Assignee: kostello → tague
Tague, I'm not sure what you want us to do with this bug since none of us have IME support on our system. I am reassigning this back to you. Please diagnose and get support from Kin if you can identify the error in editor code.
Assignee: tague → kostello
The problem is that the document content is not being redrawn after entering the IME text into the DOM - so the display always ends up being one transaction "behind" the content buffer. This is caused by an interaction between tranaction batching and display. Forcing a redraw after the EndTransaction works around the problem for now.
Assignee: kostello → kin
Target Milestone: M7
There are a number of bugs related to redraw problems. Kin will be looking into this as a general problem. He will be addressing this in the M7 milestone.
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
This should work now. I checked in a change to nsEditor.cpp (revision 1.89) that forces a refresh in nsEditor::EndTransaction(). I'm not sure if tague disabled the batching in the IME code or not. Tague? If so, it should be re-enabled.
Tague, can you verify this one and mark it VERIFIED_FIXED ? thanks!
Status: RESOLVED → VERIFIED
Sure, I reported it and fixed the problem that I wanted fixed, so let's close this out.
You need to log in before you can comment on or make changes to this bug.