[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: General applet test-case: Color change
From:       Marco Martin <notmart () gmail ! com>
Date:       2008-12-28 18:52:39
Message-ID: 200812281952.39138.notmart () gmail ! com
[Download RAW message or body]

On Sunday 28 December 2008, Jamboarder wrote:
> --- On Sun, 12/28/08, Marco Martin <notmart@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@kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel


_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]