Timezones not in time order
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| Ayatana Design |
Medium
|
Matthew Paul Thomas | ||
| Indicator Date and Time |
Fix Released
|
Medium
|
Charles Kerr | |
| indicator-datetime (Ubuntu) |
Medium
|
Unassigned |
Bug Description
They used to be but not anymore.
Related branches
- Ted Gould (community): Approve on 2012-03-03
-
Diff: 406 lines (+141/-188)1 file modifiedsrc/datetime-service.c (+141/-188)
- Ken VanDine: Pending requested 2012-04-04
-
Diff: 777 lines (+263/-171)13 files modifiedChangeLog (+28/-0)
Makefile.am (+1/-35)
Makefile.am.coverage (+48/-0)
Makefile.in (+47/-41)
configure (+100/-64)
configure.ac (+4/-2)
data/Makefile.in (+0/-1)
debian/changelog (+10/-0)
m4/gcov.m4 (+13/-10)
src/Makefile.in (+0/-1)
src/datetime-prefs.c (+1/-8)
src/datetime-service.c (+11/-8)
tests/Makefile.in (+0/-1)
James Haigh (james.r.haigh) wrote : | #1 |
James Haigh (james.r.haigh) wrote : | #2 |
tags: | added: regression-release |
Changed in indicator-datetime: | |
status: | New → Confirmed |
importance: | Undecided → Medium |
Changed in indicator-datetime: | |
status: | Confirmed → In Progress |
Charles Kerr (charlesk) wrote : | #3 |
James, just to make sure I'm understanding this ticket correctly -- you're referring to the order of the times shown inbetween "Add Event..." and "Time & Date Settings..." in your screenshot, is that correct?
Changed in indicator-datetime: | |
status: | In Progress → Incomplete |
Raphaël Badin (rvb) wrote : | #4 |
Same pb here (Precise), I think James was indeed referring to the order of the times shown inbetween "Add Event..." and "Time & Date Settings...".
It is:
- UTC 19:54
- Paris 21:54
- London 20:54
and it should be (and it used to be):
- UTC 19:54
- London 20:54
- Paris 21:54
Changed in indicator-datetime: | |
status: | Incomplete → In Progress |
Good stuff, Charles!
Changed in indicator-datetime: | |
status: | In Progress → Fix Committed |
Changed in ayatana-design: | |
status: | New → Incomplete |
Matthew Paul Thomas (mpt) wrote : | #6 |
The design is not Incomplete; it has been specified since July 2010. "The order of locations in the menu should always match the order of locations in the window ... Clicking the “Location” header cell should sort the locations alphabetically, and clicking it again should sort them reverse alphabetically. Clicking the “Time” header cell should sort the locations earliest first, and clicking it again should sort them latest first." <https:/
The first part works in indicator-datetime 0.3.90-0ubuntu1: the menu updates live to match whatever order you specify in the window. If the "/* sort the list by timezone offset */" code changes this, that's a regression. Sorting by timezone offset should happen only if you click the "Time" column header, which in 0.3.90-0ubuntu1 does nothing.
Changed in ayatana-design: | |
assignee: | nobody → Matthew Paul Thomas (mpt) |
status: | Incomplete → Invalid |
Charles Kerr (charlesk) wrote : | #7 |
mpt: I think it was marked as incomplete this ticket is about sorting the locations inbetween "Add Event..." and "Time & Date Settings..." in the menu, not about the Locations window.
The design currently says the locations should be listed, but doesn't address order: "Whenever “Time in other locations” is checked, the times in the chosen locations should be present in the menu. Activating any of these items should change the time zone to that location (with the authorization dialog acting as a confirmation step). " <https:/
Changed in ayatana-design: | |
status: | Invalid → Incomplete |
Charles Kerr (charlesk) wrote : | #8 |
Ah, I read through comment #6 too quickly.
Matthew Paul Thomas (mpt) wrote : | #9 |
Following our discussion Tuesday and yesterday, I've updated the specification to include "Sort by Name" and "Sort by Time" buttons in the "Locations" window. <https:/
Changed in ayatana-design: | |
status: | Incomplete → Triaged |
Changed in indicator-datetime: | |
milestone: | none → 0.3.91 |
status: | Fix Committed → Fix Released |
Charles Kerr (charlesk) wrote : | #10 |
Re-opening as per the refined spec
Changed in indicator-datetime: | |
assignee: | nobody → charles (charlesk) |
status: | Fix Released → Triaged |
Olli Ries (ories) wrote : | #11 |
MPT, re comment #6, I meant to express that we'd need some input from Design - seems we have everything needed. Charles will attach a screenshot for design review.
Charles Kerr (charlesk) wrote : | #12 |
Charles Kerr (charlesk) wrote : | #13 |
tags: | added: udp |
Changed in ayatana-design: | |
importance: | Undecided → Medium |
Matthew Paul Thomas (mpt) wrote : | #14 |
Charles, that looks pretty good, thank you. Just two minor glitches:
(1) There should be as much padding to the left of the "+" button as there is above and below it.
(2) When sorted by time, Budapest should go before Paris, because they're the same time but Budapest is alphabetically earlier. I've added this to the spec. <https:/
Changed in ayatana-design: | |
status: | Triaged → Fix Committed |
Changed in indicator-datetime (Ubuntu): | |
status: | New → Triaged |
importance: | Undecided → Medium |
Changed in indicator-datetime: | |
status: | Triaged → Fix Released |
Changed in indicator-datetime (Ubuntu): | |
status: | Triaged → Fix Released |
Matthew Paul Thomas (mpt) wrote : | #15 |
Tested with indicator-datetime 12.10.2-0ubuntu1 in Ubuntu Q. This all seems to be working now.
Changed in ayatana-design: | |
status: | Fix Committed → Fix Released |
tags: | removed: udp |
Subtask of bug #823181.