Bug 431977 - Restore scroll to switch timezone option
Summary: Restore scroll to switch timezone option
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Digital Clock (show other bugs)
Version: 5.20.90
Platform: Other Linux
: HI normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2021-01-23 07:54 UTC by Bhushan Shah
Modified: 2021-01-27 21:33 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Bhushan Shah 2021-01-23 07:54:40 UTC
SUMMARY

Commit in https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/414 removed the option to switch timezone using wheel.

I was a great fan of this feature and in my opinion while it fixed original bug 428096, this introduced a massive regression for me (and others who were using these option).

I work with colleagues who are in 3 different timezones, so some meetings as well as events are generally scheduled in different timezones, and accordingly I decide what clock to show at a time.

For example, During work hours, I set clock to Berlin time, if I have some event (Recent Wayland goal meeting for example) then I set it to UTC.

New option of the switching timezone by the Clock settings is simply not initiative. It requires me to go through 5 different steps where I could previously do this with simple mouse scroll.

- Right click on digital clock
- Digital clock settings
- Timezone
- Select option
- Apply

This is just needless extra work and yes I can see the time in the tooltip, but I much prefer it to be visible in panel rather then it showing up only when I hover mouse over it.

This may seem super insignificant for people who are mostly interacting with 1 timezone but for others this makes it much much annoying.
Comment 1 Nate Graham 2021-01-26 03:25:45 UTC
Why not have three clocks in the panel, one per timezone?

Personally I have a global shortcut (Meta+C) attached to my clock popup and whenever I need to see what time it is in Berlin, I just hit that shortcut and look at the popup. and Meta+C is easy to hit with one hand  on my keyboard so it's fast.

I'm open to re-adding the removed feature if necessary, but I'd like to  make sure we don't regress things too much with respect to Bug 428096.
Comment 2 Bug Janitor Service 2021-01-26 22:25:20 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/599
Comment 3 Nate Graham 2021-01-27 21:28:26 UTC
Git commit a699acd3ff3e831212fb7c27c183e0056bf69821 by Nate Graham.
Committed on 27/01/2021 at 21:27.
Pushed by ngraham into branch 'master'.

[applets/digital clock] Restore scroll-to-change-displayed-timezone

After the rest of the UI changes made to the applet and the time zone
config page, I don't think users will be confused by the differences
between the timezone displayed in the clock, and the systemwide local
timezone anymore. Therefore, it should be safe enough to bring back the
wheely time feature.

M  +4    -0    applets/digital-clock/package/contents/config/main.xml
M  +37   -0    applets/digital-clock/package/contents/ui/DigitalClock.qml
M  +14   -2    applets/digital-clock/package/contents/ui/configTimeZones.qml

https://invent.kde.org/plasma/plasma-workspace/commit/a699acd3ff3e831212fb7c27c183e0056bf69821
Comment 4 Nate Graham 2021-01-27 21:33:38 UTC
Git commit 51d1c4f89ace19f355bf0b7a07b9fbe7b0620e58 by Nate Graham.
Committed on 27/01/2021 at 21:33.
Pushed by ngraham into branch 'Plasma/5.21'.

[applets/digital clock] Restore scroll-to-change-displayed-timezone

After the rest of the UI changes made to the applet and the time zone
config page, I don't think users will be confused by the differences
between the timezone displayed in the clock, and the systemwide local
timezone anymore. Therefore, it should be safe enough to bring back the
wheely time feature.


(cherry picked from commit a699acd3ff3e831212fb7c27c183e0056bf69821)

M  +4    -0    applets/digital-clock/package/contents/config/main.xml
M  +37   -0    applets/digital-clock/package/contents/ui/DigitalClock.qml
M  +14   -2    applets/digital-clock/package/contents/ui/configTimeZones.qml

https://invent.kde.org/plasma/plasma-workspace/commit/51d1c4f89ace19f355bf0b7a07b9fbe7b0620e58