After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 644390 - Reimplement timezone display
Reimplement timezone display
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: calendar
3.0.x
Other Linux
: Normal enhancement
: ---
Assigned To: David Zeuthen (not reading bugmail)
gnome-shell-maint
: 648583 674099 683313 687916 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-03-10 13:50 UTC by Nguyen Thai Ngoc Duy
Modified: 2014-11-07 14:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
dateMenu: Add "Open Clocks" entry (1.68 KB, patch)
2012-11-10 01:14 UTC, Mathieu Bridon
none Details | Review
How it looks (646.17 KB, image/png)
2012-11-10 01:17 UTC, Mathieu Bridon
  Details
dateMenu: Add "Open Clocks" entry (1.53 KB, patch)
2012-11-13 12:00 UTC, Mathieu Bridon
none Details | Review
How it looks (540.71 KB, image/png)
2012-11-13 12:02 UTC, Mathieu Bridon
  Details
dateMenu: Add "Open Clocks" entry (2.01 KB, patch)
2012-11-13 13:00 UTC, Mathieu Bridon
none Details | Review
How it looks (538.53 KB, image/png)
2012-11-13 13:01 UTC, Mathieu Bridon
  Details
dateMenu: Add "Open Clocks" entry (3.71 KB, patch)
2012-12-15 10:40 UTC, Mathieu Bridon
reviewed Details | Review
dateMenu: Add "Open Clocks" entry (3.93 KB, patch)
2012-12-19 21:24 UTC, Mathieu Bridon
needs-work Details | Review
dateMenu: Move the "Open Calendar" entry (2.97 KB, patch)
2012-12-21 22:30 UTC, Mathieu Bridon
accepted-commit_now Details | Review
dateMenu: Add "Open Clocks" entry (2.34 KB, patch)
2012-12-21 22:31 UTC, Mathieu Bridon
accepted-commit_now Details | Review
[PATCH 1/2] dateMenu: Move the "Open Calendar" entry (2.35 KB, patch)
2013-01-16 03:00 UTC, Mathieu Bridon
committed Details | Review
dateMenu: Add "Open Clocks" entry (2.42 KB, patch)
2013-01-16 03:01 UTC, Mathieu Bridon
committed Details | Review

Description Nguyen Thai Ngoc Duy 2011-03-10 13:50:22 UTC
I work with people from three different timezones so looking up what time it is at someone's place is quite frequent task for me. No need of fancy global map, just a line each zone would be enough.
Comment 1 Allan Day 2011-05-20 16:12:25 UTC
*** Bug 648583 has been marked as a duplicate of this bug. ***
Comment 2 Allan Day 2011-05-20 16:14:12 UTC
Relevant design page:

http://live.gnome.org/GnomeShell/Design/Whiteboards/DateNTime
Comment 3 David Zeuthen (not reading bugmail) 2011-05-20 16:25:48 UTC
(In reply to comment #2)
> Relevant design page:
> 
> http://live.gnome.org/GnomeShell/Design/Whiteboards/DateNTime

That's fine but this doesn't tell us where to get the data (=locations to display in the calednar dropdown). FWIW, I'd expect such data to be configured in

 https://live.gnome.org/Design/SystemSettings/DateAndTime

And, FWIW, the date+time control panel still conflates locations with entries in the Olsson timezone database (which it also confusingly renders as Region/City)... in particular for the "show locations in calendar dropdown" feature discussed here you'd want the same fine-grained resolution as we had in GNOME 2.

E.g. you want to be able to choose Boston, MA or Providence, RI and not having to chose New York, NY just because of how time-zones work on the technical level. Current behavior is tantamount to asking Londoners to choose "Frankfurt, DE" as their "Region/City"....
Comment 4 Florian Müllner 2011-05-20 16:43:42 UTC
(In reply to comment #3)
> Current behavior is tantamount to asking Londoners to choose "Frankfurt,
> DE" as their "Region/City"....

Uh? You made that up, didn't you? London and Frankfurt are in different timezones ;-)
Comment 5 Florian Müllner 2011-05-20 16:45:07 UTC
(In reply to comment #4)
> (In reply to comment #3)
> > Current behavior is tantamount to asking Londoners to choose "Frankfurt,
> > DE" as their "Region/City"....
> 
> Uh? You made that up, didn't you? London and Frankfurt are in different
> timezones ;-)

Ugh, ignore that comment - I just looked up "tantamount" ...
Comment 6 David Zeuthen (not reading bugmail) 2011-05-20 16:48:40 UTC
(In reply to comment #4)
> (In reply to comment #3)
> > Current behavior is tantamount to asking Londoners to choose "Frankfurt,
> > DE" as their "Region/City"....
> 
> Uh? You made that up, didn't you? London and Frankfurt are in different
> timezones ;-)

Ah, yeah, except for that tiny detail.. :-) ... s/Londoners/Parisians/ then... my point was just that the entries in the Olsson database isn't very suitable for display to the end user. You only end up making people bitter about national / regional things or sports teams or stuff like that.
Comment 7 Dan Winship 2011-05-20 17:51:45 UTC
a) libgweather has abstractions for nicer (and translated) timezone names (which for most countries ends up just being the country name). The widget that exposes them (GWeatherTimezoneMenu) is unwieldy, but we can come up with something better.

b) there is a Summer of Code student (Stéphane Maniaci) working on this this summer
Comment 8 Allan Day 2011-05-20 19:40:59 UTC
Subscribing Stéphane. Here's the page for the GSoC project: https://live.gnome.org/freesteph_Libgweather_GnomeShell
Comment 9 Fabio Durán Verdugo 2011-09-10 00:39:19 UTC
*** Bug 658675 has been marked as a duplicate of this bug. ***
Comment 10 Luc Pi 2011-09-10 01:02:43 UTC
What is the situation with the GSoc now? 
I can't see updates on the given links
Comment 11 Javier Jardón (IRC: jjardon) 2011-10-18 04:04:51 UTC
What is the status of this? Steph?
Comment 12 Stéphane Maniaci 2011-10-18 04:32:51 UTC
Hey!

The status of my project is failed/stalled, because I couldn't figure out design plans and understand which directions to work. The code I had is a panel in the control-center that lets you add/remove locations, and then display them in the top panel of GNOME Shell.

It's working, but it's very GNOME2-ish. There is no geo-detection from the network, or other "smart" features like we'd like to see in GNOME 3.

I would really like to work on this and finish the feature for 3.4. I will try to poke the design team sometimes during the week.
Comment 13 Gerald Pfeifer 2011-11-12 00:51:26 UTC
How comes this is only labled an enhancement when, really, this is
a clear regression from GNOME 2?
Comment 14 André Klapper 2011-11-12 04:09:44 UTC
(In reply to comment #13)
> How comes this is only labled an enhancement when, really, this is
> a clear regression from GNOME 2?

Because there is no plan to make GNOME3 provide all the features that GNOME2 had - Instead it's many of them, plus new and more important ones.
Comment 15 Allan Day 2012-07-17 10:43:56 UTC
GNOME Clocks will provide a world clock function when it is released (probably with GNOME 3.6). We could show its world clocks in the calendar drop down...
Comment 16 Florian Müllner 2012-07-17 10:49:07 UTC
(In reply to comment #15)
> We could show its world clocks in the calendar drop down...

Mockup!
Comment 17 Allan Day 2012-07-17 10:52:33 UTC
(In reply to comment #16)
> (In reply to comment #15)
> > We could show its world clocks in the calendar drop down...
> 
> Mockup!

It'd be the same as the existing mockup [1] but with 'Clocks' instead of 'Change Locations >'.

My only concern with this is that the calendar could get very tall.

[1] http://git.gnome.org/browse/gnome-shell-design/plain/mockups/static/date-n-time.png
Comment 18 Allan Day 2012-10-03 10:08:56 UTC
*** Bug 683313 has been marked as a duplicate of this bug. ***
Comment 19 Mathieu Bridon 2012-10-30 15:12:55 UTC
Now that the Clocks application is coming along, couldn't we just add a « Open Clocks » entry above/below the « Open Calendar » one?

This would keep the GNOME Shell time/calendar menu simple and clean (and prevent it from growing outrageously if I have many clocks for many locations), but still make it easy to get to all my clocks.
Comment 20 Florian Müllner 2012-11-08 17:01:29 UTC
*** Bug 674099 has been marked as a duplicate of this bug. ***
Comment 21 Matthias Clasen 2012-11-09 23:45:29 UTC
Yes, we should add "Open Clocks"
Comment 22 Mathieu Bridon 2012-11-10 01:14:57 UTC
Created attachment 228615 [details] [review]
dateMenu: Add "Open Clocks" entry
Comment 23 Mathieu Bridon 2012-11-10 01:17:02 UTC
Created attachment 228616 [details]
How it looks

I added the entry the same way the « Open Calendar » is added.

It looks quite bad I think, but I'd rather have designer input than try to do something only half better myself. :)
Comment 24 Matthias Clasen 2012-11-11 04:43:27 UTC
*** Bug 687916 has been marked as a duplicate of this bug. ***
Comment 25 Allan Day 2012-11-12 16:14:15 UTC
(In reply to comment #23)
> Created an attachment (id=228616) [details]
> How it looks
> 
> I added the entry the same way the « Open Calendar » is added.
> 
> It looks quite bad I think, but I'd rather have designer input than try to do
> something only half better myself. :)

Thanks for working on this, Mathieu! I'd suggest adding the clocks item to the other pane, above "Date and Time Settings". (We might be hiding the events pane in some situations.)
Comment 26 Mathieu Bridon 2012-11-13 12:00:02 UTC
Created attachment 228874 [details] [review]
dateMenu: Add "Open Clocks" entry
Comment 27 Mathieu Bridon 2012-11-13 12:02:08 UTC
Created attachment 228875 [details]
How it looks

This is how it looks with the new patch, based on Allan's suggestion.

Should it be placed above or below the « Date and Time Settings » entry?

Should there be an additional separator between it and the calendar widget?
Comment 28 Peter Robinson 2012-11-13 12:14:37 UTC
(In reply to comment #27)
> Created an attachment (id=228875) [details]
> How it looks
> 
> This is how it looks with the new patch, based on Allan's suggestion.

This doesn't reimplement the timezone display though. It basically gives you a link to open another application. It's not very helpful like the old implementation. If I have a meeting scheduled on one timezone or I need to call someone on another timezone and I want to quickly check what the time is in Perth when I'm in Helsinki I don't want to have to wait for an entire new app to open, interrupt my workflow only to have to quit it seconds later.
Comment 29 Mathieu Bridon 2012-11-13 13:00:32 UTC
Created attachment 228879 [details] [review]
dateMenu: Add "Open Clocks" entry

Moved the entry below the separator, as Allan suggested in #gnome-design
Comment 30 Mathieu Bridon 2012-11-13 13:01:51 UTC
Created attachment 228881 [details]
How it looks

New screenshot with the latest patch.
Comment 31 Luc Pi 2012-11-13 13:50:49 UTC
Of course there are various logic to read this dialogue.

For one of them, 
you may consider aligning "Open Clocks" and "Open Calendar"

 
 |                         |                         |
 | Date and time sett.     |                         |
 |                         |                         |
 | Open Clocks             | Open Calendar           |
 +-------------------------+-------------------------+

it would ease to reach the button "Open Clocks", which should be a significantly more frequent use case than "Date and time settings"


and maybe move settings under a Settings icon

 |                         |                         |
 |                         |                         |
 |                         |                         |
 | Open Clocks             | Open Calendar        (*)|
 +-------------------------+-------------------------+


a suggestion
Comment 32 Matthias Clasen 2012-11-19 03:52:18 UTC
The settings menuitem is supposed to be at the very bottom to parallel the other status menus, which all have a settings menuitem at the bottom.
Comment 33 Allan Day 2012-12-14 10:19:23 UTC
(In reply to comment #30)
> Created an attachment (id=228881) [details]
> How it looks
> 
> New screenshot with the latest patch.

Sorry Mathieu, I dropped the ball on this one. My suggestion would be to put all the "open" menu items in the left-hand pane, since having two items on one side and one on the other will look unbalanced.

 |                         |                         |
 | Open Calendar           |                         |
 |                         |                         |
 | Open Clocks             |                         |
 |                         |                         |
 | Date and Time Settings  |                         |
 +-------------------------+-------------------------+

This approach will also mean that the calendar will still be accessible after bug 680083 is resolved.
Comment 34 Mathieu Bridon 2012-12-14 10:59:59 UTC
There's a separator currently between the calendar and the « Date and Time Settings » entry.

Should that separator now be between the calendar and the « Open Calendar » entry?

Should there be other separators? (e.g between « Open Clocks » and « Date and Time Settings »)
Comment 35 Allan Day 2012-12-14 11:03:44 UTC
(In reply to comment #34)
...

I'd keep the single separator between the three menu items and the calendar grid:

 | |                     | |                         |
 | +---------------------+ |                         |
 |                         |                         |
 |  ---------------------  |                         |
 |                         |                         |
 | Open Calendar           |                         |
 |                         |                         |
 | Open Clocks             |                         |
 |                         |                         |
 | Date and Time Settings  |                         |
 +-------------------------+-------------------------+
Comment 36 Mathieu Bridon 2012-12-15 10:40:25 UTC
Created attachment 231617 [details] [review]
dateMenu: Add "Open Clocks" entry

This implements the changes requested by Allan in comment 35.
Comment 37 Gerry T. 2012-12-15 12:37:23 UTC
That's in fact great news! I am working constantly with people in four/five time zones and this feature was badly missing since the introduction of Gnome 3. I am glad that it will be back. 

Will it be integrated in Gnome 3.8? I hope so?!?
Comment 38 Nguyen Thai Ngoc Duy 2012-12-16 04:44:45 UTC
I think the concern in comment #28 is still valid. Is it decided to drop the extra clocks in calendar panel?
Comment 39 Giovanni Campagna 2012-12-19 13:58:55 UTC
Review of attachment 231617 [details] [review]:

source

::: js/ui/dateMenu.js
@@ +101,3 @@
+        this._openClocksItem.connect('activate', Lang.bind(this, this._onOpenClocksActivate));
+        this._openClocksItem.actor.can_focus = false;
+

Given that gnome-clocks is not a core application, it would be very nice if you checked for its existence and hide the launcher if not installed.
Otherwise, you need to check for the return value of lookup_app and show a failure notification.
Comment 40 Mathieu Bridon 2012-12-19 21:24:59 UTC
Created attachment 231937 [details] [review]
dateMenu: Add "Open Clocks" entry

(In reply to comment #39)
> Review of attachment 231617 [details] [review]:
> Given that gnome-clocks is not a core application, it would be very nice if you
> checked for its existence and hide the launcher if not installed.

New patch does that.

Was that the right way to check for it?
Comment 41 Giovanni Campagna 2012-12-20 21:52:39 UTC
Review of attachment 231937 [details] [review]:

source

::: js/ui/dateMenu.js
@@ +98,3 @@
+        this._calendarSettingsChanged();
+
+        this._openCalendarItem.actor.can_focus = false;

This is the right way to check if an application exists, but it doesn't work if you install gnome-clocks, until the next logout.
You need to connect to installed-changed, and check again.
Comment 42 Mathieu Bridon 2012-12-21 22:30:01 UTC
Created attachment 232097 [details] [review]
dateMenu: Move the "Open Calendar" entry

This is the part of the previous patch which moves the « Open Calendar » item.

I thought separating it to its own patch would make the review easier. (at least it made further changes of the rest easier for me)
Comment 43 Mathieu Bridon 2012-12-21 22:31:50 UTC
Created attachment 232098 [details] [review]
dateMenu: Add "Open Clocks" entry

(In reply to comment #41)
> This is the right way to check if an application exists, but it doesn't work if
> you install gnome-clocks, until the next logout.
> You need to connect to installed-changed, and check again.

This is done with this version of the patch.

It also removes the « Open Clocks » entry if gnome-clocks is uninstalled.
Comment 44 Jasper St. Pierre (not reading bugmail) 2013-01-15 19:55:54 UTC
Review of attachment 232097 [details] [review]:

Looks OK to me
Comment 45 Jasper St. Pierre (not reading bugmail) 2013-01-15 20:00:50 UTC
Review of attachment 232098 [details] [review]:

OK.
Comment 46 Mathieu Bridon 2013-01-16 03:00:06 UTC
Created attachment 233561 [details] [review]
[PATCH 1/2] dateMenu: Move the "Open Calendar" entry

The previous two patches didn't apply on master any more, so I'm rebasing them.
Comment 47 Mathieu Bridon 2013-01-16 03:01:29 UTC
Created attachment 233562 [details] [review]
dateMenu: Add "Open Clocks" entry

As for the other patch, this is just a rebase on master.
Comment 48 Zeeshan Ali 2013-01-16 03:09:34 UTC
Patches pushed on behalf of Mathieu Bridon.
Comment 49 Allan Day 2014-11-07 14:58:36 UTC
I think we've taken this as far as we can for now. We might re-examine in the future, but we don't need the bug report for that.