-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Food Item Size Adjustment #39203
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
Food Item Size Adjustment #39203
Conversation
|
Oops, that long baguette is causing some issues... |
|
Grilled cheese guy here, I definitely screwed up by making them 1x1, they should be 1x2 like you have here. I like this, good changes and cleanup. |
Yes, they can't fit in the survival kit anymore (for mimes). I think it's OK to remove them from there. This PR is a good balance change, and mimes can |
|
Mimes will now have a Nutribrick in there starting box instead of a baguette. Also the "combat bakery kit" is a 4x4 box with 4x4 internal storage to fit the baguette. The PR has been updated to reflect this. |
This comment was marked as abuse.
This comment was marked as abuse.
|
I think a tray of brownies would be about the size of a loaf of bread, plus they already can be cut like pizza. I think the important thing is that a whole tray of brownies is not a 1x1 any more. |
|
baguette 1 by 3? and put it vertically? |
I don't think that would help. The survival boxes only have a 1x2 space for their food item, and the combat bakery kit is jammed full from how the croissants are handgun shaped now. It would be nice if baguettes would stay 1x4 to match the area of the now 2x2 bread loafs. |
|
oops, clicking random buttons on github. sorry about that. |
Resources/Prototypes/Entities/Objects/Consumable/Food/Baked/bread.yml
Outdated
Show resolved
Hide resolved
No worries. When you're done making changes feel free to hit the request review button. It lets me know you're ready for me to review it so I don't forget. I noticed one thing and the rest looks good. |
|
Test fail real. Seems you need to update either the HappyHonk fill table or adjust the container size of the HappyHonk. |
I'm confused by that test fail because I couldn't recreate it running the test locally. I'm guessing because the crazy burger is bigger now, there was no space for it. I removed it from that fill list, and it looks like its working now. |
Yeah that's the exact issue, should also remove any other food item that's too big. That test specifically will fail randomly because the contain fills are well... random. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. I think the crazy hamburger being removed from happy honks is only a minor loss considering people were gambling for the toys.
|
P1 due to #40046 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's some minor issue with bowls where they still use the crunch sound instead of drinking, but the prototype itself seems set up correctly. Should be fixed but I see no point in delaying the PR for such a minor error.
Thank you for your contribution.
| name: cornuto | ||
| parent: FoodFrozenBase | ||
| id: FoodFrozenCornuto | ||
| name: strawberry ice-cream sandwich |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like a copy paste issue, gonna fix that soon
Why is a pat of butter bigger then a whole tray of brownies?
About the PR
I went through all of the food/chef items, and tweaked their size and sprite offsets.
Why / Balance
I play a lot of chef and have noticed some common issues with many of their items:
Items are too big or too small.
Many items inherit the default 1x2 size, regardless if it makes sense. Items that can be sliced (butter, raw meat) turn into many items that are the same size. Very big meals with lots of ingredients and reagents are the same size as a normal sandwich.
Sprites spilling over the edge of their item frame.
This sort of ties in with the previous point. Many items have larger or some times fantastical sprites, but still have the boring old 1x2 size. Many sprites are just too big for their item frame, or need to have there item frame rotated to fit the sprite.
Sprites are off center in their item frames.
Not much to say. Egg cartons, Pizzas, and pizza boxes are some of the worst offenders.
Items by default get placed sideways in inventories.
Not the end of the world, but its odd to have so many things rotated 90 degrees from how they appear in the world. Soups, and plates of food look especially strange shoved sideways.
Here are some of the new items sizes. Some are bigger now, some are smaller. This is not an exhaustive list, but should cover the biggest changes:
Update:
Because of the new size of baguettes, mimes will now start with a Nutribrick instead. The "combat bakery kit" has to be a 4x4 to accommodate the new baguette size.
Technical details
I got rid of the -90 stored rotation offsets. Some custom item shapes were use to accommodate this.
Just about every food item was spawned in and given a custom storage offset, if needed.
I did a small rename of the plastic plates found in the condiment vender. Both the smaller plate and larger plate were both named "plastic plate". I renamed them to "large plastic plate" and "small plastic plate" to match the naming convention of their ceramic counter parts.
Media
All items have been placed with there default rotation.
Before:
After:
Before:
After:
Requirements
Breaking changes
None, yml changes only.
Changelog
🆑