<div dir="auto"><div>Cole, can I request that you reconsider how niche this is or isn't? I routinely work in the manner described here. Maybe I'm also just unusual!<div dir="auto"><br></div><div dir="auto">Cheers, </div><div dir="auto"><br></div><div dir="auto">Peter </div><br><br><div class="gmail_quote"><div dir="ltr">On Mon, 7 Jan 2019, 23:15 Cole Robinson <<a href="mailto:crobinso@redhat.com">crobinso@redhat.com</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 01/01/2019 07:28 PM, Povilas Kanapickas wrote:<br>
> The screen estate used by the virt-manager itself in the console mode reduces<br>
> the amount of screen estate available to the VM. virt-manager currently supports<br>
> the fullscreen option to make maximum use of the available screen. Unfortunately,<br>
> this hides the OS taskbar, which makes this approach less usable for workflows<br>
> that need to use the taskbar, e.g. when switching between many different VMs at<br>
> the same time.<br>
> <br>
> The PR introduces a "hide decorations" menu option. This option hides the menubar<br>
> and the OS window decorations of the VM viewer. Essentially, it's the fullscreen<br>
> option without actually going full screen, i.e. the window mechanics are still<br>
> preserved and the window can still be hidden, obscured and so on. This mode can<br>
> be turned off much like the fullscreen option - my moving mouse near the top edge<br>
> which shows the usual two-button box.<br>
> <br>
> The PR also slightly changes how the "revealer" box works. In non-fullscreen mode<br>
> the assumption that it's easy to navigate to the exact top pixel of the window<br>
> no longer holds. So the eventbox is now a completely separate rectangle from the<br>
> revealer itself and thus can span much wider span of the window and be more than<br>
> a single pixel high. This way we don't grab mouse events from much larger area<br>
> than we want whenever the box with buttons is shown which would have been an<br>
> issue with the previous implementation if I just increased the width of the<br>
> eventbox.<br>
> <br>
<br>
Thanks for the patches. It's an interesting idea but the use case seems <br>
very niche to have explicit support in virt-manager IMO. Even the idea <br>
of hiding window decorations seems quite obscure, not even <br>
vncviewer/tigervnc has it AFAICT.<br>
<br>
Also in my testing it doesn't look to be implemented for gtk on wayland <br>
which is kinda the way forward, so we end up with a UI element that is a <br>
no-op depending on the desktop config. Maybe that's fixable but I don't <br>
know offhand<br>
<br>
I like the idea of splitting out the overlay logic to its own class <br>
though, helps readability, so I pushed patch #1<br>
<br>
Thanks,<br>
Cole<br>
<br>
_______________________________________________<br>
virt-tools-list mailing list<br>
<a href="mailto:virt-tools-list@redhat.com" target="_blank" rel="noreferrer">virt-tools-list@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/virt-tools-list" rel="noreferrer noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/virt-tools-list</a><br>
</blockquote></div></div></div>