Re: [PATCH]:virt-viewer: Load "Send key" menu from XML file

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

 



On Tue, Dec 16, 2008 at 01:50:22PM -0700, Pat Campbell wrote:
> 
> Attached patch modifies virt-viewer so that it acquires the "Send key"
> menu contents from an xml file and allows user to load a browsed for
> keymap def file while running.

I like the idea of making it configurable, but not the idea of having
the user create & load XML files for this.

If a user wants a custom send-key set, they'll likely want it preserved
and used across all instances of virt-viewer automatically every time
it is started rather than having to load the file.

I think we should store the custom send-key sets in GConf, rather than
XML files, and at the bottom of the 'send key' menu have an final
entry 'Edit keys...' which pops up a dialog allowing the user to add
or remove keys from the menu. Just letting them enter the key name (which
we can validate by just doing a lookup on the string they enter). GConf
would ensure changes in one instance are instant applied to all other
running instances.

If we want to get adventurous later on we could add 'profiles' to define
a set of keys, allowing switching between profiles, and remember which
profile is associated with each VM UUID. This would allow user to have
one set of keys for Windows VMs, with anothe for Linux, and another
for BSD or whatever.

Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

_______________________________________________
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