Re: [PATCH virt-viewer] Fix bug with initial placement of fullscreen windows

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

 



On Wed, Oct 15, 2014 at 2:28 AM, Fabiano Fidêncio <fidencio@xxxxxxxxxx> wrote:
> On Thu, 2014-10-09 at 11:21 -0500, Jonathon Jongsma wrote:
>> The function app_window_try_fullscreen() will lookup the initial monitor
>> for the nth monitor internally, so we should pass in the display ID to the function
>> rather than the mapped monitor ID. This was causing 2 monitors on the
>> same monitor with a configuration like this:
>>
>>     monitor-mapping=1:2;2:1
>> ---
>>  src/virt-viewer-app.c | 3 +--
>>  1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c
>> index f0c378d..1402986 100644
>> --- a/src/virt-viewer-app.c
>> +++ b/src/virt-viewer-app.c
>> @@ -885,8 +885,7 @@ ensure_window_for_display(VirtViewerApp *self, VirtViewerDisplay *display)
>>              g_debug("Found a window without a display, reusing for this display...");
>>              virt_viewer_app_set_window_subtitle(self, win, nth);
>>              if (self->priv->fullscreen && !self->priv->kiosk)
>> -                app_window_try_fullscreen(self, win,
>> -                                          virt_viewer_app_get_initial_monitor_for_display(self, nth));
>> +                app_window_try_fullscreen(self, win, nth);
>>          } else {
>>              win = virt_viewer_app_window_new(self, nth);
>>          }
>
> ACK!
>

Pushed as
c64a3c8f707fcbb45c21cb6b4f5fc0b799ca6df5

Best Regards,
-- 
Fabiano Fidêncio

_______________________________________________
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