Red Hat Bugzilla – Bug 1027381
[RFE] Deprecate "basic fullscreen" mode
Last modified: 2014-11-10 14:42:28 EST
virt-viewer currently has a --full-screen option that has an optional 'auto-conf' argument. The main difference between the basic fullscreen mode and the auto-conf fullscreen mode is that basic fullscreen mode creates a window for each display that is currently configured on the guest and then makes these windows fullscreen. If there are fewer guest displays than client monitors, the client monitors are left empty. If there are *more* guest displays than client monitors, then all of the extra displays become windows hidden behind eachother on the last client monitor.
In contrast, when possible (i.e. with spice + vdagent) the auto-conf fullscreen mode tries to enable exactly the same number of guest displays as client monitors. This results in a much better user experience. If auto-conf is unsupported, it behaves the same as basic fullscreen mode.
I propose to remove the distinction between these two modes and simply have a single mode that behaves as auto-conf currently does.
I think that's the way to go, that's way the patch in virt-viewer is already on it's way upstream. Hopefully we don't have to carry a RHEV only patch.
no objection from rhev side, we do need to clean up the mess in arguments being passed to spice in this specific area (ctrl-alt-del as well:)
When is this RFE target? the earlier the better imho
ok, let's do that...so what do we pass, what parameters and values exactly
- for .vv file
- for Firefox XPI / IE ActiveX plugin
Which version will support it? What about older clients behavior?
For new clients, fullscreen will imply auto-conf. So any explicit auto-conf arguments will simply be ignored. So we should try to make both new and old clients always do auto-conf when fullscreen is selected.
In the case of the spi/activex plugins, you should probably set both FULL_SCREEN and AUTO_DISPLAY_RES to true. The new client will simply ignore the AUTO_DISPLAY_RES flag, but will do auto-conf regardless.
For the vv-file, it's a bit more difficult to make old and new clients behave exactly the same because the vv-file format does not currently have a field for specifying auto-conf. There is only a single 'fullscreen' field that can be set. If 'fullscreen' is set to 1, the new client would do auto-conf, but the old client would not. This issue is not specific to this change, however, since it also affects current RHEVM. See Bug 1083203 for discussion on addressing this issue.
*** Bug 1002690 has been marked as a duplicate of this bug. ***
ok, so it seems currently we need to:
1) on SPICE side to ignore the auto_res property. We are passing fullscreen to .vv and plugins accordingly.
2) in Basic User Portal correctly set default to "fullscreen" (doesn't work in 3.3)
Closing because since 3.4 fullscreen flag default is set correctly to true in basic user portal.
I'd like to handle this bug (I closed it already but I'm not sure if it's a good think). I'd like to ask you if Michal's 1st point in comment #10 is taken care of.
I'm not entirely sure what Michal meant by "on SPICE side" in point #1. If it means the spice client (e.g. remote-viewer), then it is being handled.
(In reply to Jonathon Jongsma from comment #13)
can you please link the corresponding bug # for the spice-client just so we have a complete set of prerequisites to test this?
(In reply to Frantisek Kobzik from comment #11)
doesn't seem to be set correctly in 3.4 GA
After further investigation it appears the fullscreen flag is set correctly:
- true in _basic_ user portal
- false otherwise.