always execute display area update function on window resize

and remember to remove the event listener on dispose. Resolves #18
SunnyLi · Jan 9, 2017 · b8064f6 · b8064f6
1 parent cbaf2cd
commit b8064f6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "videojs-ass",
-  "version": "0.6.2",
+  "version": "0.6.3",
   "author": "Sunny Li",
   "description": "ASS/SSA subtitle overlay for videojs",
   "repository": {

diff --git a/src/videojs.ass.js b/src/videojs.ass.js
@@ -77,16 +77,14 @@
       }, 100);
     }
 
-    if (player.fluid()) {
       window.addEventListener('resize', updateDisplayArea);
-    }
-
     player.on('loadedmetadata', updateDisplayArea);
     player.on('resize', updateDisplayArea);
     player.on('fullscreenchange', updateDisplayArea);
 
     player.on('dispose', function () {
       clock.disable();
+      window.removeEventListener('resize', updateDisplayArea);
     });
 
     libjass.ASS.fromUrl(options.src, libjass.Format.ASS).then(