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

implement pocket max item size #40214

Merged
merged 1 commit into from
May 6, 2020

Conversation

KorGgenT
Copy link
Member

@KorGgenT KorGgenT commented May 5, 2020

Summary

SUMMARY: Features "implement pocket max item size"

Purpose of change

We needed a few more attributes to limit items that can be placed in other items. This and length i think should cover the vast majority of situations.

Describe the solution

add a new member to pocket_data, read it, add it to item_pocket:;can_contain and item_pocket::info functions to make them usable.

Describe alternatives you've considered

aperture size and adding dimension values to every item

Testing

edited a 2 liter bottle to have this new member, based on having a 1" opening and doing a simple calculation. spawned one in, and tried to pick up things. they asked me to wield them instead.

Additional context

image

@KorGgenT KorGgenT added <Enhancement / Feature> New features, or enhancements on existing [C++] Changes (can be) made in C++. Previously named `Code` Items: Containers Things that hold other things labels May 5, 2020
@KorGgenT KorGgenT force-pushed the max-item-size-in-pocket branch from de34675 to f8f68f9 Compare May 5, 2020 21:50
@KorGgenT KorGgenT force-pushed the max-item-size-in-pocket branch from f8f68f9 to e0e975b Compare May 6, 2020 17:14
@ZhilkinSerg ZhilkinSerg merged commit 646e397 into CleverRaven:master May 6, 2020
@KorGgenT KorGgenT deleted the max-item-size-in-pocket branch August 13, 2020 04:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` <Enhancement / Feature> New features, or enhancements on existing Items: Containers Things that hold other things
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants