Always use getAABB for bitmap skins

Author: ktbee

src/BitmapSkin.js

@@ -23,6 +23,9 @@ class BitmapSkin extends Skin {
 
         /** @type {Array<int>} */
         this._textureSize = [0, 0];
+
+        /** @type {string} */
+        this._typeOfSkin = 'BITMAP';
     }
 
     /**

src/PenSkin.js

@@ -88,6 +88,9 @@ class PenSkin extends Skin {
         /** @type {HTMLCanvasElement} */
         this._canvas = document.createElement('canvas');
 
+        /** @type {string} */
+        this._typeOfSkin = 'PEN';
+
         /** @type {WebGLTexture} */
         this._texture = null;
 

src/RenderWebGL.js

@@ -1316,9 +1316,15 @@ class RenderWebGL extends EventEmitter {
 
         const dx = x - drawable._position[0];
         const dy = y - drawable._position[1];
-        const aabb = drawable.getFastBounds();
-        const inset = Math.floor(Math.min(aabb.width, aabb.height) / 2);
+        let aabb;
 
+        if (drawable._skin.typeOfSkin === 'BITMAP') {
+            aabb = drawable.getAABB();
+        } else {
+            aabb = drawable.getFastBounds();
+        }
+
+        const inset = Math.floor(Math.min(aabb.width, aabb.height) / 2);
         const sx = this._xRight - Math.min(FENCE_WIDTH, inset);
         if (aabb.right + dx < -sx) {
             x = Math.ceil(drawable._position[0] - (sx + aabb.right));

src/SVGSkin.js

@@ -30,6 +30,9 @@ class SVGSkin extends Skin {
 
         /** @type {Number} */
         this._maxTextureScale = 0;
+
+        /** @type {string} */
+        this._typeOfSkin = 'SVG';
     }
 
     /**

src/Skin.js

@@ -30,6 +30,9 @@ class Skin extends EventEmitter {
         /** @type {int} */
         this._id = id;
 
+        /** @type {string} */
+        this._typeOfSkin = 'SKIN';
+
         /** @type {Vec3} */
         this._rotationCenter = twgl.v3.create(0, 0);
 
@@ -98,6 +101,14 @@ class Skin extends EventEmitter {
         return [0, 0];
     }
 
+    /**
+     * @abstract
+     * @return {string} the type of skin, as determined by the class's properties.
+     */
+    get typeOfSkin () {
+        return this._typeOfSkin;
+    }
+
     /**
      * Set the origin, in object space, about which this Skin should rotate.
      * @param {number} x - The x coordinate of the new rotation center.