Review Request: accept KAssitantDialog when calling next() on last page and if that is valid

Aaron J. Seigo aseigo at kde.org
Mon Aug 29 09:42:34 BST 2011


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

Ship it!



kdeui/dialogs/kassistantdialog.cpp
<http://git.reviewboard.kde.org/r/102475/#comment5394>

    should probably be changed to use kdelibs style ...


- Aaron J.


On Aug. 28, 2011, 12:51 p.m., Milian Wolff wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102475/
> -----------------------------------------------------------
> 
> (Updated Aug. 28, 2011, 12:51 p.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> The apidox for KAssitantDialog::next says:
> 
>         /**
>          * Called when the user clicks the Next/Finish button.
>          *
>          * This function will show the next relevant page in the sequence.
>          * If the current page is the last page, it will call accept()
>          */
> 
> The last sentence is actually not true and this patch introduces this behavior.
> 
> 
> Diffs
> -----
> 
>   kdeui/dialogs/kassistantdialog.cpp 0cb361a 
> 
> Diff: http://git.reviewboard.kde.org/r/102475/diff
> 
> 
> Testing
> -------
> 
> You can add this before the "return a.exec();" in kassistantdialogtest.cpp to verify that it now works as intended:
> 
>   for(int i = 1; i < 11; i++)
>   {
> 	QMetaObject::invokeMethod(dlg, "next", Qt::QueuedConnection);
>   }
>   
> 
> 
> Thanks,
> 
> Milian
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110829/9a9c6ba8/attachment.htm>


More information about the kde-core-devel mailing list