fix: switch requestAnimationFrame to setInterval to handle monitors r…

…efresh rates
eugenioenko · Aug 15, 2024 · 37aab62 · 37aab62
1 parent 7c98fc3
commit 37aab62
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 32 deletions.
diff --git a/dist/firenice.js b/dist/firenice.js
diff --git a/dist/firenice.min.js b/dist/firenice.min.js
diff --git a/src/game.js b/src/game.js
@@ -13,26 +13,16 @@ export class Game {
    */
   constructor(canvas, resources) {
     this.time = performance.now();
-    this.skip = 0;
     this.engine = new Engine(canvas, resources);
     this.levels = levels;
     this.state = Consts.GameStatePlay;
     this.engine.sound.soundtrack();
+    this.lastFrame = 0;
     this.gameLoop = this.gameLoop_.bind(this); // jshint ignore:line
-    this.gameLoop();
+    setInterval(() => this.gameLoop(), 1000 / 60);
   }
 
   gameLoop_() {
-    const fps = Math.floor(1 / ((performance.now() - this.time) / 1000));
-    // skip every two frames if framerate is above 60Hz
-    if (fps > 90) {
-      this.skip += 1;
-    }
-    if (this.skip >= 2) {
-      this.skip = 0;
-      window.requestAnimationFrame(this.gameLoop);
-      return;
-    }
     switch (this.state) {
       case Consts.GameStateIntro:
         this.doIntro();
@@ -42,9 +32,6 @@ export class Game {
         this.engine.move();
         break;
     }
-
-    this.time = performance.now();
-    window.requestAnimationFrame(this.gameLoop);
   }
 
   createIntro() {