Browse Source

fix(config): Bring back minHDHeight

j8
Leonard Kim 8 years ago
parent
commit
0f0ff6788c
2 changed files with 4 additions and 4 deletions
  1. 3
    3
      config.js
  2. 1
    1
      modules/UI/videolayout/LargeVideoManager.js

+ 3
- 3
config.js View File

@@ -76,9 +76,9 @@ var config = { // eslint-disable-line no-unused-vars
76 76
     'During that time service will not be available. ' +
77 77
     'Apologise for inconvenience.',*/
78 78
     disableThirdPartyRequests: false,
79
-    // The minumum value a video's height or width can be, whichever is
80
-    // smaller, to be considered high-definition.
81
-    minHDSize: 540,
79
+    // The minumum value a video's height (or width, whichever is smaller) needs
80
+    // to be in order to be considered high-definition.
81
+    minHDHeight: 540,
82 82
     // If true - all users without token will be considered guests and all users
83 83
     // with token will be considered non-guests. Only guests will be allowed to
84 84
     // edit their profile.

+ 1
- 1
modules/UI/videolayout/LargeVideoManager.js View File

@@ -569,7 +569,7 @@ export default class LargeVideoManager {
569 569
      */
570 570
     _updateVideoResolutionStatus() {
571 571
         const { height, width } = this.videoContainer.getStreamSize();
572
-        const isCurrentlyHD = Math.min(height, width) >= config.minHDSize;
572
+        const isCurrentlyHD = Math.min(height, width) >= config.minHDHeight;
573 573
 
574 574
         APP.store.dispatch(setLargeVideoHDStatus(isCurrentlyHD));
575 575
     }

Loading…
Cancel
Save