---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/891/#review1384
---
Much better than mine!
It would be excellent if we could trigger an update of all the containers
attached but I just can't find the way to do it (is it possible?)
Right now it waits for next trigger to update all clocks, which can be in fact
be quite confusing, as the user can have to wait 1 minute to have it set.
- Jacopo
On 2009-06-25 16:34:27, Aaron Seigo wrote:
---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/891/
---
(Updated 2009-06-25 16:34:27)
Review request for Plasma.
Summary
---
Alternate patch to http://reviewboard.kde.org/r/889/
from 889: The plasma timeengine reacts to timezone changes, but the
timesource does not; this patch filles the gap.
the benefit of this patch is that the change is done in one place, no
synchronization issues based on who gets a signal first. (i wonder if it
needs to update all the timezones, even, or just the local one? not that
updateAllSources is slow, so it shouldn't matter..)
Diffs
-
trunk/KDE/kdebase/workspace/plasma/dataengines/time/timeengine.h 980060
trunk/KDE/kdebase/workspace/plasma/dataengines/time/timeengine.cpp 980060
trunk/KDE/kdebase/workspace/plasma/dataengines/time/timesource.h 980060
trunk/KDE/kdebase/workspace/plasma/dataengines/time/timesource.cpp 980060
Diff: http://reviewboard.kde.org/r/891/diff
Testing
---
Thanks,
Aaron
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel