Re: [PATCH] window: Do not allow resizing when not in "Normal Size"

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

 



Jonathon,

On Tue, Jan 12, 2016 at 5:21 PM, Jonathon Jongsma <jjongsma@xxxxxxxxxx> wrote:
> I agree that disabling resize when zoom is enabled makes some things simpler
> from an implementation perspective, but I don't think it's a good idea.

This patch was a tentative to have a well defined behavior on zoom an
then avoid the rabbit holes we have with respect to the zooming
behavior.
I will try to check what other viewers do and then get back to this
discussion (zoom behavior) at some point.

>
> >From a user's point of view, there's no obvious reason why zooming should
> disable resizing, so they won't expect it. The more astute users may notice a
> correlation between zoom enabled and resize not working, but most users will
> probably just think resizing stopped working randomly. So we'll probably
> suddenly start getting complaints that resizing stopped working.

Well, it must be documented somewhere, with or without this patch.
It's not the first time (and probably not the last) we will end up
discussing what should be the expected behavior for something related
to zoom.

>
> >From a developer point of view, it makes it much more difficult to test
> resolutions that are higher than your physical client monitor.

That's a really good point and I have to agree. But I really don't see
any other reason for having zoom + resizing apart from us testing
higher resolutions.

Best Regards,

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



[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux