Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New CVar: mp_team_flash #693

Merged
merged 3 commits into from
Sep 5, 2023
Merged

Conversation

aleeperezz16
Copy link
Contributor

@aleeperezz16 aleeperezz16 commented Oct 24, 2021

Related to #572

Instead of a "No Team Flash" idea, this command alters the Flashbang's performance.
You can flash your teammates, flash yourself only or neither two cases.
This also affects on mp_freeforall as the players on your team are now enemies.

@StevenKal
Copy link
Contributor

StevenKal commented Oct 25, 2021

Good, but just "max" the CVar to -1 in order to consider other negative values admins could put (like -5).
->
Q_max((int)teamflash.value, -1)

@Vaqtincha
Copy link
Contributor

#451
my version was better (you could set the transparency / duration of the effect) It's a pity. no one wants to accept my pr. They hang dumbly like a woman's breasts.

@StevenKal
Copy link
Contributor

StevenKal commented Oct 26, 2021

my version was better (you could set the transparency / duration of the effect) It's a pity. no one wants to accept my pr.

Devs are a bit slow to accept PRs, I think they usually need to get "shaked" in order to make them hurry up, every user seems concerned, you are not the only one.
Besides, if you "close your PR" like you did before for some once "this takes too much time to be merged", or when people do suggestions you do not like, do not expect it to be in the official release!
Altering the duration & cie can be interesting, but his version can also skip flasher. So maybe two CVars could be added regarding this, in order to control the "damage" in the "RadiusFlash" function.

They hang dumbly like a woman's breasts.

Haha like your cock when you do not look deeply at them!

@s1lentq s1lentq merged commit ee34b06 into s1lentq:master Sep 5, 2023
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.

4 participants