[Kde-imaging] Selecting images for plugins

Renchi Raju renchi at pooh.tam.uiuc.edu
Mon Jun 28 17:51:45 CEST 2004



On Mon, 28 Jun 2004, Aurélien Gâteau wrote:

> Le lundi 28 Juin 2004 15:26, Jesper K. Pedersen a écrit :
> e the following API:
> > |
> > |   // Returns the images of all selected image collection.
> > |   // An image won't appear twice in the returned list.
> > |   KURL::List images() const;
> >
> > images() is not needed is it? that is returned from ImageCollection.
>
> It is for plugins which don't care about albums, but only want a set of
> images, like CD archiving plugin.

you mean all the images are returned as one list? i don't see how its
useful, as the cdarchiving plugin (if it uses this list) will have to put
all the images in one folder on the cd.

> Yes. A better API would be:
>
> class AlbumSelector : public QWidget {
>   bool multiSelection() const;
>   void setMultiSelection(bool);
>
>   // Returns a list of all selected image collections.
>   // If multiSelection is off, the list will only contain one item
>   QValueList<ImageCollection> selectedImageCollections() const;
>
>   // Returns the images of all selected image collections.
>   // An image won't appear twice in the returned list.
>   KURL::List images() const;
> }

imo the multiselection option is not very useful. if an album doesn't want
multiple albums, there is no point providing a selector to choose
albums.

renchi


More information about the Kde-imaging mailing list