Remove target from calculateHeight

Author: binarykitchen

package-lock.json

@@ -120,7 +120,7 @@
     "type-fest": "5.2.0",
     "typescript": "5.9.3",
     "typescript-eslint": "8.41.0",
-    "vitest": "4.0.10"
+    "vitest": "4.0.13"
   },
   "engines": {
     "node": "^24.11.0",

package.json

@@ -9,7 +9,6 @@ function calculateHeight(
   responsive: boolean,
   videoWidth: number | undefined,
   options: VideomailClientOptions,
-  target: string,
   ratio?: number,
   element?: HTMLElement | null,
 ) {
@@ -19,12 +18,7 @@ function calculateHeight(
 
   let width: number | undefined = videoWidth;
 
-  if (width === 0) {
-    throw createError({
-      message: `Unable to calculate height for target ${target} when width is zero, while responsive mode is set to ${responsive}`,
-      options,
-    });
-  } else if (responsive && element) {
+  if (responsive && element) {
     const limitedDimension = limitWidth(element, options, width);
     width = limitedDimension.value;
   }

src/util/html/dimensions/calculateHeight.ts

@@ -1606,21 +1606,17 @@ class Recorder extends Despot {
 
   public calculateHeight(responsive: boolean) {
     let videoWidth: number | undefined;
-    let target = "(unknown)";
 
     if (this.userMedia) {
-      target = "userMedia";
       videoWidth = this.userMedia.getVideoWidth();
     } else if (this.recorderElement) {
-      target = "recorderElement";
       videoWidth = this.recorderElement.videoWidth || this.recorderElement.width;
     }
 
     return calculateHeight(
       responsive,
       videoWidth,
       this.options,
-      target,
       this.getRatio(),
       this.recorderElement,
     );

src/wrappers/visuals/recorder.ts

@@ -95,23 +95,12 @@ class Replay extends Despot {
 
     if (!height) {
       let element = this.visuals.getElement();
-      let target: string;
 
-      if (element) {
-        target = "visualsElement";
-      } else {
+      if (!element) {
         element = document.body;
-        target = "document body";
       }
 
-      height = calculateHeight(
-        responsive,
-        videoWidth,
-        this.options,
-        target,
-        ratio,
-        element,
-      );
+      height = calculateHeight(responsive, videoWidth, this.options, ratio, element);
     }
 
     if (width > 0) {