From 14d078fb9fd4d9398f1302ec7f9b39232953d7a7 Mon Sep 17 00:00:00 2001
From: moex3 <46636583+moex3@users.noreply.github.com>
Date: Wed, 21 Apr 2021 00:24:07 +0200
Subject: [PATCH] Fix a zooming bug on linux

The preview window would change in width, when the video was being zoomed.
Possibly, because on windows, zooming only calls UpdateSize once, but on
linux, it gets called multiple, like 8 times.
Rounding fixed it.
---
 src/video_display.cpp | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/video_display.cpp b/src/video_display.cpp
index 243047486..57db99195 100644
--- a/src/video_display.cpp
+++ b/src/video_display.cpp
@@ -330,11 +330,12 @@ void VideoDisplay::UpdateSize() {
 		wxWindow *top = GetParent();
 		while (!top->IsTopLevel()) top = top->GetParent();
 
-		wxSize cs = GetClientSize();
-		float csAr = (float)cs.GetWidth() / (float)cs.GetHeight();
+		wxSize oldClientSize = GetClientSize();
+		double csAr = (double)oldClientSize.GetWidth() / (double)oldClientSize.GetHeight();
+		wxSize newClientSize = wxSize(std::lround(provider->GetHeight() * csAr), provider->GetHeight()) * windowZoomValue / scale_factor;
 		wxSize oldSize = top->GetSize();
-		top->SetSize(top->GetSize() + wxSize(provider->GetHeight() * csAr, provider->GetHeight()) * windowZoomValue / scale_factor - cs);
-		SetClientSize(cs + top->GetSize() - oldSize);
+		top->SetSize(oldSize + (newClientSize - oldClientSize));
+		SetClientSize(oldClientSize + (top->GetSize() - oldSize));
 	}
 	else {
 		wxSize newSize = wxSize(provider->GetWidth(), provider->GetHeight()) * windowZoomValue / scale_factor;