AIX: Preedit string deleted when arrow keys are pressed

RESOLVED INCOMPLETE

Status

()

Core
Widget: Gtk
--
major
RESOLVED INCOMPLETE
16 years ago
2 years ago

People

(Reporter: Philip K. Warren, Unassigned)

Tracking

({inputmethod})

Trunk
Other
AIX
inputmethod
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

16 years ago
When editing the preedit string, if any arrow keys are pressed, Mozilla deletes
the preedit string. This makes it impossible to navigate the preedit string.
(Reporter)

Comment 1

16 years ago
Created attachment 118785 [details]
Log

This is a log which gives specific details of what occurs when typing in a
preedit string and moving the arrow key. In the last call to the preedit draw
callback, XIMText->length is 12, but strlen(XIMText->string.multi_byte) is
zero. This causes  a call to the Reset() function call in the following
location (which deletes the preedit string):

http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/widget/src/gtk/nsGtkIMEHelper.cpp&rev=1.47&root=/cvsroot&mark=279-282#266
(Reporter)

Comment 2

16 years ago
Created attachment 118944 [details] [diff] [review]
Patch v1

Don't reset the preedit string on AIX if composeUniStringLen != preeditLen. Use
preeditLen instead of composeUniStringLen when building the feedback array
(because composeUniStringLen is zero on AIX). None of these changes should
affect other platforms, because one section is #ifdef _AIX, and for all of the
other platforms, composeUniStringLen is always equal to preeditLen, so they can
be used interchangeably.
(Reporter)

Updated

16 years ago
Attachment #118944 - Flags: superreview?(blizzard)
Attachment #118944 - Flags: review?(katakai)
(Reporter)

Updated

16 years ago
Blocks: 199964

Updated

16 years ago
Attachment #118944 - Flags: review?(katakai) → review+
(Reporter)

Comment 3

13 years ago
Reassigning bugs to new owner. This bug is likely still present on AIX, but no longer an issue with default builds because we now build using GTK1.
Assignee: pkwarren → rupeshkt
(Reporter)

Updated

13 years ago
Attachment #118944 - Flags: superreview?(blizzard)
QA Contact: amyy → i18n
Keywords: inputmethod
Currently, we only support GTK widget for *nix and I've improved a lot of IME handling and the de facto standard of IME on *nix has been changed a couple of times in this decade. Therefore, I'm marking this as INCOMPLETE.

If you still have some problem on AIX, please file new bugs with STR and information of your environment.
Assignee: rupeshkt → nobody
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Component: Internationalization → Widget: Gtk
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.