79135 - NS_COMPOSITION_QUERY does not return the same value for height per line

Status: RESOLVED FIXED

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

Description

[Masaki Katakai]

NS_COMPOSITION_QUERY is used to get the position and width,
height of current composition. Linux/UNIX nsWidget uses
this interface to get the position and height. The height
will be used for the height of composition text in
over-the-spot mode, which is the mode that XIM server draws
the composition text.

nsWidget::UpdateICSpot()
{
   compEvent.message = NS_COMPOSITION_QUERY;
   compEvent.eventStructType = NS_COMPOSITION_QUERY;
   compEvent.compositionMessage = NS_COMPOSITION_QUERY;

   this->OnComposition(compEvent);

   spot.x = compEvent.theReply.mCursorPosition.x;
   spot.y = compEvent.theReply.mCursorPosition.y +
                compEvent.theReply.mCursorPosition.height;
   SetXICBaseFontSize( compEvent.theReply.mCursorPosition.height - 1);
}

However, it seems that the height is changed even when
the cursor is in the same line. For example, when the cursor
is in "Subject" field,

cursor is in the beginning of the field, height is returned as 16 on
my environment,

	x=85 y=199 h=16

It seems that 16 point is correct.

But if you type one character, the value of height is changed to
19 from 16.

	x=149 y=198 h=19

It seems that 19 point is bigger that expected.

Please look at the snapshots. You can not find the problem that
the size of composition text is not same even when we're trying
to enter characters in the Subject field.

Comment 1

[Masaki Katakai]

Attachment: drawing composed text in 16 point

Comment 2

[Masaki Katakai]

Attachment: but wrong height is returned, so drawing in 19 point

Comment 3

[Andreas Becker]

Changing QA contact to [e-mail] for now.

Comment 4

[nhottanscp]

Katakai san, should this be assigned to widget or editor? It doesn't look like
an i18n bug.

Comment 5

[Masaki Katakai]

Hi Hotta-san, yes, can you move this to editor compoment?

Comment 6

[nhottanscp]

Change component to editor and reassign.

Comment 7

[rubydoo123]

assign to anthonyd

Comment 8

[Frank Tang]

andreas, I don't understand why you assign this QA contact to me. This bug can 
be seen by the user/tester. reassign the qa contact back to andreas

I don't think this should be an moz0.9.2 bug. The current behavior is not good 
but it is still good enough for a product. katakai- is that ok?

Comment 9

[rubydoo123]

had a mail exchange with ftang, we agreed this can go to 1.0

Comment 10

[Masaki Katakai]

OK.

This is a problem when we use over-the-spot mode, which is not
the default input style. The default is on-the-spot. But some
users in japanese prefer over-the-spot. Anyway, target should be 1.0.

Thanks.

Comment 11

[Andreas Becker]

Changing QA contact to [e-mail]

Comment 12

[rubydoo123]

--> kin

Comment 13

[kinmoz]

Bulk move of mozilla1.0 bugs to mozilla.1.0.1. I will try to pull some of these
back in if I can.

Comment 14

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

should be fixed by bug 528435 and bug 520732.