Merge lp:~robert-ancell/unity-greeter/bug-844044 into lp:unity-greeter

Proposed by Robert Ancell on 2012-03-09
Status: Merged
Merged at revision: 345
Proposed branch: lp:~robert-ancell/unity-greeter/bug-844044
Merge into: lp:unity-greeter
Diff against target: 177 lines (+62/-20)
5 files modified
NEWS (+1/-0)
configure.ac (+1/-1)
src/fixes.vapi (+3/-1)
src/unity-greeter.vala (+27/-16)
src/user-list.vala (+30/-2)
To merge this branch: bzr merge lp:~robert-ancell/unity-greeter/bug-844044
Reviewer Review Type Date Requested Status
Michael Terry (community) 2012-03-09 Approve on 2012-03-09
Review via email: mp+96704@code.launchpad.net

Description of the Change

Requires lightdm 1.1.7

To post a comment you must log in.
343. By Robert Ancell on 2012-03-09

Depend on correct version of liblightdm-gobject-1

Michael Terry (mterry) wrote :

Looks good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'NEWS'
2--- NEWS 2012-03-08 04:15:45 +0000
3+++ NEWS 2012-03-09 04:48:17 +0000
4@@ -4,6 +4,7 @@
5 * Show manual entry when no users available
6 * Fix prompt entry text not showing in some themes
7 * Fix Orca command line so it works with version 3.3.90+
8+ * Show manual login option if hint is present
9
10 Overview of changes in unity-greeter 0.2.4
11
12
13=== modified file 'configure.ac'
14--- configure.ac 2012-02-23 13:35:53 +0000
15+++ configure.ac 2012-03-09 04:48:17 +0000
16@@ -22,7 +22,7 @@
17 gtk+-3.0
18 gdk-x11-3.0
19 indicator3-0.4
20- liblightdm-gobject-1 >= 1.1.3
21+ liblightdm-gobject-1 >= 1.1.7
22 freetype2
23 cairo-ft
24 libcanberra
25
26=== modified file 'src/fixes.vapi'
27--- src/fixes.vapi 2012-03-07 04:52:42 +0000
28+++ src/fixes.vapi 2012-03-09 04:48:17 +0000
29@@ -28,8 +28,10 @@
30 public const int KEY_0;
31 public const int KEY_g;
32 public const int KEY_G;
33+ public const int KEY_h;
34+ public const int KEY_m;
35+ public const int KEY_M;
36 public const int KEY_s;
37- public const int KEY_h;
38 public const int KEY_plus;
39 public const int KEY_minus;
40 public const int KEY_equal;
41
42=== modified file 'src/unity-greeter.vala'
43--- src/unity-greeter.vala 2012-03-08 03:15:53 +0000
44+++ src/unity-greeter.vala 2012-03-09 04:48:17 +0000
45@@ -153,7 +153,7 @@
46 }
47
48 while (add_test_entry ());
49- user_list.add_entry ("*guest", _("Guest Session"));
50+ user_list.offer_guest = true;
51
52 main_window.key_press_event.connect (key_press_cb);
53
54@@ -163,7 +163,7 @@
55 else
56 {
57 user_list.default_session = greeter.default_session_hint;
58-
59+ user_list.always_show_manual = greeter.show_manual_login_hint;
60 if (!greeter.hide_users_hint)
61 {
62 var users = LightDM.UserList.get_instance ();
63@@ -177,7 +177,7 @@
64 if (greeter.has_guest_account_hint)
65 {
66 debug ("Adding guest account entry");
67- user_list.add_entry ("*guest", _("Guest Session"));
68+ user_list.offer_guest = true;
69 }
70
71 if (greeter.select_user_hint != null)
72@@ -200,24 +200,35 @@
73 if ((event.state & Gdk.CONTROL_MASK) == 0)
74 return false;
75
76- if (event.keyval == Gdk.KEY_plus)
77+ switch (event.keyval)
78+ {
79+ case Gdk.KEY_plus:
80 add_test_entry ();
81- if (event.keyval == Gdk.KEY_minus)
82+ break;
83+ case Gdk.KEY_minus:
84 remove_test_entry ();
85- if (event.keyval == Gdk.KEY_0)
86- {
87+ break;
88+ case Gdk.KEY_0:
89 while (remove_test_entry ());
90- user_list.remove_entry ("*guest");
91- }
92- if (event.keyval == Gdk.KEY_equal)
93- {
94+ user_list.offer_guest = false;
95+ break;
96+ case Gdk.KEY_equal:
97 while (add_test_entry ());
98- user_list.add_entry ("*guest", _("Guest Session"));
99+ user_list.offer_guest = true;
100+ break;
101+ case Gdk.KEY_g:
102+ user_list.offer_guest = false;
103+ break;
104+ case Gdk.KEY_G:
105+ user_list.offer_guest = true;
106+ break;
107+ case Gdk.KEY_m:
108+ user_list.always_show_manual = false;
109+ break;
110+ case Gdk.KEY_M:
111+ user_list.always_show_manual = true;
112+ break;
113 }
114- if (event.keyval == Gdk.KEY_g)
115- user_list.remove_entry ("*guest");
116- if (event.keyval == Gdk.KEY_G)
117- user_list.add_entry ("*guest", _("Guest Session"));
118
119 return false;
120 }
121
122=== modified file 'src/user-list.vala'
123--- src/user-list.vala 2012-03-07 04:52:42 +0000
124+++ src/user-list.vala 2012-03-09 04:48:17 +0000
125@@ -53,6 +53,34 @@
126 public MenuBar menubar;
127 public UserEntry? selected_entry {get; private set; default = null;}
128
129+ private bool _offer_guest = false;
130+ public bool offer_guest
131+ {
132+ get { return _offer_guest; }
133+ set
134+ {
135+ _offer_guest = value;
136+ if (value)
137+ add_entry ("*guest", _("Guest Session"));
138+ else
139+ remove_entry ("*guest");
140+ }
141+ }
142+
143+ private bool _always_show_manual = false;
144+ public bool always_show_manual
145+ {
146+ get { return _always_show_manual; }
147+ set
148+ {
149+ _always_show_manual = value;
150+ if (value)
151+ add_manual_entry ();
152+ else if (have_users ())
153+ remove_entry ("*other");
154+ }
155+ }
156+
157 private List<UserEntry> entries = null;
158
159 private Gdk.Pixbuf message_pixbuf;
160@@ -384,7 +412,7 @@
161 select_entry (selected_entry, 1.0);
162
163 /* Remove manual option when have users */
164- if (have_users ())
165+ if (have_users () && !always_show_manual)
166 remove_entry ("*other");
167
168 redraw_user_list ();
169@@ -420,7 +448,7 @@
170 }
171 return false;
172 }
173-
174+
175 private void add_manual_entry ()
176 {
177 var text = manual_username;

Subscribers

People subscribed via source and target branches