On Mon, Feb 17, 2014 at 01:05:29AM -0500, Chen Hanxiao wrote: > We should disable storage removal by default > and let user select it. >
We chose to default it to True since there is a confirmation needed anyway. Look at the commit b2a7c396 (git blame on master on the line you've changed). If you really need it not to be selected by default, please make it configurable. Although it'll add a lot of unnecessary code :( Martin > Signed-off-by: Chen Hanxiao <[email protected]> > --- > virtManager/delete.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/virtManager/delete.py b/virtManager/delete.py > index 33860bf..96cac58 100644 > --- a/virtManager/delete.py > +++ b/virtManager/delete.py > @@ -101,7 +101,7 @@ class vmmDeleteDialog(vmmGObjectUI): > self.widget("delete-warn-running-vm-box"), vm_active) > > # Disable storage removal by default > - self.widget("delete-remove-storage").set_active(True) > + self.widget("delete-remove-storage").set_active(False) > self.widget("delete-remove-storage").toggled() > > populate_storage_list(self.widget("delete-storage-list"), > -- > 1.8.5.3 > > _______________________________________________ > virt-tools-list mailing list > [email protected] > https://www.redhat.com/mailman/listinfo/virt-tools-list
signature.asc
Description: Digital signature
_______________________________________________ virt-tools-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/virt-tools-list
