> On Sept. 24, 2014, 2:42 nachm., Eike Hein wrote: > > Doesn't always keeping Local enabled create an awkward situation if you > > want to default a clock to a non-Local timezone but want wheel scrolling > > disabled for accident-safety a la sebas? > > > > Then you need to: > > > > 1. Enable additional timezone > > 2. Enable wheelscroll > > 3. OK dialog > > 4. Change TZ by scrolling > > 5. Go back to dialog to disable wheelscroll
In Plasma 4.x there is a combo box that allows you to change the timezone without a mousewheel (which should be possible nonetheless, since mousewheels are neither always available nor discoverable). And imho a message box is the worst possible solution. A message widget I could live with but then we don't have that in QML yet. - Kai Uwe ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120314/#review67360 ----------------------------------------------------------- On Sept. 24, 2014, 8:40 nachm., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120314/ > ----------------------------------------------------------- > > (Updated Sept. 24, 2014, 8:40 nachm.) > > > 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
