Indicator Location

Merge lp:~ted/indicator-location/add-title into lp:indicator-location/13.10

Proposed by Ted Gould on 2013-09-11

Status:

Merged

Approved by:

Charles Kerr on 2013-09-11

Approved revision:

Merged at revision:

Proposed branch:

Merge into:

Diff against target:

1 file modified

src/phone.cc (+3/-0)

To merge this branch:

bzr merge lp:~ted/indicator-location/add-title

Related bugs:

High

Fix Released

Link a bug report

Reviewer	Review Type	Date Requested	Status
Charles Kerr (community)		2013-09-11	Approve on 2013-09-11
PS Jenkins bot (community)	continuous-integration		Approve on 2013-09-11
Review via email: mp+184913@code.launchpad.net

Add a title to the root action state

Very exciting merge!

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-09-11:

review: Approve (continuous-integration)

Revision history for this message

Charles Kerr (charlesk) on 2013-09-11:

review: Approve

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk

People subscribed via source and target branches

to all changes:

1	=== modified file 'src/phone.cc'
2	--- src/phone.cc 2013-08-26 21:38:43 +0000
3	+++ src/phone.cc 2013-09-11 02:49:14 +0000
4	@@ -77,6 +77,9 @@
5	const char * a11y = _("Location");
6	g_variant_builder_add (&builder, "{sv}", "accessible-desc", g_variant_new_string (a11y));
7
8	+ const char * title = _("Location");
9	+ g_variant_builder_add (&builder, "{sv}", "title", g_variant_new_string (title));
10	+
11	gboolean visible = should_be_visible ();
12	g_variant_builder_add (&builder, "{sv}", "visible", g_variant_new_boolean (visible));
13