Review Request: Fetch a person's OCS social network link icons through Attica.

Teo Mrnjavac teo at kde.org
Thu Dec 23 19:44:12 GMT 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6192/
-----------------------------------------------------------

Review request for kdelibs.


Summary
-------

This patch removes the placeholder icons from the buttons that launch a person's social network profiles in the Social About dialog and fetches the needed icons from the OCS provider through Attica.
Currently 4.6 has only placeholders, so we definitely need to replace those with proper icons.
This has been made possible with a recent addition to the OCS API.
Most of the icons are trademarks so I believe it's best that we don't distribute them, and this seems like the cleanest solution, also because we have no use for those icons except for the Social About dialog anyway.
As for the implementation, I've created a job-like class which fetches the icons sequentially and caches them so we avoid fetching the same icons more than once.


Diffs
-----

  /branches/KDE/4.6/kdelibs/kdeui/dialogs/kaboutapplicationpersonmodel_p.h 1208864 
  /branches/KDE/4.6/kdelibs/kdeui/dialogs/kaboutapplicationpersonmodel_p.cpp 1208864 

Diff: http://svn.reviewboard.kde.org/r/6192/diff


Testing
-------

Tested during and after coding with several OCS profiles.


Thanks,

Teo

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20101223/55cdec28/attachment.htm>


More information about the kde-core-devel mailing list