From 80efc68832233850687145abf606b1e57b8e4adc Mon Sep 17 00:00:00 2001
From: Benjamin Paine <painebenjamin@gmail.com>
Date: Thu, 28 Dec 2023 20:10:34 -0500
Subject: [PATCH] fix video offset in ui

---
 src/js/nodes/image-editor/video.mjs | 40 +++++++----------------------
 src/js/view/video.mjs               | 36 ++++++++++++++++++++++----
 2 files changed, 40 insertions(+), 36 deletions(-)

diff --git a/src/js/nodes/image-editor/video.mjs b/src/js/nodes/image-editor/video.mjs
index 3429d10b..60236e7f 100644
--- a/src/js/nodes/image-editor/video.mjs
+++ b/src/js/nodes/image-editor/video.mjs
@@ -19,20 +19,6 @@ class ImageEditorVideoNodeView extends ImageEditorNodeView {
      */
     static nodeTypeName = "Video";
 
-    /**
-     * @var array<string> All fit modes.
-     */
-    static allFitModes = ["actual", "stretch", "cover", "contain"];
-
-    /**
-     * @var array<string> All anchor modes.
-     */
-    static allAnchorModes = [
-        "top-left", "top-center", "top-right",
-        "center-left", "center-center", "center-right",
-        "bottom-left", "bottom-center", "bottom-right"
-    ];
-    
     /**
      * @var string Add the classname for CSS
      */
@@ -61,7 +47,7 @@ class ImageEditorVideoNodeView extends ImageEditorNodeView {
     async updateOptions(newOptions) {
         // Reflected in DOM
         this.updateFit(newOptions.fit);
-        this.updateAnchor(newOptions.anchor);
+        this.updateAnchor(newOptions.anchor, newOptions.offsetX, newOptions.offsetY);
         this.updateOpacity(newOptions.opacity);
     };
 
@@ -70,27 +56,17 @@ class ImageEditorVideoNodeView extends ImageEditorNodeView {
      */
     async updateFit(newFit) {
         this.fit = newFit;
-        this.content.fit = newFit;
-        for (let fitMode of this.constructor.allFitModes) {
-            this.content.removeClass(`fit-${fitMode}`);
-        }
-        if (!isEmpty(newFit)) {
-            this.content.addClass(`fit-${newFit}`);
-        }
+        this.content.setFit(newFit);
     };
 
     /**
      * Updates the video anchor
      */
-    async updateAnchor(newAnchor) {
+    async updateAnchor(newAnchor, offsetX, offsetY) {
         this.anchor = newAnchor;
-        this.content.anchor = newAnchor;
-        for (let anchorMode of this.constructor.allAnchorModes) {
-            this.content.removeClass(`anchor-${anchorMode}`);
-        }
-        if (!isEmpty(newAnchor)) {
-            this.content.addClass(`anchor-${newAnchor}`);
-        }
+        this.offsetX = offsetX || 0;
+        this.offsetY = offsetY || 0;
+        this.content.setAnchor(this.anchor, this.offsetX, this.offsetY);
     }
 
     /**
@@ -132,6 +108,8 @@ class ImageEditorVideoNodeView extends ImageEditorNodeView {
         state.src = includeImages ? this.content.src : null;
         state.anchor = this.anchor || null;
         state.fit = this.fit || null;
+        state.offsetX = this.offsetX || null;
+        state.offsetY = this.offsetY || null;
         state.opacity = this.opacity || 1.0;
         return state;
     }
@@ -146,9 +124,9 @@ class ImageEditorVideoNodeView extends ImageEditorNodeView {
         } else {
             await this.setContent(new VideoView(this.config, newState.src));
         }
-        await this.updateAnchor(newState.anchor);
         await this.updateFit(newState.fit);
         await this.updateOpacity(newState.opacity);
+        await this.updateAnchor(newState.anchor, newState.offsetX, newState.offsetY);
     }
 
     /**
diff --git a/src/js/view/video.mjs b/src/js/view/video.mjs
index 5f5ac250..b641c0a8 100644
--- a/src/js/view/video.mjs
+++ b/src/js/view/video.mjs
@@ -72,6 +72,31 @@ class VideoView extends View {
         }
     }
 
+    /**
+     * Sets the fit mode
+     */
+    setFit(newFitMode) {
+        for (let fitMode of ["actual", "stretch", "cover", "contain"]) {
+            let fitModeClass = `fit-${fitMode}`;
+            if (fitMode === newFitMode) {
+                this.addClass(fitModeClass);
+            } else {
+                this.removeClass(fitModeClass);
+            }
+        }
+    }
+
+    /**
+     * Sets the anchor position
+     */
+    setAnchor(newAnchorMode, offsetX, offsetY) {
+        let [topPart, leftPart] = newAnchorMode.split("-"),
+            topPercent = topPart == "bottom" ? 100 : topPart == "center" ? 50 : 0,
+            leftPercent = leftPart == "right" ? 100 : leftPart == "center" ? 50 : 0;
+
+        this.css("object-position", `calc(${leftPercent}% + ${offsetX}px) calc(${topPercent}% + ${offsetY}px)`);
+    }
+
     /**
      * Build the container and append the DOM node
      */
@@ -82,16 +107,17 @@ class VideoView extends View {
     }
 }
 
+/**
+ * An extension of the above that adds controls
+ */
 class VideoPlayerView extends VideoView {
+    /**
+     * After setting video, set controls
+     */
     setVideo(src) {
         super.setVideo(src);
         this.video.controls = true;
     }
-
-    async build() {
-        let node = await super.build();
-        return node;
-    }
 }
 
 export {