[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice-devel
Subject:    Re: Patch for KIllustrator and KWord (XIM)
From:       David Faure <david () mandrakesoft ! com>
Date:       2001-07-15 21:58:23
[Download RAW message or body]

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 ? (Simply use cPoint.y() + h)
Getting the bottom of the current line, including the linespacing ?

Why did you change the 4th argument (the height) from h to 0 ? Isn't it supposed
to be the height ?
[I know nothing about the xim position stuff ... :( ]

-- 
David FAURE, david@mandrakesoft.com, faure@kde.org
http://perso.mandrakesoft.com/~david/, http://www.konqueror.org/
KDE, Making The Future of Computing Available Today

_______________________________________________
Koffice-devel mailing list
Koffice-devel@master.kde.org
http://master.kde.org/mailman/listinfo/koffice-devel

[prev in list] [next in list] [prev in thread] [next in thread]