The setTintFill method would ignore the alpha value of the Game Object in the shader. The alpha value is now blended with the tint fill, allowing you to properly alpha out tint-filled Game Objects. Fix phaserjs#3992

Author: photonstorm

CHANGELOG.md

@@ -201,6 +201,7 @@ You can find the source for Camera 3D in the new `plugins/camera3d` folder, alon
 * Arcade Physics World has gained two new private properties `_tempMatrix` and `_tempMatrix2`. These are used by all bodies in the simulation that need a temporal matrix for calculations, rather than having their own instances.
 * The Input Manager has gained a new private property `_tempMatrix2`. This is used internally in the hitTest checks to avoid constant matrix creation.
 * The Transform Matrix has a new method `applyInverse` which will take an x/y position and inverse translate it through the current matrix.
+* Using `keyboard.addKeys("W, A, S, D")` would fail because of the spacing between the characters. `addKeys` will now trim the input allowing you to space characters out if you prefer (thanks @dhruvyad)
 
 ### Game Config Resolution Specific Bug Fixes
 
@@ -246,6 +247,7 @@ Setting the `resolution` property in the Game Config to a value other than 1 wou
 * The `InputManager.inputCandidate` method, which determines if a Game Object can be interacted with by a given Pointer and Camera combination, now takes the full camera status into consideration. This means if a Camera is set to ignore a Game Object you can now longer interact with it, or if the Camera is ignoring a Container with an interactive Game Object inside it, you cannot interact with the Container children any more. Previously they would interact regardless of the Camera state. Fix #3984 (thanks @NemoStein @samid737)
 * `Transform.getWorldTransformMatrix` has been recoded to iterate the transform parents correctly, applying the matrix multiplications as it goes. This (along with some changes in the Input Manager) fix the issue with Game Objects inside of Containers failing hit tests between certain angles. Fix #3920 (thanks @chaping @hackhat)
 * Calling Arcade Physics `collide` during an `update` method wouldn't inject the results back into the Body parent, causing the bodies to carry on moving. Using Colliders worked, but manually checking did not. Now, both methods work. Fix #3777 (thanks @samme)
+* The `setTintFill` method would ignore the `alpha` value of the Game Object in the shader. The alpha value is now blended with the tint fill, allowing you to properly alpha out tint-filled Game Objects. Fix #3992 (thanks @trl-bsd)
 
 ### Examples, Documentation and TypeScript
 

src/renderer/webgl/shaders/GBuffer-frag.js

@@ -17,13 +17,14 @@ module.exports = [
     '',
     '    if (outTintEffect == 0.0)',
     '    {',
-    '        //  Multiply tint',
+    '        //  Multiply texture tint',
     '        color = texture * texel;',
     '    }',
     '    else if (outTintEffect == 1.0)',
     '    {',
-    '        //  Solid texture-based tint',
-    '        color.rgb = mix(texture.rgb, outTint.rgb, texture.a);',
+    '        //  Solid color + texture alpha',
+    '        color.rgb = mix(texture.rgb, outTint.rgb * outTint.a, texture.a);',
+    '        color.a = texture.a * texel.a;',
     '    }',
     '    else if (outTintEffect == 2.0)',
     '    {',

src/renderer/webgl/shaders/TextureTint-frag.js

@@ -22,7 +22,8 @@ void main()
     else if (outTintEffect == 1.0)
     {
         //  Solid color + texture alpha
-        color.rgb = mix(texture.rgb, outTint.rgb, texture.a);
+        color.rgb = mix(texture.rgb, outTint.rgb * outTint.a, texture.a);
+        color.a = texture.a * texel.a;
     }
     else if (outTintEffect == 2.0)
     {