[PATCH] virt-manager: populate_device_list on media_toggled()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,
otherwise we fill up the device list when the physical device combo box
gets deactivated - also select a default entry since the combo box
othewise looks empty.
Cheers,
 -- Guido
# HG changeset patch
# User "Guido Günther <agx@xxxxxxxxxxx>"
# Date 1222957855 -7200
# Node ID a510b28cd7f71f5c5871089f60369f327c83b6d3
# Parent  003db80b54c615696cac682c1c74f696f1f30d45
call populate_opt_media when the widget gets activated

not when it gets deactivated. Also select the first entry since we always have
one. Prevents the physical media combo box from suddenly becomming empty.

diff -r 003db80b54c6 -r a510b28cd7f7 src/virtManager/choosecd.py
--- a/src/virtManager/choosecd.py	Thu Oct 02 15:43:27 2008 +0200
+++ b/src/virtManager/choosecd.py	Thu Oct 02 16:30:55 2008 +0200
@@ -114,6 +114,8 @@
 
     def media_toggled(self, ignore1=None, ignore2=None):
         if self.window.get_widget("physical-media").get_active():
+            self.populate_opt_media()
+            self.window.get_widget("cd-path").set_active(0)
             self.window.get_widget("cd-path").set_sensitive(True)
             self.window.get_widget("iso-path").set_sensitive(False)
             self.window.get_widget("iso-file-chooser").set_sensitive(False)
@@ -121,7 +123,6 @@
             self.window.get_widget("cd-path").set_sensitive(False)
             self.window.get_widget("iso-path").set_sensitive(True)
             self.window.get_widget("iso-file-chooser").set_sensitive(True)
-            self.populate_opt_media()
 
     def change_cd_path(self, ignore1=None, ignore2=None):
         pass
_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/et-mgmt-tools

[Index of Archives]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux