From 0f0ff6788c29f81bc1dee88f366b0fb4cc9e7825 Mon Sep 17 00:00:00 2001 From: Leonard Kim Date: Wed, 17 May 2017 12:57:22 -0700 Subject: [PATCH] fix(config): Bring back minHDHeight --- config.js | 6 +++--- modules/UI/videolayout/LargeVideoManager.js | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/config.js b/config.js index cbc6799b9..47c9a8e29 100644 --- a/config.js +++ b/config.js @@ -76,9 +76,9 @@ var config = { // eslint-disable-line no-unused-vars 'During that time service will not be available. ' + 'Apologise for inconvenience.',*/ disableThirdPartyRequests: false, - // The minumum value a video's height or width can be, whichever is - // smaller, to be considered high-definition. - minHDSize: 540, + // The minumum value a video's height (or width, whichever is smaller) needs + // to be in order to be considered high-definition. + minHDHeight: 540, // If true - all users without token will be considered guests and all users // with token will be considered non-guests. Only guests will be allowed to // edit their profile. diff --git a/modules/UI/videolayout/LargeVideoManager.js b/modules/UI/videolayout/LargeVideoManager.js index 26ee4faf6..4130cc5fd 100644 --- a/modules/UI/videolayout/LargeVideoManager.js +++ b/modules/UI/videolayout/LargeVideoManager.js @@ -569,7 +569,7 @@ export default class LargeVideoManager { */ _updateVideoResolutionStatus() { const { height, width } = this.videoContainer.getStreamSize(); - const isCurrentlyHD = Math.min(height, width) >= config.minHDSize; + const isCurrentlyHD = Math.min(height, width) >= config.minHDHeight; APP.store.dispatch(setLargeVideoHDStatus(isCurrentlyHD)); }