Changeset 520be1

Timestamp:

06/10/2009 10:50:57 PM (16 years ago)

Author:

Andrew Resch <andrewresch@gmail.com>

Branches:

2.0.x, develop, extjs4-port, master

Children:

ca7f33d

Parents:

e36ca0

Message:

Fix turning off Classic mode dialog
Fix showing the connection manager icon on the toolbar depending on classic mode setting

Location:

deluge/ui/gtkui

Files:

• gtkui.py (modified) (2 diffs)
• toolbar.py (modified) (3 diffs)

deluge/ui/gtkui/gtkui.py

@@ -238 +238 @@
                 client.start_classic_mode()
             except deluge.error.DaemonRunningError:
-                response = dialogs.YesNoDialog(
+                d = dialogs.YesNoDialog(
                     _("Turn off Classic Mode?"),
                     _("It appears that a Deluge daemon process (deluged) is already running.\n\n\
@@ -244 +244 @@
 
                 self.started_in_classic = False
-                if response != gtk.RESPONSE_YES:
-                    # The user does not want to turn Classic Mode off, so just quit
-                    reactor.stop()
-                    return
-                # Turning off classic_mode
-                self.config["classic_mode"] = False
+                def on_dialog_response(response):
+                    if response != gtk.RESPONSE_YES:
+                        # The user does not want to turn Classic Mode off, so just quit
+                        reactor.stop()
+                        return
+                    # Turning off classic_mode
+                    self.config["classic_mode"] = False
+                    self.__start_non_classic()
+
+                d.addCallback(on_dialog_response)
             else:
                 component.start()
                 return
 
-        # Autoconnect to a host
-        if self.config["autoconnect"]:
-            for host in self.connectionmanager.config["hosts"]:
-                if host[0] == self.config["autoconnect_host_id"]:
-                    def on_connect(connector):
-                        component.start()
-                    client.connect(*host[1:]).addCallback(on_connect)
-
-        if self.config["show_connection_manager_on_start"]:
-            # XXX: We need to call a simulate() here, but this could be a bug in twisted
-            reactor.simulate()
-            self.connectionmanager.show()
+        else:
+            self.__start_non_classic()
+
+    def __start_non_classic(self):
+            # Autoconnect to a host
+            if self.config["autoconnect"]:
+                for host in self.connectionmanager.config["hosts"]:
+                    if host[0] == self.config["autoconnect_host_id"]:
+                        def on_connect(connector):
+                            component.start()
+                        client.connect(*host[1:]).addCallback(on_connect)
+
+            if self.config["show_connection_manager_on_start"]:
+                # XXX: We need to call a simulate() here, but this could be a bug in twisted
+                reactor.simulate()
+                self.connectionmanager.show()
 
 

deluge/ui/gtkui/toolbar.py

@@ -75 +75 @@
         ]
 
-        if self.config["classic_mode"]:
-            self.window.main_glade.get_widget("toolbutton_connectionmanager").hide()
+        self.config.register_set_function("classic_mode", self._on_classic_mode, True)
 
         # Hide if necessary
@@ -82 +81 @@
 
     def start(self):
+        if not self.config["classic_mode"]:
+            self.window.main_glade.get_widget("toolbutton_connectionmanager").show()
+
         for widget in self.change_sensitivity:
             self.window.main_glade.get_widget(widget).set_sensitive(True)
@@ -177 +179 @@
         log.debug("on_toolbutton_queue_down_clicked")
         component.get("MenuBar").on_menuitem_queue_down_activate(data)
+
+    def _on_classic_mode(self, key, value):
+        w = self.window.main_glade.get_widget("toolbutton_connectionmanager")
+        if value:
+            w.hide()
+        else:
+            w.show()