On 04/17/2014 11:13 AM, Chen Hanxiao wrote: > From: Chen Hanxiao <[email protected]> > > We could not delete USB controller by "Remove" button. > But we could delete it by right click option. > This patch will disable remove popup menu for USB controller. > > Signed-off-by: Chen Hanxiao <[email protected]> > --- > virtManager/details.py | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/virtManager/details.py b/virtManager/details.py > index 2e8991b..beb2dcd 100644 > --- a/virtManager/details.py > +++ b/virtManager/details.py > @@ -1081,7 +1081,8 @@ class vmmDetails(vmmGObjectUI): > return > > rmdev = self._addhwmenuitems["remove"] > - if hasattr(devobj, "virtual_device_type"): > + if (hasattr(devobj, "virtual_device_type") and > + devobj.type != "usb"): > rmdev.show() > else: > rmdev.hide() >
This would require duplicating all the logic in two places. And this fix isn't exhaustive, since for example it will still allow right-click removal of ps2 mouse which is also disallowed. Can we do something like check if the 'remove' button is sensitive or visible? And make the same change for the popup window. - Cole _______________________________________________ virt-tools-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/virt-tools-list
