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 641049 - Calendar: First day of week issues
Calendar: First day of week issues
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2011-01-31 16:58 UTC by David Zeuthen (not reading bugmail)
Modified: 2011-01-31 20:58 UTC
See Also:
GNOME target: 3.0
GNOME version: ---


Attachments
calendar: Fix event list for week starts other than Sunday (3.28 KB, patch)
2011-01-31 19:59 UTC, Florian Müllner
committed Details | Review

Description David Zeuthen (not reading bugmail) 2011-01-31 16:58:01 UTC
The calendar currently does not deal properly with first day of week not being a Sunday.

See bug 632109 comment 57 for details.
Comment 1 Florian Müllner 2011-01-31 19:59:20 UTC
Created attachment 179743 [details] [review]
calendar: Fix event list for week starts other than Sunday

In non-US locales, Monday is generally considered the first day
of the week. Take this into account when building the event
lists displayed under "This week"/"Next week".
Comment 2 David Zeuthen (not reading bugmail) 2011-01-31 20:33:17 UTC
Btw, there's also the question of how the grid is displayed. For example, if the week starts on a Monday, should we instead be showing

 M T W T F S S

Jon, Jimmac?
Comment 3 Florian Müllner 2011-01-31 20:36:46 UTC
(In reply to comment #2)
> Btw, there's also the question of how the grid is displayed. For example, if
> the week starts on a Monday, should we instead be showing
> 
>  M T W T F S S

Yes - we already do this (assuming that with "grid" you mean the calendar itself)
Comment 4 David Zeuthen (not reading bugmail) 2011-01-31 20:42:54 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > Btw, there's also the question of how the grid is displayed. For example, if
> > the week starts on a Monday, should we instead be showing
> > 
> >  M T W T F S S
> 
> Yes - we already do this (assuming that with "grid" you mean the calendar
> itself)

Oh. I think I broke this with my css-collapse hack

 http://people.freedesktop.org/~david/calendar-grid-non-US-locale-collapse-bug.png
Comment 5 David Zeuthen (not reading bugmail) 2011-01-31 20:48:15 UTC
(In reply to comment #1)
> Created an attachment (id=179743) [details] [review]
> calendar: Fix event list for week starts other than Sunday
> 
> In non-US locales, Monday is generally considered the first day
> of the week. Take this into account when building the event
> lists displayed under "This week"/"Next week".

Looks good to me. Committed. Thanks!

(I'll keep the bug open until the issue pointed out in comment 4 is fixed. I'm working on that right now.)
Comment 6 David Zeuthen (not reading bugmail) 2011-01-31 20:48:49 UTC
Comment on attachment 179743 [details] [review]
calendar: Fix event list for week starts other than Sunday

Committed as d6749589e8c960266f434cb12f5709fa0fbf99ad
Comment 7 David Zeuthen (not reading bugmail) 2011-01-31 20:58:03 UTC
The grid was fixed with this commit:

 http://git.gnome.org/browse/gnome-shell/commit/?id=25434e42d0d933f12fcc2a6436407218e5496268

I guess we can close this bug now. Thanks.