On 04/21/2014 11:28 PM, Chen Hanxiao wrote: > Sometimes we need to set remove button as none-sensitive. > > This patch introduces show_remove_button method as > the only interface for showing/removing, > also easy to maintain. > > Signed-off-by: Chen Hanxiao <chenhanxiao@xxxxxxxxxxxxxx> > --- > v2: remove checks in refresh_* function, > invoked show_remove_button in hw_selected > > virtManager/details.py | 43 +++++++++++++++++++++++++++++++++++-------- > 1 file changed, 35 insertions(+), 8 deletions(-) > TBH I still don't like moving this logic away from where it already is. If we had to do this for many devices I can understand that a bit, but it's a very special case (and even USB _is_ removable we just don't implement it). So NACK - Cole _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
