Review Request 121125: Port ContactInfoDialog away from kdelibs4support

David Edmundson david at davidedmundson.co.uk
Sat Nov 15 03:11:18 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121125/#review70386
-----------------------------------------------------------

Ship it!



KTp/contact-info-dialog.cpp
<https://git.reviewboard.kde.org/r/121125/#comment49210>

    I would have expected you to have to add this to the layout?


Is this class not deprecated? In KPeople mode we don't use this dialog and kpeople was going to become a mandatory dependency.

- David Edmundson


On Nov. 14, 2014, 7:20 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121125/
> -----------------------------------------------------------
> 
> (Updated Nov. 14, 2014, 7:20 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Repository: ktp-common-internals
> 
> 
> Description
> -------
> 
> As summary; as it has #include <KDialog> in the public header, it prevents other components from finishing kdelibs4support-free ports, so I ported it.
> 
> RR 121124 then moves it to KTp/Widgets where it belongs.
> 
> 
> Diffs
> -----
> 
>   KTp/contact-info-dialog.h 833c74f 
>   KTp/contact-info-dialog.cpp 47206cd 
> 
> Diff: https://git.reviewboard.kde.org/r/121125/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20141115/000e1c93/attachment.html>


More information about the KDE-Telepathy mailing list