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

Increased volume and length of western holster to accommodate other revolvers (#48532) #49883

Merged
merged 3 commits into from Jul 12, 2021

Conversation

ghost
Copy link

@ghost ghost commented Jul 12, 2021

Summary

Bugfixes "Increased volume and length of western holster to accommodate larger revolvers as expected"

Purpose of change

Fixes #48532 - resized holster to fit classic cap and ball firearms as well as other revolvers.

Describe the solution

Due to the western holster being viewed as a holster for classic six-shooters, I have modified the western holster to hold a volume of 855 ml (large enough for the Lemat revolver) and and length of 38 cm (long enough for the Colt M1860 Army and Lemat).

Describe alternatives you've considered

Reducing the size of the revolvers to fit in the smaller holster, though as discussed in the issue this would require a larger overhaul of the guns themselves.

Testing

Have playtested with the increased holster size and can confirm that Letmat, Colt Army and Colt Navy, S&W 10 and S&W 619 can now all be holstered in the main pocket.

@actual-nh actual-nh added <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Items: Containers Things that hold other things labels Jul 12, 2021
@I-am-Erk I-am-Erk merged commit fec9886 into CleverRaven:master Jul 12, 2021
@ghost ghost deleted the testbranch branch July 12, 2021 19:03
ZhilkinSerg added a commit to ZhilkinSerg/Cataclysm-DDA that referenced this pull request Aug 12, 2021
ZhilkinSerg added a commit that referenced this pull request Aug 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) Items: Containers Things that hold other things [JSON] Changes (can be) made in JSON
Projects
None yet
Development

Successfully merging this pull request may close these issues.

western holster has too little storage volume
3 participants