[et-mgmt-tools] [PATCH] Add a11y names for virt-manager components

[Cole Robinson]

Hugh Brock wrote:
> Cole Robinson wrote:
>> Hugh Brock wrote:
>>> Cole Robinson wrote:
>>>> Hello all,
>>>>
>>>> Attached is a patch that adds some simple accessibility names for input
>>>> components of virt-manager. This is to facilitate future gui testing.
>>>>
>>>> Thanks,
>>>> Cole
>>>>
>>> The patch doesn't seem to break anything, but I am getting a warning: 
>>> libglade-WARNING **: Unexpected element <accessibility> inside <widget>. 
>>> You know anything about this?
>>>
>>> Thanks,
>>> --Hugh
>>>
>> Hmm, I tried to reproduce it but I'm not seeing it on fully updated F7
>> at least. I'm guessing this happens when you start up the patched
>> virt-manager?
>>
> 
> Yeah, I get it when I open the create wizard actually. I was testing on, 
> umm, latest F7 I think. Could be a bad merge since your patch to 
> vmm-create.glade merged with offsets. However the message itself is odd; 
> <accessibility> should be legal inside <widget>.
> 
> Note I see the warning when I run virt-manager from the command line a 
> la "python virt-manager.py" -- so I am getting all the gtk spew as well.
> 
> I'm inclined to ignore the message unless we think it indicates 
> something more serious...
> 
> --H
> 

When I 'hg import' the patch against upstream tip I don't see any
messages about offsets. It seems to go smoothly. Maybe whatever we are
doing differently is the culprit.

- Cole

-- 
Cole Robinson
crobinso at redhat.com