----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120314/#review67205 -----------------------------------------------------------
The following steps lead to unexpected behavior for me: a) Add Digital Clock to panel. b) Set display to "Timezone city". c) Disable "Local". d) Enable "Manila". e) Set default to "Manila". f) Click OK. Expected result: A Digital Clock showing Manila. Actual result: No changes. Opening the config again, Manila is disabled and default is Berlin. All changes seem to have been lost. Seems to be related to disabling "Local". - Eike Hein On Sept. 22, 2014, 2:33 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120314/ > ----------------------------------------------------------- > > (Updated Sept. 22, 2014, 2:33 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > ------- > > I've picked up Kai's branch where he started the timezone support and > finished it. > > New features: > * timezones tab in the config > * changing timezones by scrolling a wheel on the clock > * that's optional, defaults to off > * displaying non-local timezones as either city or abbreviation > * "Show time zone" config option is now changed to "Show local time zone" as > foreign zones are always displayed by default > > What's still missing: > * selecting default timezone in the config (should be fairly easy; currently > the last used is restored on login > * timezones in the popup, this would require either large changes to the > applet (moving functions around) or displaying the time in non-locale way > > > > > Diffs > ----- > > applets/digital-clock/package/contents/config/config.qml f897c9a > applets/digital-clock/package/contents/config/main.xml cbeb02f > applets/digital-clock/package/contents/ui/DigitalClock.qml 614e172 > applets/digital-clock/package/contents/ui/configAppearance.qml 01f6878 > applets/digital-clock/package/contents/ui/configTimeZones.qml PRE-CREATION > applets/digital-clock/package/contents/ui/main.qml e78f1f3 > applets/digital-clock/plugin/CMakeLists.txt 0590e6e > applets/digital-clock/plugin/digitalclockplugin.h PRE-CREATION > applets/digital-clock/plugin/digitalclockplugin.cpp PRE-CREATION > applets/digital-clock/plugin/processrunnerplugin.h 2b6317b > applets/digital-clock/plugin/processrunnerplugin.cpp ed039a3 > applets/digital-clock/plugin/qmldir 3d57bbb > applets/digital-clock/plugin/timezonedata.h PRE-CREATION > applets/digital-clock/plugin/timezonemodel.h PRE-CREATION > applets/digital-clock/plugin/timezonemodel.cpp PRE-CREATION > dataengines/time/timesource.cpp b79fd7c > > Diff: https://git.reviewboard.kde.org/r/120314/diff/ > > > Testing > ------- > > I've been using it for couple days, all those features from above work nicely > here, with all sorts of config configurations (eg. if date is displayed, it > also changes properly with timezone change). > > > File Attachments > ---------------- > > Timezone & date displayed > > https://git.reviewboard.kde.org/media/uploaded/files/2014/09/22/5462a885-827c-4979-acf4-55da7ea061d4__plasma_clock_tz1.png > Timezone without date > > https://git.reviewboard.kde.org/media/uploaded/files/2014/09/22/fcae7191-84f1-4377-b26a-5d39035c1ca7__plasma_clock_tz2.png > > > Thanks, > > Martin Klapetek > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
