Re: [PATCH] Add a11y names for virt-manager components

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx

_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/et-mgmt-tools

[Index of Archives]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux