General applet test-case: Color change

Marco Martin notmart at gmail.com
Sun Dec 28 19:52:39 CET 2008


On Sunday 28 December 2008, Jamboarder wrote:
> --- On Sun, 12/28/08, Marco Martin <notmart at gmail.com> wrote:
> > yes, in themes without a colors file the
> > kdisplayPaletteChanged() signal is
> > connected to themeChanged() signal, so is the only one
> > needed
>
> Just noticed that themeChanged() was being emitted at the same time the
> Plasma color scheme was being updated in Plasma::Theme.  This meant that
> recipients of the themeChanged() signal could potentially react before the
> plasma color scheme completed updating.
>
> Quick, simple fix to emit the themeChanged signal after the plasma color
> scheme update has been committed. This fixes all the problems I listed in
> the original report except for RSSNOW. (I haven't tested Sysem Monitor but
> if it's reacting properly to themeChanged() then it should be ok).

+1, good catch :)
just changed rssnow, now it reacts correctly

Cheers,
Marco Martin

> Hope this helps,
> Andrew
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel




More information about the Plasma-devel mailing list