[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 6/9] gtk: add support for screen scaling and ful
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH 6/9] gtk: add support for screen scaling and full screen (v3) |
Date: |
Tue, 19 Feb 2013 17:29:40 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, Feb 19, 2013 at 10:10:18AM -0600, Anthony Liguori wrote:
> Kevin Wolf <address@hidden> writes:
>
> > On Mon, Feb 18, 2013 at 05:57:02PM -0600, Anthony Liguori wrote:
> >> Basic menu items to enter full screen mode and zoom in/out. Unlike SDL, we
> >> don't allow arbitrary scaling based on window resizing. The current
> >> behavior
> >> with SDL causes a lot of problems for me.
> >>
> >> Sometimes I accidentally resize the window a tiny bit while trying to move
> >> it
> >> (Ubuntu's 1-pixel window decorations don't help here). After that,
> >> scaling is
> >> now active and if the screen changes size again, badness ensues since the
> >> aspect ratio is skewed.
> >>
> >> Allowing zooming by 25% in and out should cover most use cases. We can
> >> add a
> >> more flexible scaling later but for now, I think this is a more friendly
> >> behavior.
> >>
> >> Signed-off-by: Anthony Liguori <address@hidden>
> >
> > Zoom to fit doesn't only change whether the contents of the window is
> > scaled or not, but it also resizes the window to 100% when it's toggled.
> > I don't think this is the right behaviour.
>
> This behavior is controlled by two lines in gd_menu_zoom_fit:
>
> s->scale_x = 1.0;
> s->scale_y = 1.0;
>
> If you remove them, it keeps the window size from zooming but works
> you're expecting.
I thought the same at the first sight, but when I tried that it didn't
work and just caused strange behaviour. That's why I commented on the
patch in general.
> I'm not sure what the Right Way is from a Gnome point of view but
> evince's "Best Fit" doesn't alter the window size which I think is a
> close analogy. So I'm inclined to agree with your assessment. I'll
> remove these lines from the next revision.
Thanks.
Kevin