From 6897ad0f1eb7b4eaab12d0a74d87696d9f328f21 Mon Sep 17 00:00:00 2001
From: Dennis Postma <dennis@directonline.io>
Date: Sun, 16 Mar 2025 01:21:31 +0100
Subject: [PATCH] Renamed file, removed redundant fields

---
 .../partials/sprite/SpriteDetails.vue         |  90 ++++-----
 .../sprite/partials/SpriteImagesInput.vue     | 185 ------------------
 .../sprite/partials/SpriteImagesPreview.vue   |  38 ++++
 3 files changed, 75 insertions(+), 238 deletions(-)
 delete mode 100644 src/components/gameMaster/assetManager/partials/sprite/partials/SpriteImagesInput.vue
 create mode 100644 src/components/gameMaster/assetManager/partials/sprite/partials/SpriteImagesPreview.vue

diff --git a/src/components/gameMaster/assetManager/partials/sprite/SpriteDetails.vue b/src/components/gameMaster/assetManager/partials/sprite/SpriteDetails.vue
index 7ff430b..feedb57 100644
--- a/src/components/gameMaster/assetManager/partials/sprite/SpriteDetails.vue
+++ b/src/components/gameMaster/assetManager/partials/sprite/SpriteDetails.vue
@@ -28,39 +28,19 @@
           </div>
         </template>
         <template #content>
-<!--          IMAGES COME HERE, REMOVE FIELDS -->
-          <form class="flex gap-2.5 flex-wrap" @submit.prevent="saveSprite">
-            <div class="form-field-full">
-              <label for="action">Action</label>
-              <input v-model="action.action" class="input-field" type="text" name="action" placeholder="Action" />
-            </div>
-            <div class="form-field-half">
-              <label for="origin-x">Origin X</label>
-              <input v-model.number="action.originX" class="input-field" type="number" step="any" name="origin-x" placeholder="Origin X" />
-            </div>
-            <div class="form-field-half">
-              <label for="origin-y">Origin Y</label>
-              <input v-model.number="action.originY" class="input-field" type="number" step="any" name="origin-y" placeholder="Origin Y" />
-            </div>
-            <div class="form-field-full">
-              <label for="frame-speed">Frame rate</label>
-              <input v-model.number="action.frameRate" class="input-field" type="number" step="any" name="frame-speed" placeholder="Frame rate" />
-            </div>
-            <div class="form-field-full">
-              <SpriteActionsInput v-model="action.sprites" @tempOffsetChange="(index, offset) => handleTempOffsetChange(action, index, offset)" />
-            </div>
-          </form>
+          <SpriteImagesPreview :spriteActionImages="action.sprites" @tempOffsetChange="(index, offset) => handleTempOffsetChange(action, index, offset)" />
         </template>
       </Accordion>
       <SpriteEditor
-        v-if="selectedAction"
-        :sprites="selectedAction.sprites"
-        :frame-rate="selectedAction.frameRate"
-        :is-modal-open="isModalOpen"
-        :temp-offset-index="tempOffsetData.index"
-        :temp-offset="tempOffsetData.offset"
-        @update:frame-rate="updateFrameRate"
-        @update:is-modal-open="isModalOpen = $event"
+        v-for="[actionId, editorData] in Array.from(openEditors.entries())"
+        :key="actionId"
+        :sprites="editorData.action.sprites"
+        :frame-rate="editorData.action.frameRate"
+        :is-modal-open="editorData.isOpen"
+        :temp-offset-index="getTempOffsetIndex(editorData.action)"
+        :temp-offset="getTempOffset(editorData.action)"
+        @update:frame-rate="(value) => updateFrameRate(editorData.action, value)"
+        @update:is-modal-open="(value) => handleEditorModalClose(editorData.action, value)"
       />
     </div>
   </div>
@@ -70,8 +50,8 @@
 import { SocketEvent } from '@/application/enums'
 import type { Sprite, SpriteAction } from '@/application/types'
 import { downloadCache, uuidv4 } from '@/application/utilities'
-import SpriteActionsInput from '@/components/gameMaster/assetManager/partials/sprite/partials/SpriteImagesInput.vue'
 import SpriteEditor from '@/components/gameMaster/assetManager/partials/sprite/partials/SpriteEditor.vue'
+import SpriteImagesPreview from '@/components/gameMaster/assetManager/partials/sprite/partials/SpriteImagesPreview.vue'
 import Accordion from '@/components/utilities/Accordion.vue'
 import { socketManager } from '@/managers/SocketManager'
 import { SpriteStorage } from '@/storage/storages'
@@ -81,11 +61,11 @@ import { computed, onBeforeUnmount, onMounted, ref, watch } from 'vue'
 const assetManagerStore = useAssetManagerStore()
 
 const selectedSprite = computed(() => assetManagerStore.selectedSprite)
-
+const tempOffsetData = ref<Map<string, { index: number | undefined; offset: { x: number; y: number } | undefined }>>(new Map())
 const spriteName = ref('')
 const spriteActions = ref<SpriteAction[]>([])
-const isModalOpen = ref(false)
-const selectedAction = ref<SpriteAction | null>(null)
+
+const openEditors = ref(new Map<string, { action: SpriteAction; isOpen: boolean }>())
 
 if (!selectedSprite.value) {
   console.error('No sprite selected')
@@ -192,37 +172,41 @@ function sortSpriteActions(actions: SpriteAction[]): SpriteAction[] {
 }
 
 function openEditorModal(action: SpriteAction) {
-  selectedAction.value = action
-  isModalOpen.value = true
+  const newOpenEditors = new Map(openEditors.value)
+  newOpenEditors.set(action.id, { action, isOpen: true })
+  openEditors.value = newOpenEditors
 }
 
-function updateFrameRate(value: number) {
-  if (selectedAction.value) {
-    selectedAction.value.frameRate = value
-  }
+function updateFrameRate(action: SpriteAction, value: number) {
+  action.frameRate = value
 }
 
-const tempOffsetData = ref<{ index: number | undefined; offset: { x: number; y: number } | undefined }>({
-  index: undefined,
-  offset: undefined
-})
+function handleEditorModalClose(action: SpriteAction, isOpen: boolean) {
+  if (isOpen) return
+  const newOpenEditors = new Map(openEditors.value)
+  newOpenEditors.delete(action.id)
+  openEditors.value = newOpenEditors
+}
 
 function handleTempOffsetChange(action: SpriteAction, index: number, offset: { x: number; y: number }) {
-  if (selectedAction.value === action) {
-    tempOffsetData.value = { index, offset }
-  }
+  const newTempOffsetData = new Map(tempOffsetData.value)
+  newTempOffsetData.set(action.id, { index, offset })
+  tempOffsetData.value = newTempOffsetData
+}
+
+function getTempOffsetIndex(action: SpriteAction): number | undefined {
+  return tempOffsetData.value.get(action.id)?.index
+}
+
+function getTempOffset(action: SpriteAction): { x: number; y: number } | undefined {
+  return tempOffsetData.value.get(action.id)?.offset
 }
 
 watch(selectedSprite, (sprite: Sprite | null) => {
   if (!sprite) return
   spriteName.value = sprite.name
   spriteActions.value = sortSpriteActions(sprite.spriteActions)
-})
-
-watch(isModalOpen, (newValue) => {
-  if (!newValue) {
-    selectedAction.value = null
-  }
+  openEditors.value = new Map()
 })
 
 onMounted(() => {
diff --git a/src/components/gameMaster/assetManager/partials/sprite/partials/SpriteImagesInput.vue b/src/components/gameMaster/assetManager/partials/sprite/partials/SpriteImagesInput.vue
deleted file mode 100644
index 79f1f44..0000000
--- a/src/components/gameMaster/assetManager/partials/sprite/partials/SpriteImagesInput.vue
+++ /dev/null
@@ -1,185 +0,0 @@
-<template>
-  <div class="flex flex-wrap gap-3">
-    <div v-for="(image, index) in modelValue" :key="index" class="h-20 w-20 p-4 bg-gray-300 bg-opacity-50 rounded text-center relative group cursor-move" draggable="true" @dragstart="dragStart($event, index)" @dragover.prevent @dragenter.prevent @drop="drop($event, index)">
-      <img :src="image.url" class="max-w-full max-h-full object-contain pointer-events-none" alt="Uploaded image" @load="updateImageDimensions($event, index)" />
-      <div v-if="imageDimensions[index]" class="absolute bottom-1 right-1 bg-black/50 text-white text-xs px-1 py-0.5 rounded transition-opacity font-default">{{ imageDimensions[index].width }}x{{ imageDimensions[index].height }}</div>
-      <div class="absolute top-1 left-1 flex-row space-y-1">
-        <button @click.stop="deleteImage(index)" class="bg-red-500 text-white rounded-full w-6 h-6 flex items-center justify-center cursor-pointer opacity-0 group-hover:opacity-100 transition-opacity" aria-label="Delete image">
-          <svg xmlns="http://www.w3.org/2000/svg" class="h-4 w-4" fill="none" viewBox="0 0 24 24" stroke="currentColor">
-            <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M6 18L18 6M6 6l12 12" />
-          </svg>
-        </button>
-        <button @click.stop="openOffsetModal(index)" class="bg-blue-500 text-white rounded-full w-6 h-6 flex items-center justify-center cursor-pointer opacity-0 group-hover:opacity-100 transition-opacity" aria-label="Scope image">
-          <svg width="50px" height="50px" viewBox="0 0 16 16" fill="none" xmlns="http://www.w3.org/2000/svg">
-            <path d="M8.29289 3.70711L1 11V15H5L12.2929 7.70711L8.29289 3.70711Z" fill="white" />
-            <path d="M9.70711 2.29289L13.7071 6.29289L15.1716 4.82843C15.702 4.29799 16 3.57857 16 2.82843C16 1.26633 14.7337 0 13.1716 0C12.4214 0 11.702 0.297995 11.1716 0.828428L9.70711 2.29289Z" fill="white" />
-          </svg>
-        </button>
-      </div>
-
-      <Modal :is-modal-open="selectedImageIndex === index" :modal-width="300" :modal-height="210" :is-resizable="false" bg-style="none" @modal:close="closeOffsetModal">
-        <template #modalHeader>
-          <h3 class="m-0 font-medium shrink-0 text-white">Action offset ({{ selectedImageIndex }})</h3>
-        </template>
-        <template #modalBody>
-          <div class="m-4">
-            <form method="post" @submit.prevent="saveOffset(index)" class="inline">
-              <div class="gap-2.5 flex flex-wrap">
-                <div class="form-field-half">
-                  <label for="offsetX">Offset X</label>
-                  <input class="input-field max-w-64" v-model="tempOffset.x" name="offsetX" id="offsetX" type="number" />
-                </div>
-                <div class="form-field-half">
-                  <label for="offsetY">Offset Y</label>
-                  <input class="input-field max-w-64" v-model="tempOffset.y" name="offsetY" id="offsetY" type="number" />
-                </div>
-              </div>
-              <button class="btn-cyan px-4 py-1.5 min-w-24" type="submit">Save</button>
-            </form>
-          </div>
-        </template>
-      </Modal>
-    </div>
-    <div class="h-20 w-20 p-4 bg-gray-200 bg-opacity-50 rounded justify-center items-center flex hover:cursor-pointer" @click="triggerFileInput" @drop.prevent="onDrop" @dragover.prevent>
-      <svg xmlns="http://www.w3.org/2000/svg" class="h-6 w-6 invert" fill="none" viewBox="0 0 24 24" stroke="currentColor">
-        <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M12 6v6m0 0v6m0-6h6m-6 0H6" />
-      </svg>
-    </div>
-  </div>
-  <input type="file" ref="fileInput" @change="onFileChange" multiple accept="image/png" class="hidden" />
-</template>
-
-<script setup lang="ts">
-import Modal from '@/components/utilities/Modal.vue'
-import { ref, watch } from 'vue'
-
-interface SpriteImage {
-  url: string
-  offset: {
-    x: number
-    y: number
-  }
-}
-
-interface Props {
-  modelValue: SpriteImage[]
-}
-
-const props = withDefaults(defineProps<Props>(), {
-  modelValue: () => []
-})
-
-const emit = defineEmits<{
-  (e: 'update:modelValue', value: SpriteImage[]): void
-  (e: 'close'): void
-  (e: 'tempOffsetChange', index: number, offset: { x: number; y: number }): void
-}>()
-
-const fileInput = ref<HTMLInputElement | null>(null)
-const draggedIndex = ref<number | null>(null)
-const selectedImageIndex = ref<number | null>(null)
-const tempOffset = ref({ x: 0, y: 0 })
-
-const triggerFileInput = () => {
-  fileInput.value?.click()
-}
-
-const onFileChange = (event: Event) => {
-  const target = event.target as HTMLInputElement
-  if (target.files) {
-    handleFiles(target.files)
-  }
-}
-
-const onDrop = (event: DragEvent) => {
-  if (event.dataTransfer?.files) {
-    handleFiles(event.dataTransfer.files)
-  }
-}
-
-const handleFiles = (files: FileList) => {
-  Array.from(files).forEach((file) => {
-    if (!file.type.startsWith('image/')) {
-      return
-    }
-
-    const reader = new FileReader()
-    reader.onload = (e) => {
-      if (typeof e.target?.result === 'string') {
-        const newImage: SpriteImage = {
-          url: e.target.result,
-          offset: { x: 0, y: 0 }
-        }
-        updateImages([...props.modelValue, newImage])
-      }
-    }
-    reader.readAsDataURL(file)
-  })
-}
-
-const updateImages = (newImages: SpriteImage[]) => {
-  emit('update:modelValue', newImages)
-}
-
-const deleteImage = (index: number) => {
-  const newImages = [...props.modelValue]
-  newImages.splice(index, 1)
-  updateImages(newImages)
-}
-
-const dragStart = (event: DragEvent, index: number) => {
-  draggedIndex.value = index
-  if (event.dataTransfer) {
-    event.dataTransfer.effectAllowed = 'move'
-    event.dataTransfer.dropEffect = 'move'
-  }
-}
-
-const drop = (event: DragEvent, dropIndex: number) => {
-  event.preventDefault()
-  if (draggedIndex.value !== null && draggedIndex.value !== dropIndex) {
-    const newImages = [...props.modelValue]
-    const [reorderedItem] = newImages.splice(draggedIndex.value, 1)
-    newImages.splice(dropIndex, 0, reorderedItem)
-    updateImages(newImages)
-  }
-  draggedIndex.value = null
-}
-
-const openOffsetModal = (index: number) => {
-  selectedImageIndex.value = index
-  tempOffset.value = { ...props.modelValue[index].offset }
-}
-
-const closeOffsetModal = () => {
-  selectedImageIndex.value = null
-}
-
-const saveOffset = (index: number) => {
-  const newImages = [...props.modelValue]
-  newImages[index] = {
-    ...newImages[index],
-    offset: { ...tempOffset.value }
-  }
-  updateImages(newImages)
-  closeOffsetModal()
-}
-
-const onOffsetChange = () => {
-  if (selectedImageIndex.value !== null) {
-    emit('tempOffsetChange', selectedImageIndex.value, tempOffset.value)
-  }
-}
-
-watch(tempOffset, onOffsetChange, { deep: true })
-
-const imageDimensions = ref<{ [key: number]: { width: number; height: number } }>({})
-
-const updateImageDimensions = (event: Event, index: number) => {
-  const img = event.target as HTMLImageElement
-  imageDimensions.value[index] = {
-    width: img.naturalWidth,
-    height: img.naturalHeight
-  }
-}
-</script>
diff --git a/src/components/gameMaster/assetManager/partials/sprite/partials/SpriteImagesPreview.vue b/src/components/gameMaster/assetManager/partials/sprite/partials/SpriteImagesPreview.vue
new file mode 100644
index 0000000..2cebb67
--- /dev/null
+++ b/src/components/gameMaster/assetManager/partials/sprite/partials/SpriteImagesPreview.vue
@@ -0,0 +1,38 @@
+<template>
+  <div class="flex flex-wrap gap-3">
+    <div v-for="(image, index) in spriteActionImages" :key="index" class="h-20 w-20 p-4 bg-gray-300 bg-opacity-50 rounded text-center relative group">
+      <img :src="image.url" class="max-w-full max-h-full object-contain pointer-events-none" alt="Uploaded image" @load="updateImageDimensions($event, index)" />
+      <div v-if="imageDimensions[index]" class="absolute bottom-1 right-1 bg-black/50 text-white text-xs px-1 py-0.5 rounded transition-opacity font-default">{{ imageDimensions[index].width }}x{{ imageDimensions[index].height }}</div>
+    </div>
+  </div>
+</template>
+
+<script setup lang="ts">
+import { ref } from 'vue'
+
+interface SpriteImage {
+  url: string
+  offset: {
+    x: number
+    y: number
+  }
+}
+
+interface Props {
+  spriteActionImages: SpriteImage[]
+}
+
+const props = withDefaults(defineProps<Props>(), {
+  spriteActionImages: () => []
+})
+
+const imageDimensions = ref<{ [key: number]: { width: number; height: number } }>({})
+
+const updateImageDimensions = (event: Event, index: number) => {
+  const img = event.target as HTMLImageElement
+  imageDimensions.value[index] = {
+    width: img.naturalWidth,
+    height: img.naturalHeight
+  }
+}
+</script>