Privacy Manager for Zeitgeist

Merge lp:~malizor/activity-log-manager/fix-967150 into lp:~activity-log-manager/activity-log-manager/vala

Proposed by Nicolas Delvaux on 2012-09-20
Status: Merged
Merged at revision: 126
Proposed branch: lp:~malizor/activity-log-manager/fix-967150
Merge into: lp:~activity-log-manager/activity-log-manager/vala
Diff against target: 11 lines (+1/-1) 1 file modified
To merge this branch: bzr merge lp:~malizor/activity-log-manager/fix-967150
Reviewer Review Type Date Requested Status
Didier Roche 2012-09-20 Approve on 2012-12-07
Review via email: mp+125556@code.launchpad.net

Description of the Change

The "Forget activities" string was separated from its colon.
Because of a Rosetta bug, this led to bad translations (eg. in French, impossible to add a space between the string and the colon).

This patch has zero regression potential, but it creates a new string.
So this means translations have to be updated accordingly, which is the point of the related bug anyway (to be able to translate this string properly).

To post a comment you must log in.
Didier Roche (didrocks) wrote :

Looking good, Thanks Nicolas :)

Better to merge now so that on next release, people will have time to translate it.

review: Approve
Nicolas Delvaux (malizor) wrote :

This fix was merged more than 6 months ago and and it has yet to reach any Ubuntu version.
I understand doing a release only for such a small fix is not really exciting, but what is the point of writing patches if they may never reach users?

So, if no more work is planned on Activity Log Manager for Saucy, could someone just release a new package?

Thanks!

I will have a look at this patch and make sure it makes it on 0.9.7 release in maybe a week. Thanks for bringing it to my notice.

At the moment the focus of development is a different branch, but I will apply this patch to the focus of development branch

Nicolas Delvaux (malizor) wrote :

Thanks Manish, let me know if I need to submit a new merge request against any other branch.

It's fine. I can just merge it directly if ubuntu focus of development branch does not have this patch applied

Preview Diff

1=== modified file 'src/history-widget.vala'
2--- src/history-widget.vala 2012-03-18 09:11:52 +0000
3+++ src/history-widget.vala 2012-09-20 17:37:35 +0000
4@@ -58,7 +58,7 @@
5
6 var text_box = new Box(orientation=Orientation.VERTICAL, 0);
7 var header = new Label("");
8- header.set_markup("<b>%s:</b>".printf(_("Forget activities")));
9+ header.set_markup("<b>%s</b>".printf(_("Forget activities:")));
10 header.set_alignment(0, (float)0.5);
11 header.set_padding(0, 0);
12

Subscribers

People subscribed via source and target branches