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

List:       kde-pim
Subject:    Re: [Kde-pim]  [PATCH] What's This support for KPrefsDialog and
From:       Tim Jansen <ml () tjansen ! de>
Date:       2003-07-29 18:27:16
[Download RAW message or body]

On Sunday 27 July 2003 22:52, Cornelius Schumacher wrote:
> The plugin itself doesn't have a configuration dialog yet, but we could
> put the checkbox into the export dialog (where also the filename is
> set).

This does not work, since the dialog has only an 'Export' action and 'Cancel'. 
So you could not deactivate the option without exporting first...

There are several ways to solve this:
- change the export dialog to have 'Export' and 'Close', or 'Export', 'Apply' 
and 'Cancel'. I don't really like this one...
- Add a configure() method to KOrg::Part, as parts can not have configuration 
dialogs right now. It would be the simplest solution, but would break binary 
compatibility for plugins
- Make the Web Export plugin a KOrg::Part and KOrg::Plugin at the same time. I 
tried this, but there are several problems (the plugin dialog displays them 
twice and strange problems in the factory, somehow the wrong create() method 
is invoked by the plugin loader). It is also ugly as this requires a separate 
class for KOrg::Plugin, as the ctor of WebExport adds xmlgui and so on, so 
there would be two WebExport menu entries...

Any suggestions?

bye...



_______________________________________________
kde-pim mailing list
kde-pim@mail.kde.org
http://mail.kde.org/mailman/listinfo/kde-pim
kde-pim home page at http://pim.kde.org/
[prev in list] [next in list] [prev in thread] [next in thread]