On Tue, Jun 07, 2016 at 01:34:10PM -0400, Cole Robinson wrote: > On 06/07/2016 11:20 AM, Pavel Hrdina wrote: > > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1302175 > > > > Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx> > > --- > > virtManager/console.py | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > diff --git a/virtManager/console.py b/virtManager/console.py > > index aa208d0..66e89d8 100644 > > --- a/virtManager/console.py > > +++ b/virtManager/console.py > > @@ -623,11 +623,11 @@ class vmmConsolePages(vmmGObjectUI): > > self.widget("console-auth-username").set_text(username) > > self.widget("console-auth-password").set_text(pw) > > > > - self.widget("console-auth-remember").set_sensitive( > > - bool(self.config.has_keyring())) > > Is console-auth-remember insensitive by default? Otherwise moving this bit > into the lower block may not be correct, in the case when there isn't any > 'Secrets' API dbus provider available the checkbox may still be set as sensitive That's a good point, I'll change only the condition for set_active(), otherwise it would be sensitive and could confuse user. Thanks, Pavel _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
