On Mon, 2013-09-23 at 16:40 -0400, David Shea wrote:
On 09/23/2013 01:25 PM, Vratislav Podzimek wrote:
Otherwise we don't know which repository if the one with updates.
Signed-off-by: Vratislav Podzimek vpodzime@redhat.com
pyanaconda/ui/gui/spokes/source.py | 24 +++++++++++++++++++----- 1 file changed, 19 insertions(+), 5 deletions(-)
diff --git a/pyanaconda/ui/gui/spokes/source.py b/pyanaconda/ui/gui/spokes/source.py index bc72564..27ffd90 100644 --- a/pyanaconda/ui/gui/spokes/source.py +++ b/pyanaconda/ui/gui/spokes/source.py @@ -545,6 +545,9 @@ class SourceSpoke(NormalSpoke): # updates option container self._updatesBox = self.builder.get_object("updatesBox")
self._proxyButton = self.builder.get_object("proxyButton")
self._nfsOptsBox = self.builder.get_object("nfsOptsBox")
def initialize(self): NormalSpoke.initialize(self)
@@ -753,6 +756,15 @@ class SourceSpoke(NormalSpoke): enabled = button.get_active() relatedBox.set_sensitive(enabled)
if button in (self._autodetectButton, self._isoButton):
# just make updates check box sensitive and unchecked by default
self._noUpdatesCheckbox.set_active(False)
self._updatesBox.set_sensitive(True)
else:
# network source selected, setup updates check box based on protocol
# chosen
self._protocolComboBox.emit("changed")
I'd rather see this conditional flipped around so that the changed signal is emitted if button == self._networkButton, else make the checkbox sensitive and unchecked. That way if we add another radio button it doesn't have to be added here unless it needs to interact with the updates checkbox. Otherwise, ack.
Good idea, thanks! Changing locally.