Remove 3 unused instance variables and added an explanation of why th…

…e aspect ratio of the video is important
AlexxIT · Jun 18, 2023 · 15147f4 · 15147f4
1 parent da58ed5
commit 15147f4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/custom_components/webrtc/www/digital-ptz.js b/custom_components/webrtc/www/digital-ptz.js
@@ -22,9 +22,6 @@ const DEFAULT_OPTIONS = {
   persist: true,
 };
 export class DigitalPTZ {
-  lastTouches;
-  lastMouse;
-  lastTap = 0;
   containerEl;
   videoEl;
   resizeObserver;

diff --git a/custom_components/webrtc/www/ditigal-ptz-transform.js b/custom_components/webrtc/www/ditigal-ptz-transform.js
@@ -30,6 +30,9 @@ export class Transform {
       // Once it loads, the videometadata listener will call this function again.
       return;
     }
+    // When in full screen, and if the aspect ratio of the screen differs from that of the video,
+    // black bars will be shown either to the sides or above/below the video.
+    // This needs to be accounted for when panning, the code below keeps track of that.
     const screenAspectRatio =
       this.containerRect.width / this.containerRect.height;
     const videoAspectRatio = videoEl.videoWidth / videoEl.videoHeight;
@@ -55,6 +58,7 @@ export class Transform {
       );
     }
   }
+  // dx,dy are deltas.
   move(dx, dy) {
     if (!this.videoRect) return;
     const bound = (this.scale - 1) / 2;