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

Add precise strike to batons/tonfas, remove from qiang, fix description #57015

Merged
merged 1 commit into from
Apr 22, 2022

Conversation

Aerin-of-the-Toast
Copy link
Contributor

Summary

None

Purpose of change

As per #56953 "precise strike" is meant to represent a weapon being able to "harry" an opponent, an important part of the identity of the cudgel. The two batons and the three tonfas are used in a similar manner and also fall under a similar "light, speedy harrying bludgeon" category. Also, the technique's description wasn't updated for the new melee 3 requirement.

Describe the solution

Added precise strike to the wooden tonfa, the tonfa, the tactical tonfa, and the extended forms of the PR24 baton and expandable baton. Removed precise strike from the qiang, as it was an outlier being the only weapon with precise strike that wasn't a baton or fencing weapon. Added "min 3 melee" to the description of precise strike.

Describe alternatives you've considered

It's possible there are some other improvised light bludgeons that could qualify for precise strike, like a wrench or the like, but the ones I looked at I figured were a bit too unwieldy/not agile enough in the hand to qualify, usually being metal tools that are a bit top-heavy. I also only looked at weapons in the "batons" category, so there's possibly some "baton-like" weapons I missed as well that aren't in that category for some reason.

Testing

Loaded up the game and made sure the description updated properly and the weapons had the technique properly.

Additional context

I didn't look at the "fencing weapons" side of things at all, only the batons, so that half of the precise strike list possibly needs a look as well.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Apr 22, 2022
@I-am-Erk I-am-Erk merged commit 61562af into CleverRaven:master Apr 22, 2022
@RandDenner
Copy link
Contributor

RandDenner commented Apr 22, 2022

Yeah, I think the fencing weapons are just a mistake.
I suspect the "precise" technique was added for the wrong reason: that they are weapons that are used to strike precisely so as to damage an opponent. Without regard to the fact that "precise" = stunning strike.

Maybe the whole "precise" business should be re-named?

@I-am-Erk
Copy link
Member

All of the weapon special attacks need revisiting and integration with weakpoints, but that's a conversation for another place.

@NetSysFire NetSysFire added the Melee Melee weapons, tactics, techniques, reach attack label Apr 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Melee Melee weapons, tactics, techniques, reach attack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants