Merge lp:~mterry/update-manager/avail-cleanups into lp:update-manager

Proposed by Michael Terry
Status: Merged
Approved by: Didier Roche-Tolomelli
Approved revision: 2453
Merged at revision: 2508
Proposed branch: lp:~mterry/update-manager/avail-cleanups
Merge into: lp:update-manager
Prerequisite: lp:~mterry/update-manager/dialogs
Diff against target: 71 lines (+25/-2)
2 files modified
UpdateManager/UpdatesAvailable.py (+23/-0)
data/gtkbuilder/UpdateManager.ui (+2/-2)
To merge this branch: bzr merge lp:~mterry/update-manager/avail-cleanups
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
Michael Vogt Pending
Review via email: mp+110914@code.launchpad.net

Description of the change

A couple spec-requested changes to the available-updates dialog.

Specifically, "Description" -> "Technical description" and changing the "Close" button to either a "Cancel" or "Remind Me Later" button depending on whether automatically checking for updates is enabled.

To post a comment you must log in.
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Looks safe and straightforward to me. Approving :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'UpdateManager/UpdatesAvailable.py'
2--- UpdateManager/UpdatesAvailable.py 2012-06-18 21:20:24 +0000
3+++ UpdateManager/UpdatesAvailable.py 2012-06-18 21:20:25 +0000
4@@ -115,6 +115,11 @@
5 # useful exit stuff
6 self.button_close.connect("clicked", lambda w: self.window_main.exit())
7
8+ # If auto-updates are on, change cancel label
9+ self.notifier_settings = Gio.Settings("com.ubuntu.update-notifier")
10+ self.notifier_settings.connect("changed::auto-launch", lambda s, p: self.update_close_button())
11+ self.update_close_button()
12+
13 # the treeview (move into it's own code!)
14 self.store = Gtk.ListStore(str, str, GObject.TYPE_PYOBJECT,
15 GObject.TYPE_PYOBJECT, bool)
16@@ -171,6 +176,21 @@
17 self.alert_watcher.connect("battery-alert", self._on_battery_alert)
18 self.alert_watcher.connect("network-3g-alert", self._on_network_3g_alert)
19
20+ def is_auto_update(self):
21+ update_days = apt_pkg.config.find_i("APT::Periodic::Update-Package-Lists")
22+ auto_launch = self.notifier_settings.get_boolean("auto-launch")
23+ return update_days >= 1 and auto_launch
24+
25+ def update_close_button(self):
26+ if self.is_auto_update():
27+ self.button_close.set_label(_("_Remind Me Later"))
28+ self.button_close.set_use_stock(False)
29+ self.button_close.set_use_underline(True)
30+ else:
31+ self.button_close.set_label(Gtk.STOCK_CANCEL)
32+ self.button_close.set_use_stock(True)
33+ self.button_close.set_use_underline(False)
34+
35 def install_all_updates (self, menu, menuitem, data):
36 self.select_all_updgrades (None)
37 self.on_button_install_clicked (None)
38@@ -487,8 +507,11 @@
39 while Gtk.events_pending():
40 Gtk.main_iteration()
41 time.sleep(0.05)
42+ apt_pkg.init_config()
43 self.window_main.refresh_cache()
44 self.fillstore()
45+ self.update_close_button()
46+ self.window_main.set_sensitive(True)
47
48 def on_button_install_clicked(self, widget):
49 #print("on_button_install_clicked")
50
51=== modified file 'data/gtkbuilder/UpdateManager.ui'
52--- data/gtkbuilder/UpdateManager.ui 2012-06-18 21:20:24 +0000
53+++ data/gtkbuilder/UpdateManager.ui 2012-06-18 21:20:25 +0000
54@@ -198,7 +198,7 @@
55 <object class="GtkLabel" id="label13">
56 <property name="visible">True</property>
57 <property name="can_focus">False</property>
58- <property name="label" translatable="yes">Description</property>
59+ <property name="label" translatable="yes">Technical description</property>
60 </object>
61 </child>
62 </object>
63@@ -454,7 +454,7 @@
64 </child>
65 <child>
66 <object class="GtkButton" id="button_close">
67- <property name="label">gtk-close</property>
68+ <property name="label">gtk-cancel</property>
69 <property name="use_action_appearance">False</property>
70 <property name="visible">True</property>
71 <property name="can_focus">True</property>

Subscribers

People subscribed via source and target branches

to status/vote changes: