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

Kitchen cleanup: Cogmap 1 #17658

Merged

Conversation

CalliopeSoups
Copy link
Contributor

[A-Mapping] [C-QoL]

About the PR

While clutter can improve the flavour of an area, I believe some kitchens go a bit far. This is planned to be a set of PRs that increase space in catering storages, increase the amount of empty table space the chef has to work with and neaten up anything in the way.

Why's this needed?

It's not evident much on 90-100 minute shifts, but in a classic timeframe cleaning your space up can be a not so insignificant amount of time. Plus, more space is always nice.

This PR specifically also shuffles some pipes around to reduce the number of under-wall pipes in the affected area.

Changelog

(u)CalliopeSoups
(+)Cogmap1 catering has been neatened up and widened a little.

@boring-cyborg boring-cyborg bot added the A-Mapping A mapping change label Jan 14, 2024
@github-actions github-actions bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jan 14, 2024
@CalliopeSoups
Copy link
Contributor Author

Low-res overview of the affected area
image

Copy link
Contributor

@TobleroneSwordfish TobleroneSwordfish left a comment

Choose a reason for hiding this comment

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

Neat, but a liittle too much decluttering imo:

  • No one's going to answer that phone, keep it in the main area
  • The dramatic chef horn is funny, I will die on this hill
  • Having a random loaf of bread on a plate and a knife is good to give newbie chefs something to start from, they can at least make a sandwich or something

The rest is mostly useless clutter.

@CalliopeSoups
Copy link
Contributor Author

I'll re-add those in a bit!

@github-actions github-actions bot added the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Jan 17, 2024
@pgmzeta pgmzeta added the C-QoL A quality of life improvement that makes the game easier to play label Jan 17, 2024
@TobleroneSwordfish TobleroneSwordfish merged commit ea1a24c into goonstation:master Jan 29, 2024
22 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Mapping A mapping change C-QoL A quality of life improvement that makes the game easier to play S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants