dateMenu: Make clock offsets relative to local time
We recently added offsets to world clocks that represent the location's timezone as UTC offset. However for most users, that representation is overly technical and less helpful than the difference to their local time. https://gitlab.gnome.org/GNOME/gnome-shell/issues/1157
This commit is contained in:
parent
22883f2fa2
commit
2fab75f448
@ -136,6 +136,8 @@ var WorldClocksSection = class WorldClocksSection {
|
|||||||
layout.attach(header, 0, 0, 2, 1);
|
layout.attach(header, 0, 0, 2, 1);
|
||||||
this.actor.label_actor = header;
|
this.actor.label_actor = header;
|
||||||
|
|
||||||
|
let localOffset = GLib.DateTime.new_now_local().get_utc_offset();
|
||||||
|
|
||||||
for (let i = 0; i < this._locations.length; i++) {
|
for (let i = 0; i < this._locations.length; i++) {
|
||||||
let l = this._locations[i].location;
|
let l = this._locations[i].location;
|
||||||
|
|
||||||
@ -148,7 +150,8 @@ var WorldClocksSection = class WorldClocksSection {
|
|||||||
|
|
||||||
let time = new St.Label({ style_class: 'world-clocks-time' });
|
let time = new St.Label({ style_class: 'world-clocks-time' });
|
||||||
|
|
||||||
let offset = l.get_timezone().get_offset() / 60.;
|
let otherOffset = this._getTimeAtLocation(l).get_utc_offset();
|
||||||
|
let offset = (otherOffset - localOffset) / GLib.TIME_SPAN_HOUR;
|
||||||
let fmt = (Math.trunc(offset) == offset) ? '%s%.0f' : '%s%.1f';
|
let fmt = (Math.trunc(offset) == offset) ? '%s%.0f' : '%s%.1f';
|
||||||
let prefix = (offset >= 0) ? '+' : '-';
|
let prefix = (offset >= 0) ? '+' : '-';
|
||||||
let tz = new St.Label({ style_class: 'world-clocks-timezone',
|
let tz = new St.Label({ style_class: 'world-clocks-timezone',
|
||||||
@ -181,11 +184,15 @@ var WorldClocksSection = class WorldClocksSection {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
_getTimeAtLocation(location) {
|
||||||
|
let tz = GLib.TimeZone.new(location.get_timezone().get_tzid());
|
||||||
|
return GLib.DateTime.new_now(tz);
|
||||||
|
}
|
||||||
|
|
||||||
_updateLabels() {
|
_updateLabels() {
|
||||||
for (let i = 0; i < this._locations.length; i++) {
|
for (let i = 0; i < this._locations.length; i++) {
|
||||||
let l = this._locations[i];
|
let l = this._locations[i];
|
||||||
let tz = GLib.TimeZone.new(l.location.get_timezone().get_tzid());
|
let now = this._getTimeAtLocation(l.location);
|
||||||
let now = GLib.DateTime.new_now(tz);
|
|
||||||
l.actor.text = Util.formatTime(now, { timeOnly: true });
|
l.actor.text = Util.formatTime(now, { timeOnly: true });
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user