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

shoulderholster storage increase, small pistol magazine pouch can also hold pistol #6081

Closed
wants to merge 1 commit into from

Conversation

cuberound
Copy link
Contributor

@cuberound cuberound commented Apr 4, 2024

About the pull request

work in progress, the pistol magazine pouch holster will replace small magazine pouch idealy. needs sprite for said pouch.
current pistol stuff numbers are: webbing can hold pistol and 2 mags ( you can carry more in drop pouch). Small pistol pouch is just... trap it can carry 3 pistol mags, meanwhile you can carry 3 rifle mags in rifle mag pouch ( and pistol mags too). large pistol magazine pouch can carry 6 mags, that is kinda fine. pistol pouch can carry one pistol, that is just shit... The changes make shoulderholster able to carry 4 extra mags instead of 2. merge pistol pouch and small pistol magazine pouch into one aka making small magazine pouch able to hold pistol and 3 pistol mags.

changlelog not up to date will change after the changes are agreed on

Explain why it's good for the game

this should bring those storage options more in line with the other options and hopefuly allow use of sidearms and make them viable for medics and engineers for self defense when they do not need long rifle but want to use storage that allows for more then three engagments before running out of ammo
in terms of balance, this does not really effect pistols as primary weapon as for that people already just carry ammoboxes so few more mags in a pouch is not going to make it an issue. pistols as primary are extramly rare so that should not be an issue

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
balance: small pistol magazine pouch can not also hold pistol
balance: shoulder holster can carry 4 mags from 2
/:cl:

@github-actions github-actions bot added the Balance You need to be a professional veteran game maintainer to comprehend what is being done here. label Apr 4, 2024
@Drulikar
Copy link
Contributor

Drulikar commented Apr 4, 2024

I don't think these changes are necessary; nor do I have anything to suggest you should explore instead.

@Drulikar Drulikar added the Do Not Merge If you merge this PR, I will annihilate you label Apr 4, 2024
@cuberound cuberound closed this Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Do Not Merge If you merge this PR, I will annihilate you
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants