Review Request: Initial avatars support

[David Edmundson]

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



main-widget.cpp
<http://git.reviewboard.kde.org/r/101048/#comment2141>

    do you need to pass account Ids about? it's better to just pass AccountrPtrs if possible.
    
    You can use AccountManager::accountForPath()



main-widget.cpp
<http://git.reviewboard.kde.org/r/101048/#comment2140>

    foreach(const Tp::AccountPtr &account...



main-widget.cpp
<http://git.reviewboard.kde.org/r/101048/#comment2142>

    ReadWrite?


- David


On April 7, 2011, 5:15 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101048/
> -----------------------------------------------------------
> 
> (Updated April 7, 2011, 5:15 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Summary
> -------
> 
> This patch adds initial avatars support plus KConfig, which holds the selected avatar.
> 
> 
> Diffs
> -----
> 
>   main-widget.h e310b35d475509f5683a0abe15f21bbb1d8f122e 
>   CMakeLists.txt 548e8c880b5ef921606c7407b92e2e1650faef86 
>   accounts-model-item.cpp 1d207e7eebb6d286d0fc075dfa4e90628ac92bcc 
>   main-widget.cpp 2e049a13561a1dc4ca0b7117a52ed713d775b9bd 
>   main-widget.ui bc1fb9bf030a927cf635ed1f73d71d6ee33675b2 
>   main.cpp aac4f4b98f8ad809fd6380b45be793fdbcae8814 
> 
> Diff: http://git.reviewboard.kde.org/r/101048/diff
> 
> 
> Testing
> -------
> 
> Tried to change the avatar and it worked.
> 
> 
> Screenshots
> -----------
> 
> Avatars menu
>   http://git.reviewboard.kde.org/r/101048/s/114/
> 
> 
> Thanks,
> 
> Martin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110407/ec0cfad9/attachment.htm