Indicator Date and Time

Merge lp:~jjardon/indicator-datetime/fix-860586 into lp:indicator-datetime/0.3

  1. fix-860586
  2. Merge into trunk
Proposed by Javier Jardón
Status: Work in progress
Proposed branch: lp:~jjardon/indicator-datetime/fix-860586
Merge into: lp:indicator-datetime/0.3
Diff against target: 12 lines (+1/-1)
1 file modified
src/datetime-prefs.c (+1/-1)
To merge this branch: bzr merge lp:~jjardon/indicator-datetime/fix-860586
Reviewer Review Type Date Requested Status
Charles Kerr (community) Needs Information
Review via email: mp+79421@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Charles Kerr (charlesk) wrote :

Javier, could you walk me though how this patch fixes #860586? I'd like to understand it but don't see the connection -- the OP's report doesn't seem to relate to the date field at all.

review: Needs Information
Reply

Unmerged revisions

142. By Javier Jardón

datetime-prefs: Use the ISO 8601 date format in the preferences

So we can parse the contens correctly

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/datetime-prefs.c'
2--- src/datetime-prefs.c 2011-09-09 09:47:20 +0000
3+++ src/datetime-prefs.c 2011-10-14 16:38:23 +0000
4@@ -478,7 +478,7 @@
5 }
6 }
7 else {
8- format = "%x";
9+ format = "%F";
10 }
11
12 GDateTime * datetime = g_date_time_new_from_unix_local (gtk_spin_button_get_value (GTK_SPIN_BUTTON (spinner)));

Subscribers

People subscribed via source and target branches