From 2d48f8380244a6123147e32100153658fa7af42c Mon Sep 17 00:00:00 2001
From: Dennis Postma <dennis@directonline.io>
Date: Wed, 22 Jan 2025 00:31:14 +0100
Subject: [PATCH] Load tiles before showing map editor UI

---
 src/components/game/map/Effects.vue           |  2 +-
 .../mapEditor/mapPartials/MapTiles.vue        |  4 ---
 src/components/screens/Game.vue               |  2 +-
 src/components/screens/MapEditor.vue          | 16 ++++++++-
 .../game/scene/loaderComposable.ts            | 34 -------------------
 5 files changed, 17 insertions(+), 41 deletions(-)
 delete mode 100644 src/composables/game/scene/loaderComposable.ts

diff --git a/src/components/game/map/Effects.vue b/src/components/game/map/Effects.vue
index f2f5ae6..9181640 100644
--- a/src/components/game/map/Effects.vue
+++ b/src/components/game/map/Effects.vue
@@ -4,8 +4,8 @@
 
 <script setup lang="ts">
 import type { Map, WeatherState } from '@/application/types'
-import { useGameStore } from '@/stores/gameStore'
 import { MapStorage } from '@/storage/storages'
+import { useGameStore } from '@/stores/gameStore'
 import { useMapStore } from '@/stores/mapStore'
 import { Scene } from 'phavuer'
 import { onBeforeUnmount, onMounted, ref, watch } from 'vue'
diff --git a/src/components/gameMaster/mapEditor/mapPartials/MapTiles.vue b/src/components/gameMaster/mapEditor/mapPartials/MapTiles.vue
index 5676da1..4b55b04 100644
--- a/src/components/gameMaster/mapEditor/mapPartials/MapTiles.vue
+++ b/src/components/gameMaster/mapEditor/mapPartials/MapTiles.vue
@@ -216,10 +216,6 @@ onMounted(async () => {
   scene.input.on(Phaser.Input.Events.POINTER_DOWN, tilePicker)
 })
 
-onBeforeMount(async () => {
-  await loadAllTilesIntoScene(scene)
-})
-
 onUnmounted(() => {
   scene.input.off(Phaser.Input.Events.POINTER_MOVE, pencil)
   scene.input.off(Phaser.Input.Events.POINTER_MOVE, eraser)
diff --git a/src/components/screens/Game.vue b/src/components/screens/Game.vue
index e694bea..4f9dc94 100644
--- a/src/components/screens/Game.vue
+++ b/src/components/screens/Game.vue
@@ -20,7 +20,6 @@
 <script setup lang="ts">
 import config from '@/application/config'
 import 'phaser'
-import Effects from '@/components/game/map/Effects.vue'
 import CharacterProfile from '@/components/game/gui/CharacterProfile.vue'
 import Chat from '@/components/game/gui/Chat.vue'
 import Clock from '@/components/game/gui/Clock.vue'
@@ -28,6 +27,7 @@ import ExpBar from '@/components/game/gui/ExpBar.vue'
 import Hotkeys from '@/components/game/gui/Hotkeys.vue'
 import Hud from '@/components/game/gui/Hud.vue'
 import Menu from '@/components/game/gui/Menu.vue'
+import Effects from '@/components/game/map/Effects.vue'
 import Map from '@/components/game/map/Map.vue'
 import { useGameStore } from '@/stores/gameStore'
 import { Game, Scene } from 'phavuer'
diff --git a/src/components/screens/MapEditor.vue b/src/components/screens/MapEditor.vue
index 5b7e06c..7528ede 100644
--- a/src/components/screens/MapEditor.vue
+++ b/src/components/screens/MapEditor.vue
@@ -2,7 +2,8 @@
   <div class="flex justify-center items-center h-dvh relative">
     <Game :config="gameConfig" @create="createGame">
       <Scene name="main" @preload="preloadScene" @create="createScene">
-        <MapEditor :key="JSON.stringify(`${mapEditorStore.map?.id}_${mapEditorStore.map?.createdAt}_${mapEditorStore.map?.updatedAt}`)" />
+        <MapEditor :key="JSON.stringify(`${mapEditorStore.map?.id}_${mapEditorStore.map?.createdAt}_${mapEditorStore.map?.updatedAt}`)" v-if="isLoaded" />
+        <div v-else class="absolute top-1/2 left-1/2 -translate-x-1/2 -translate-y-1/2 text-white text-2xl">Loading...</div>
       </Scene>
     </Game>
   </div>
@@ -12,13 +13,17 @@
 import config from '@/application/config'
 import 'phaser'
 import MapEditor from '@/components/gameMaster/mapEditor/MapEditor.vue'
+import { loadAllTilesIntoScene } from '@/composables/mapComposable'
 import { useGameStore } from '@/stores/gameStore'
 import { useMapEditorStore } from '@/stores/mapEditorStore'
 import { Game, Scene } from 'phavuer'
+import { ref } from 'vue'
 
 const gameStore = useGameStore()
 const mapEditorStore = useMapEditorStore()
 
+const isLoaded = ref(false)
+
 const gameConfig = {
   name: config.name,
   width: window.innerWidth,
@@ -49,6 +54,15 @@ const preloadScene = async (scene: Phaser.Scene) => {
   scene.load.image('TELEPORT', '/assets/map/tp_tile.png')
   scene.load.image('blank_tile', '/assets/map/blank_tile.png')
   scene.load.image('waypoint', '/assets/waypoint.png')
+
+  await loadAllTilesIntoScene(scene)
+
+  await new Promise<void>((resolve) => {
+    scene.load.on(Phaser.Loader.Events.COMPLETE, () => {
+      resolve()
+    })
+    isLoaded.value = true
+  })
 }
 
 const createScene = async (scene: Phaser.Scene) => {}
diff --git a/src/composables/game/scene/loaderComposable.ts b/src/composables/game/scene/loaderComposable.ts
deleted file mode 100644
index a49b385..0000000
--- a/src/composables/game/scene/loaderComposable.ts
+++ /dev/null
@@ -1,34 +0,0 @@
-export function createSceneLoader(scene: Phaser.Scene) {
-  const width = scene.cameras.main.width
-  const height = scene.cameras.main.height
-
-  const progressBox = scene.add.graphics()
-  const progressBar = scene.add.graphics()
-  progressBox.fillStyle(0x222222, 0.8)
-  progressBox.fillRect(width / 2 - 180, height / 2, 320, 50)
-
-  const loadingText = scene.make.text({
-    x: width / 2,
-    y: height / 2 - 50,
-    text: 'Loading...',
-    style: {
-      font: '20px monospace',
-      // @ts-ignore
-      fill: '#ffffff'
-    }
-  })
-  loadingText.setOrigin(0.5, 0.5)
-
-  scene.load.on(Phaser.Loader.Events.PROGRESS, function (value: any) {
-    progressBar.clear()
-    progressBar.fillStyle(0x368f8b, 1)
-    progressBar.fillRect(width / 2 - 180 + 10, height / 2 + 10, 300 * value, 30)
-  })
-
-  scene.load.on(Phaser.Loader.Events.COMPLETE, function () {
-    progressBar.destroy()
-    progressBox.destroy()
-    loadingText.destroy()
-    return true
-  })
-}