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

List:       koffice-devel
Subject:    Re: kpresenter patch
From:       Thorsten Zachmann <t.zachmann () zagge ! de>
Date:       2004-03-18 5:19:20
Message-ID: 200403180619.20822.t.zachmann () zagge ! de
[Download RAW message or body]

On Thursday 18 March 2004 00:33, Percy Leonhardt wrote:
> Hi Thorsten!
>
> > What do you think?
>
> Hmm, 122 changes in kprcanvas.cc. No chance that I will understand what
> they are doing. And I must say I'm amazed about how deep you digged into
> the internals of KPresenter. I wish I could assist you a bit there... your
> code cleanups and added comments really help alot.
>

Thanks, help is allways wellcome. How about the dialogs you talked about. I 
think this is a good start.

> I can't resist to nitpick a bit:
> You use the english word "actual" in your comments and in the code which
> means in german "wirklich, tatsächlich, eigentlich" but is often wrongly
> translated as "aktuell", but "aktuell" is in english "current".
> So I think "current page number" or "current effect step" makes more sense
> and is what you really meant. Or?

You are right, I'll change it.

> Why not? I think it's always good to have some fixes in KPresenter if we
> are going to release KOffice 1.3.1.

ok I'll backport it.

Thorsten
_______________________________________________
koffice-devel mailing list
koffice-devel@mail.kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel

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