----------------------------------------------------------- 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
