[Kde-pim] Review Request 112864: kaddressbook: display all of the contact's IM addresses

[Kevin Krammer]

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



kaddressbookgrantlee/formatter/grantleecontactformatter.cpp
<http://git.reviewboard.kde.org/r/112864/#comment29949>

    one const too much :
    key.remove() in the next line won't compile, remove() is not a const method


- Kevin Krammer


On Sept. 24, 2013, 1:43 p.m., Jonathan Marten wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112864/
> -----------------------------------------------------------
> 
> (Updated Sept. 24, 2013, 1:43 p.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Description
> -------
> 
> If a contact has multiple IM addresses defined, only the one set as standard (or the first one if this has not been done) are displayed.  There is also no indication of the protocol to which this applies.
> 
> With this change, all of the IM addresses are shown with their protocol and icon.  It requires the header file and symbol export at https://git.reviewboard.kde.org/r/112844/ to be applied to kdepimlibs in order to compile kdepim, so intended for the master branch only.
> 
> This change was originally suggested for the standard contact formatter at https://git.reviewboard.kde.org/r/108837/.  There was some discussion on that with the KTelepathy people and the conclusion was that making these clickable within KAddressBook was not really a useful thing to do.  So the data is displayed only - the text can still be copied if necessary.
> 
> 
> Diffs
> -----
> 
>   kaddressbookgrantlee/formatter/grantleecontactformatter.cpp 1129e31 
>   kaddressbook/viewertemplates/contact_embedded.html 233366c 
> 
> Diff: http://git.reviewboard.kde.org/r/112864/diff/
> 
> 
> Testing
> -------
> 
> Built kaddressbook with these changes, checked contact display.
> 
> 
> File Attachments
> ----------------
> 
> All contact IM addresses displayed
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/09/21/kaddressbook-imdisplay_B1.png
> The full repertoire
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/09/21/kaddressbook-imdisplay_B2.png
> 
> 
> 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/