[Kde-pim] Review Request: add Akonadi::CollectionPropertiesDialog::setCurrentPage()

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106623/#review19567
-----------------------------------------------------------


This review has been submitted with commit ea9a9a132e09d2ac1bd359bcccf68cdde3b3c566 by Jonathan Marten to branch master.

- Commit Hook


On Sept. 28, 2012, 1:17 p.m., Jonathan Marten wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106623/
> -----------------------------------------------------------
> 
> (Updated Sept. 28, 2012, 1:17 p.m.)
> 
> 
> Review request for KDEPIM-Libraries.
> 
> 
> Description
> -------
> 
> When a CollectionPropertiesDialog is opened, it always shows the first tab page.  In some applications it may be useful to request that the dialogue opens with a specific tab shown, for example if any formerly separate dialogues were to be merged into KMail's "Folder Properties" dialogue.  The action which summoned the original dialogue can remain in the GUI, which will open the combined dialogue with the appropriate tab shown.
> 
> This will hopefully reduce user confusion (of the "where has this gone from the menu?" variety).
> 
> This change (adding a new non-virtual function) is BC according to
> http://techbase.kde.org/Policies/Binary_Compatibility_Issues_With_C++
> 
> 
> Diffs
> -----
> 
>   akonadi/collectionpropertiesdialog.h 9157c96 
>   akonadi/collectionpropertiesdialog.cpp fae9761 
> 
> Diff: http://git.reviewboard.kde.org/r/106623/diff/
> 
> 
> Testing
> -------
> 
> Built kdepimlibs with this change, and added a testing stub to KMail to call it.  Verified that the specified tab page of the dialogue is shown when it is opened.
> 
> 
> Thanks,
> 
> Jonathan Marten
> 
>

_______________________________________________
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/