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

List:       koffice-devel
Subject:    Re: Patch for KPresenter (Configure dialog)
From:       Toshitaka Fujioka <toshitaka () kde ! gr ! jp>
Date:       2001-06-02 11:46:24
[Download RAW message or body]

On Saturday 02 June 2001 18:18, David Faure wrote:
> On Saturday 02 June 2001 12:11, Toshitaka Fujioka wrote:
> > > Can you use KoDocument::defaultAutoSave()/60 for the default value for
> > > autosave ? (/60 because it's in seconds). I just added that today, so
> > > you'll need to update first. This is intended to make it easier to
> > > change the default value, it will be only once in koDocument.
> >
> > I remade patch.
>
> Hmm, you used the default value in slotDefaults(), but not when reading
> from the config file. You need to use it there too (instead of
> oldAutoSaveValue = 1, and then use oldAutoSaveValue as the default value
> when reading from the config file - see kword's kwconfig, that's where your
> code comes from anyway :-)

Oops, I'm sorry. I remade patch.
Is this OK ?

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-kpresenter-configure-dialog-20010602.diff.bz2" (application/x-bzip2)]
_______________________________________________
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]