Review Request 108498: KTp::ContactInfoDialog

[Commit Hook]

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


This review has been submitted with commit 4e0c9f95f0e43ce22d37e4811af7b8d196925f46 by Dan Vrátil to branch master.

- Commit Hook


On Jan. 20, 2013, 2 a.m., Dan Vrátil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108498/
> -----------------------------------------------------------
> 
> (Updated Jan. 20, 2013, 2 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> KTp::ContactInfoDialog is a dialog very similar to the contact info dialog in contact list, but it displays more detailed information, like full name, homepage, birthday etc.
> 
> I moved the dialog to ktp-common-internals, so that it can be shared between contactlist and kded module - see bug https://bugs.kde.org/show_bug.cgi?id=298764.
> 
> The dialog also supports editing mode (see screenshots), when you open the dialog with "yourself" (that's contact == account->connection->selfContact). This feature could be used in KCM for instance. Unfortunately when I was almost finished, I found out, that TelepathyQt4 does not have API for editing contact information (RAGE!), so the editing code is disabled for now. Upstream patch is already waiting for review: https://bugs.freedesktop.org/show_bug.cgi?id=59609
> 
> 
> Diffs
> -----
> 
>   KTp/CMakeLists.txt 10c17bf 
>   KTp/contact-info-dialog.h PRE-CREATION 
>   KTp/contact-info-dialog.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/108498/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> 
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/01/20/contactdialog.png
> 
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/01/20/contactdialog-edit.png
> 
> 
> Thanks,
> 
> Dan Vrátil
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130127/3c500234/attachment.html>