Merge lp:~marcustomlinson/indicator-datetime/lp-1236400 into lp:~indicator-applet-developers/indicator-datetime/trunk.13.10

Proposed by Marcus Tomlinson
Status: Merged
Approved by: Francis Ginther
Approved revision: 276
Merged at revision: 276
Proposed branch: lp:~marcustomlinson/indicator-datetime/lp-1236400
Merge into: lp:~indicator-applet-developers/indicator-datetime/trunk.13.10
Diff against target: 33 lines (+3/-6)
1 file modified
src/service.c (+3/-6)
To merge this branch: bzr merge lp:~marcustomlinson/indicator-datetime/lp-1236400
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Charles Kerr (community) Approve
Review via email: mp+191415@code.launchpad.net

Commit message

Fixed datetime title text to read "Date and Time" rather than the day of the week (as per spec).

To post a comment you must log in.
Revision history for this message
Charles Kerr (charlesk) wrote :

Looks good! Thanks for adding the weekday to the calendar menuitem.

review: Approve
275. By Marcus Tomlinson

Fixed date formatted string menu item to display the full name of current day rather than the shortend version.

Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Autolanding.
Approved revid is not set in launchpad. This is most likely a launchpad issue and re-approve should fix it. There is also a chance (although a very small one) this is a permission problem of the ps-jenkins bot.
http://jenkins.qa.ubuntu.com/job/indicator-datetime-autolanding/53/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-datetime-saucy-amd64-autolanding/47
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-datetime-saucy-armhf-autolanding/47
        deb: http://jenkins.qa.ubuntu.com/job/indicator-datetime-saucy-armhf-autolanding/47/artifact/work/output/*zip*/output.zip

review: Needs Fixing (continuous-integration)
276. By Marcus Tomlinson

Replace "Date and Time" menu title on phone with "Upcoming" according to updated spec.

Revision history for this message
Charles Kerr (charlesk) wrote :

Re-approving for Jenkins :)

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/service.c'
--- src/service.c 2013-10-16 14:50:25 +0000
+++ src/service.c 2013-10-16 16:35:28 +0000
@@ -522,7 +522,7 @@
522 gchar * label;522 gchar * label;
523 gboolean has_alarms;523 gboolean has_alarms;
524 gchar * a11y;524 gchar * a11y;
525 gchar * title;525 const gchar * title = _("Upcoming");
526526
527 g_variant_builder_init (&b, G_VARIANT_TYPE_VARDICT);527 g_variant_builder_init (&b, G_VARIANT_TYPE_VARDICT);
528528
@@ -550,10 +550,7 @@
550550
551 g_variant_builder_add (&b, "{sv}", "visible", g_variant_new_boolean (TRUE));551 g_variant_builder_add (&b, "{sv}", "visible", g_variant_new_boolean (TRUE));
552 g_variant_builder_add (&b, "{sv}", "label", g_variant_new_take_string (label));552 g_variant_builder_add (&b, "{sv}", "label", g_variant_new_take_string (label));
553553 g_variant_builder_add (&b, "{sv}", "title", g_variant_new_string (title));
554 /* title is day-of-week */
555 title = g_date_time_format (now, _("%A"));
556 g_variant_builder_add (&b, "{sv}", "title", g_variant_new_take_string (title));
557554
558 /* cleanup */555 /* cleanup */
559 g_date_time_unref (now);556 g_date_time_unref (now);
@@ -686,7 +683,7 @@
686 GMenu * menu = g_menu_new ();683 GMenu * menu = g_menu_new ();
687684
688 /* strftime(3) format string to show date */685 /* strftime(3) format string to show date */
689 add_localtime_menuitem (menu, self, _("%e %B %Y"), "calendar");686 add_localtime_menuitem (menu, self, _("%A, %e %B %Y"), "calendar");
690687
691 return G_MENU_MODEL (menu);688 return G_MENU_MODEL (menu);
692}689}

Subscribers

People subscribed via source and target branches