Review Request 123207: Use connectionStatus to show if we're changing presence
Aleix Pol Gonzalez
aleixpol at kde.org
Wed Apr 1 12:07:43 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123207/#review78326
-----------------------------------------------------------
Ship it!
LGTM
- Aleix Pol Gonzalez
On April 1, 2015, 2:06 p.m., David Edmundson wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123207/
> -----------------------------------------------------------
>
> (Updated April 1, 2015, 2:06 p.m.)
>
>
> Review request for Telepathy.
>
>
> Repository: ktp-common-internals
>
>
> Description
> -------
>
> This uses the same metric to show the spinner in the contact list and
> the plasmoid.
>
> It stops it spinning eternally and saves plasma-shell's CPU rendering a
> spinning thing forever.
>
> This patch is a minimal change that I want to put into 15.04 that
> doesn't alter API.
>
> Afterwards I'll make connectionStatus a property and tidy things up for
> master.
>
>
> Diffs
> -----
>
> KTp/global-presence.h d1f5dab
> KTp/global-presence.cpp 9c72d40
>
> Diff: https://git.reviewboard.kde.org/r/123207/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> David Edmundson
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20150401/e35ee427/attachment.html>
More information about the KDE-Telepathy
mailing list