[Kde-pim] Re: Review Request: Add CollectionProperties method to select default pages to display

[Tobias Koenig]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5855/#review9267
-----------------------------------------------------------


Hej David,

I reverted your patch because it breaks the system of using custom pages in the properties dialog.
Every properties page should have an unique QObject::objectName set (e.g. it's full qualified class name), so has the CollectionGeneralPage and the CachePolicyPage.
If no names are passed to the CollectionPropertiesDialog constructor, all registered default pages will be displayed, otherwise the pages with the given names.
This has the advantage that the caller of the CollectionPropertiesDialog has the full control of the pages that shall be displayed and their order. An additional option to enable/disable the default pages is just confusing and makes the API more difficult to use.
I havn't been aware of your reviewboard patch until you commited it, therefor no comments from me before. Sorry for the confusion.

Ciao,
Tobias

- Tobias


On 2010-11-16 00:03:17, David Jarvie wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5855/
> -----------------------------------------------------------
> 
> (Updated 2010-11-16 00:03:17)
> 
> 
> Review request for KDE PIM.
> 
> 
> Summary
> -------
> 
> Provide more flexible options for displaying an Akonadi collection properties dialog. Use of an enum allows an application to select which default page(s) to show. The existing useDefaultPage() is deprecated in favour of the new useDefaultPages() method.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdepimlibs/akonadi/collectionpropertiesdialog.h 1196227 
>   /trunk/KDE/kdepimlibs/akonadi/collectionpropertiesdialog.cpp 1196227 
> 
> Diff: http://svn.reviewboard.kde.org/r/5855/diff
> 
> 
> Testing
> -------
> 
> Works in KAlarm.
> 
> 
> Thanks,
> 
> David
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/