xevion/Pac-Man
1
0
Fork 0
mirror of https://github.com/Xevion/Pac-Man.git synced 2026-01-31 00:24:59 -06:00
Code Issues Packages Projects Releases Wiki Activity

fix(web): prevent stopGame from being called on non-game page transitions

Browse Source 
Only stop the game when navigating away from the game page (/) to avoid
WASM runtime errors when navigating between /leaderboard and /download.
Add error handling to gracefully handle crashes during page transitions.
This commit is contained in:
Xevion
2025-12-29 14:53:42 -06:00
parent 21078b7ada
commit e017a87e12
2 changed files with 31 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files
web/pages/+onPageTransitionEnd.ts
+19 -16
View File
@@ -23,23 +23,26 @@ function restartGame() {
const win = getPacmanWindow();
const module = win.Module;
if (module?._restart_game) {
const canvas = document.getElementById("canvas") as HTMLCanvasElement | null;
if (!canvas) {
console.error("Canvas element not found during game restart");
return;
}
if (!module?._restart_game) {
console.warn("Game restart function not available (WASM may not be initialized)");
return;
}
// Update canvas reference BEFORE restart - App::new() will read from Module.canvas
module.canvas = canvas;
// SDL2's Emscripten backend reads this for canvas lookup
win.SDL_CANVAS_ID = "#canvas";
const canvas = document.getElementById("canvas") as HTMLCanvasElement | null;
if (!canvas) {
console.error("Canvas element not found during game restart");
return;
}
try {
console.log("Restarting game with fresh App instance");
module._restart_game();
} catch (error) {
console.error("Failed to restart game:", error);
}
// Update canvas reference BEFORE restart - App::new() will read from Module.canvas
module.canvas = canvas;
// SDL2's Emscripten backend reads this for canvas lookup
win.SDL_CANVAS_ID = "#canvas";
try {
console.log("Restarting game with fresh App instance");
module._restart_game();
} catch (error) {
console.error("Failed to restart game:", error);
}
}
web/pages/+onPageTransitionStart.ts
+12 -5
View File
@@ -10,11 +10,18 @@ export const onPageTransitionStart: OnPageTransitionStartAsync = async (pageCont
setPendingNavigation(pageContext.urlPathname);
document.querySelector("body")?.classList.add("page-is-transitioning");
// Stop the game loop when navigating away from the game page
const win = getPacmanWindow();
if (win.Module?._stop_game) {
console.log("Stopping game loop for page transition");
win.Module._stop_game();
// Only stop the game when navigating AWAY FROM the game page
// Don't stop when navigating between other pages (e.g., /leaderboard <-> /download)
if (window.location.pathname === "/") {
const win = getPacmanWindow();
if (win.Module?._stop_game) {
try {
console.log("Stopping game loop for page transition");
win.Module._stop_game();
} catch (error) {
console.warn("Failed to stop game (game may have already crashed):", error);
}
}
}
// Wait for fade-out animation to complete before page content changes