Merge pull request #136 from SpringRoll/feature/pause-plugin-update

Clean up unnecessary focus management

CHANGELOG.md

@@ -5,7 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [2.4.0] unreleased
+## [2.4.1] unreleased
+
+### Fixed
+
+- Fixed PausePlugin focus management so game pauses/unpauses properly on load
+
+## [2.4.0] 2022-07-05
 
 ## Added
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "springroll-container",
-  "version": "2.3.4",
+  "version": "2.4.1",
   "description": "The iframe controller for interacting with SpringRoll applications",
   "main": "./dist/index.js",
   "license": "MIT",

src/plugins/PausePlugin.js

@@ -70,9 +70,7 @@ export class PausePlugin extends ButtonPlugin {
     this._paused = paused;
 
     this.client.send(PausePlugin.pauseKey, paused);
-    this.client.trigger(paused ? 'paused' : 'resumed', {
-      paused: paused
-    });
+    this.client.trigger(paused ? 'paused' : 'resumed', { paused });
 
     for (let i = 0, l = this._pauseButton.length; i < l; i++) {
       this._pauseButton[i].button.classList.remove('unpaused');
@@ -118,7 +116,6 @@ export class PausePlugin extends ButtonPlugin {
       return;
     }
 
-    this._containerBlurred = true;
     this.iframe.contentWindow.focus();
   }
 
@@ -208,7 +205,6 @@ export class PausePlugin extends ButtonPlugin {
    */
   onContainerFocus() {
     this._containerBlurred = false;
-    this.focusApp();
     this.manageFocus();
   }
 
@@ -257,7 +253,7 @@ export class PausePlugin extends ButtonPlugin {
     this.client.on('focus', this.onFocus);
     this.client.on('keepFocus', this.onKeepFocus);
 
-    this.pause = this.pause;
+    this.pause = this._paused;
 
     //ensure app has focus after setting up plugin
     this.focusApp();