From koffice-devel Mon Jul 16 10:44:08 2001 From: Toshitaka Fujioka Date: Mon, 16 Jul 2001 10:44:08 +0000 To: koffice-devel Subject: Re: Patch for KIllustrator and KWord (XIM) X-MARC-Message: https://marc.info/?l=koffice-devel&m=99528013109802 On Monday 16 July 2001 06:58, David Faure wrote: > On Monday 16 July 2001 00:11, Toshitaka Fujioka wrote: > > On Friday 13 July 2001 19:42, Toshitaka Fujioka wrote: > > > Oops! :o Sorry, You're 100% right. This was bad patch(KWord). :( > > > I remade patch. This patch completely works. ;) > > > > Oops!! Oh no, I did it again. :o > > Sorry, the patch was not complete. :( > > When "Line Spacing" was not "Single", XIM position was incomplete. > > I remade a patch. Please review. > > Eek, this is horrible (sorry). We want to add more linespacings in the > future, without having to update this code. To get the linespacing in > pixels, just use parag->lineSpacing( linenumber ) - but I'm surprised you > even have to do this. > > What are you trying to do exactly ? > Getting the bottom of the cursor ? Yes. >(Simply use cPoint.y() + h) Hmm, this is right with "Line Spacing is Single". > Getting the bottom of the current line, including the linespacing ? This is a problem. Because, value is added to "h" when I changed "Line Spacing". Position of XIM falls down to only value added to "h". I must subtract added value from "h" in order to set position of XIM to a bottom of cursor. > Why did you change the 4th argument (the height) from h to 0 ? Isn't it > supposed to be the height ? Because position of "x" were incomplete, I used QPoint ximPoint = viewMode->normalToView( clip.bottomRight() ); . ximPoint.y() is same as cPoint.y()+h. Thank you. -- Toshitaka Fujioka http://www.kde.org The K Desktop Environment Project fujioka@kde.org http://www.kde.gr.jp Japan KDE User's Group toshitaka@kde.gr.jp -- We offer the best Desktop Environment to you. (This is my goal. ;-)) -- _______________________________________________ Koffice-devel mailing list Koffice-devel@master.kde.org http://master.kde.org/mailman/listinfo/koffice-devel