Fixed overhead icons issue (#1861)

Author: mexikoedi

CHANGELOG.md

@@ -23,6 +23,7 @@ All notable changes to TTT2 will be documented here. Inspired by [keep a changel
 - Fixed continuous use not working correctly with entities like health stations (by @saibotk)
 - Fixed vFire explosions still killing players with `NoExplosionDamage` equipped (by @TW1STaL1CKY)
 - Fixed vFire explosions sometimes not damaging entities in the way they should, like explosive barrels (by @TW1STaL1CKY)
+- Fixed an issue where overhead icons from `TTT2ModifyOverheadIcon` were ignored when `shouldDrawDefault` was false (by @mexikoedi)
 
 ### Removed / Breaking Changes
 

gamemodes/terrortown/gamemode/client/cl_targetid.lua

@@ -197,7 +197,7 @@ function GM:PostDrawTranslucentRenderables(bDrawingDepth, bDrawingSkybox)
         local shouldDraw, material, color =
             hook.Run("TTT2ModifyOverheadIcon", ply, shouldDrawDefault)
 
-        if shouldDraw == false or not shouldDrawDefault then
+        if shouldDraw == false or (shouldDraw ~= true and not shouldDrawDefault) then
             continue
         end