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

feat(content): Thick socks #4079

Merged
merged 3 commits into from
Jan 10, 2024
Merged

Conversation

yay855
Copy link
Contributor

@yay855 yay855 commented Jan 9, 2024

Purpose of change

Normal socks are not very warm, but wool socks can be annoying to acquire. Thicker cotton socks makes perfect sense.

Describe the solution

Added in new item, thick socks, and corresponding recipe. They're half as warm as wool socks.

Describe alternatives you've considered

Testing

Additional context

Checklist

Added in thick socks as an item- literally just thicker cotton socks. Also corrected armored sandals name to 'pair of armored sandals' to match other forms of footwear.
@github-actions github-actions bot added the JSON related to game datas in JSON format. label Jan 9, 2024
Copy link
Collaborator

@RoyalFox2140 RoyalFox2140 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I own a pair of something like this IRL and I can say its at least 2 or 3 encumbrance because its slightly harder to walk around in. The extra material is a bit cumbersome but not by that much. I'd also say for balance we tend to have winter clothing give higher encumbrance, especially because of layering blunt and cut armor in winter to fight zombies without taking damage, so it needs a trade-off so it would follow more internal balance consistency to encumber a player.
image

@scarf005 scarf005 merged commit c5e11ff into cataclysmbnteam:main Jan 10, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JSON related to game datas in JSON format.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants