Skip to content

[TF2] Prevent inspecting weapons in awkward scenarios#1118

Open
Tkain wants to merge 6 commits intoValveSoftware:masterfrom
Tkain:flamethrower-inspect-fix
Open

[TF2] Prevent inspecting weapons in awkward scenarios#1118
Tkain wants to merge 6 commits intoValveSoftware:masterfrom
Tkain:flamethrower-inspect-fix

Conversation

@Tkain
Copy link
Copy Markdown
Contributor

@Tkain Tkain commented Mar 16, 2025

Description

In TF2, multiple weapons can be inspected at times that visually interrupt other actions or lead to the inspect animation being interrupted. One such example is with the flamethrower, whose inspection while being fired make its flames seem to not obey the direction of the flamethrower itself (ignore the background audio, sorry):

Inspecting.flamethrower.while.firing.mp4

This PR adds additional restrictions to when certain weapons can be inspected, preventing players from encountering visual issues like these. Currently, this includes:

  • Preventing flamethrowers from being inspected while firing
  • Preventing Medi Guns from being inspected while healing a target
  • Preventing bows, sticky bomb launchers, the Loose Cannon, and the Cow Mangler from being inspected while charging a shot
  • Preventing the PASS Time JACK from being inspected at all

This PR addresses some of the issues mentioned in ValveSoftware/Source-1-Games#4273, but does not resolve all of them.

@HyperionCoding
Copy link
Copy Markdown

Other weapons should be checked too while at it. At least minigun comes to my mind.

Tkain added 4 commits March 17, 2025 00:58
It has no inspect animation of its own, so the one it uses looks very broken. It's probably better to disable it for now.
Also applies to stickybomb launchers and the Loose Cannon.
@Tkain Tkain changed the title [TF2] Prevent inspecting the Flamethrower while firing [TF2] Prevent inspecting weapons in awkward scenarios Mar 17, 2025
Doing so would also cause the banner to activate after being inspected, which is also likely undesirable.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants