From 36b242cc727ae67adb319072d1e5c4283c8e267a Mon Sep 17 00:00:00 2001 From: Feross Aboukhadijeh Date: Thu, 3 Mar 2016 16:04:00 -0800 Subject: [PATCH] fix: maintain correct aspect ration when video is larger than screen --- main/index.js | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/main/index.js b/main/index.js index c1432c96..800c9906 100644 --- a/main/index.js +++ b/main/index.js @@ -258,9 +258,15 @@ function setDimensions (dimensions) { // Limit window size to screen size var workAreaSize = electron.remote.screen.getPrimaryDisplay().workAreaSize - var width = Math.min(dimensions.width, workAreaSize.width) - var height = Math.min(dimensions.height, workAreaSize.height) - var aspectRatio = width / height + var aspectRatio = dimensions.width / dimensions.height + + var scaleFactor = Math.min( + Math.min(workAreaSize.width / dimensions.width, 1), + Math.min(workAreaSize.height / dimensions.height, 1) + ) + + var width = Math.floor(dimensions.width * scaleFactor) + var height = Math.floor(dimensions.height * scaleFactor) height += HEADER_HEIGHT