-
Notifications
You must be signed in to change notification settings - Fork 288
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
revert: bring shotgun ammo AoE again^2 #6034
Conversation
This reverts commit b77a71f.
That reminds me, at some point some accidental tweaks to the damage values slipped through that should not have, I'll do a separate PR in a bit to fix that. |
i guess we could merge the accidental tweaks here |
If you want I'll put up the suggestions for them in a sec then, sure. EDIT: At least one of them is out of range to make a code suggestion actually so oops, I'll just PR it separately. XD |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Flechette and Buckshot should never be AoE, we can add new shell types for this but this is a regression on the playability of the game.
With damage falloff on AoE this is just a dramatic nerf to the actual usability of them as ammunition, we can add more shell types for this. A flat revert is a Hard No.
If all the shells except slugs are aoe then you just have 3 degrees of the same thing, you have high aoe at low damage and low arpen, medium aoe at medium damage and low arpen, and then low aoe at medium damage and medium arpen, I.E. Birdshot, 00 Shot, Flechette
So we could add #4 shot instead, and now we have Buckshot AoE behavior on a different shell without making everything the same.
AoE buckshot dramatically changes the primary single target attack option for guns with one of the most common ammo types
Buckshot is in it's own niche as a great ammo type for players without gun skill and with average perception, and now it's gone. Flechettes are their own niche for high armor penetration buckshot and now that's gone too.
Just to be sure since we have at least one vital prereq PR whose balance is still undecided before this can be merged, I went ahead and converted back to draft for safekeeping. |
Purpose of change (The Why)
#4924 (comment)
#6028 and #5981 is resolved which means AoE shotshells can be reintroduced again
blocked by #6035
Describe the solution (The How)
reverts #6027
Describe alternatives you've considered
remove damage falloff and
or
making shots blocked by targets? but this makes shotguns weaker on point-blank range which i don't like (will only hit 1 targets)
Testing
Additional context
BEHOLD MY UN-NERF INATOR
https://www.youtube.com/watch?v=mHtq-mlzY7s
Checklist
Mandatory
closes #1234
in Summary of the PR so it can be closed automatically.main
so it won't cause conflict when updatingmain
branch later.