InputManager.hitTest will now factor the game resolution into account, stopping the tests from being offset if resolution didn't equal 1. phaserjs#3376

Author: photonstorm

CHANGELOG.md

@@ -17,6 +17,7 @@
 * Fixed bug in `DataManager.merge` where it would copy the object reference instead of its value (thanks @rexrainbow)
 * The SceneManager no longer copies over the `shutdown` and `destroy` callbacks in createSceneFromObject, as these are not called automatically and should be invoked via the Scene events (thanks @samme)
 * The default Gamepad Button threshold has been changed from 0 to 1. Previously the value of 0 was making all gamepad buttons appear as if they were always pressed down (thanks @jmcriat)
+* InputManager.hitTest will now factor the game resolution into account, stopping the tests from being offset if resolution didn't equal 1 (thanks @sftsk)
 
 ### Updates
 

src/input/InputManager.js

@@ -381,6 +381,8 @@ var InputManager = new Class({
 
         var point = { x: 0, y: 0 };
 
+        var res = this.game.config.resolution;
+
         for (var i = 0; i < culledGameObjects.length; i++)
         {
             var gameObject = culledGameObjects[i];
@@ -390,8 +392,8 @@ var InputManager = new Class({
                 continue;
             }
 
-            var px = tempPoint.x + (camera.scrollX * gameObject.scrollFactorX) - camera.scrollX;
-            var py = tempPoint.y + (camera.scrollY * gameObject.scrollFactorY) - camera.scrollY;
+            var px = tempPoint.x * res + (camera.scrollX * gameObject.scrollFactorX) - camera.scrollX;
+            var py = tempPoint.y * res + (camera.scrollY * gameObject.scrollFactorY) - camera.scrollY;
 
             TransformXY(px, py, gameObject.x, gameObject.y, gameObject.rotation, gameObject.scaleX, gameObject.scaleY, point);