From koffice-devel  Mon Jul 16 12:53:31 2001
From: David Faure <david () mandrakesoft ! com>
Date: Mon, 16 Jul 2001 12:53:31 +0000
To: koffice-devel
Subject: Re: Patch for KIllustrator and KWord (XIM)
X-MARC-Message: https://marc.info/?l=koffice-devel&m=99529182514375

On Monday 16 July 2001 12:44, Toshitaka Fujioka wrote:
> 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.

Ok. Does it work better this way ?

            int line;
            cursor->parag()->lineStartOfChar( cursor->index(), 0, &line );
            canvas->setXimPosition( cPoint.x(), cPoint.y(), 0, h - cursor->parag()->lineSpacing( line ) );

-- 
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