-
-
Notifications
You must be signed in to change notification settings - Fork 541
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
Sunskipper-class Culinary Vessel - the Boyardee replacement. Also, a lot of tweaks to the booze-o-mat. #2836
Conversation
We're so fucking back |
Very classy and cool. |
Very nice looking and a definite improvement over the original. Give me an @ when it's undrafted and ready for a review. |
Ping me sometime this weekend if I don't review this tonight. |
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.
The base external area makes these tiles fullbright. Use the dark subtype (/area/ship/external/dark).
I would add some cleanable decals throughout the ship, some cracked eggs or dust around for some roundstart cleaning.
...And that is all of the changes I had to request. Yep. I am stunned at the quality of this ship. The layout is good, the decaling is amazing, they start with just the right amount of stuff, and the access restrictions are well-placed. This is an excellent ship that would make it in even if it wasn't replacing an old one. Great job! Once you fix the external areas this should be ready for TM.
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 are a few things I dislike about this room. Your floor pattern is uninterrupted for this whole room, I would hihgly recommend changing up the flooring where there are impassible objects. Do not pixel shift vending machines please. Don't make scrubbers or vents inaccessible, you have a scrubber underneath a vending machine meaning that machine would need to be moved to access it. The lamp on the table is very awkward, obstructing 2 different screens on the walls. That lightswitch is behind a tile visibility muter, deprending on what tile it is actually on, the curtains could potentially hide it, put it on the other side of the door.
Your entire ship has some pretty questionable placements of wall mounts. To be fair, there are not many great examples as to what "good" wall mounts look like. But when placing wall mounts you need to think about more than wall space and room space,, you have to think about their accessibility. And different wall mounts have different accessibility needs.
Emergency cabinets such as the fire extinguisher cabinet, need to be accessible on the tile in front of them as if they were a closet (because they are). There are cases to be made for wall closets to let out onto a table.
APC's need the tile they're on to be accessible due to spawning a terminal on it. Machines and other structures should be avoided. Tables can sometimes be put in front of them if they aren't like, important tables.
You have an empty wall here, put the apc on it, put a cryo panel where the lightbulb or poster are, put a light behind one of the cryopods, and put the air alarm on the free wall behind the crate.
These bathrooms are very nice, I'm very happy you put a crew and public bathroom. Put a bar of soap in the shower.
This is an example of a perfectly acceptable apc location. However this closet being on the same tile as a sink is not. This air alarm is not 100% illegal but I'd definitely prefer it avoided. Here's a suggestion for a layout that could fix this issue.
This does not work, also please lower your buttons to somewhat rest along the floor here?
Overall give your wallmounts a good thorough sanity check.
I'm not entirely sure how I feel about this second airlock being right here. It feels like a strange choice.
Button behind canister, put it on the wall on the other side.
You need a stethoscope to get into this.
All wall lockers need to be a +/-28.
Overall I think this is a nice ship, needs a good look over its wallmounts, but otherwise you did a really solid job.
Idea is that subshuttles can dock there without obscuring the main entrance. It's pretty much only for those, for whenever they get more used.
Zevo told me to hide it - look at the plant! I can make it more obvious tho xP
Sorry... what doesn't? Don't think I'm getting you. Will handle rest of reviews later today! |
sobbign and weepi ng ! @thgvr |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
FUCK, wrong branch |
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 ready for merging.
O-O |
…lot of tweaks to the booze-o-mat. (shiptest-ss13#2836) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> **TODO** - [x] tweaks I noted down somewhere else - [x] address reviews when they pop up - [x] ~~ethics of passengers~~ - [x] cash rebalance - [x] ~~more sofas~~ rework the main room 💀 - [x] fate of the botanist - [x] add gravgen after that's merged - [x] maybe add lathe - [x] add missing regeants to the boozeomat ![scrnshot1](https://github.com/shiptest-ss13/Shiptest/assets/108196626/dd8aeb08-ce2b-41aa-bc8c-67e53c387272) ![StrongDMM-2024-05-12 20 45 19](https://github.com/shiptest-ss13/Shiptest/assets/108196626/cbb82d0d-bd89-4a4b-bbac-dffccbd7ee02) ![StrongDMM-2024-04-17 16 41 07](https://github.com/shiptest-ss13/Shiptest/assets/108196626/10854a47-c53c-4c79-8376-5af9afae0de2) Seeds in the locker (if something important is missing here, slap me with a large stick pls): ![obraz](https://github.com/shiptest-ss13/Shiptest/assets/108196626/34286116-1bec-4bdf-b8bd-5dced6e1e36c) Meds in the locker: ![obraz](https://github.com/shiptest-ss13/Shiptest/assets/108196626/b4d822d3-2a7e-4e39-9d12-9c5425344e1c) Stuff in the safe: ![obraz](https://github.com/shiptest-ss13/Shiptest/assets/108196626/42e6aa18-31c8-4b49-8f71-b4a99e33860a) New bottles! Absolutely not peak sprite design, but I needed _something_, alright? Lemon Juice, Triple Sec, Creme de Cacao, Creme de Menthe: ![obraz](https://github.com/shiptest-ss13/Shiptest/assets/108196626/604e8d99-7c25-4389-bd3f-5efd3ce20aa9) Also, as the bar doesn't have any dispensers, I have added all of dispenser reagents to the booze-o-mat instead. Also also, wiki page (WIP): https://shiptest.net/wiki/User:Sadhorizon#Sunskipper-class_Culinary_Vessel <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> Boyardee is old and really really bad at it's job as a restaurant ship. Sunskipper is meant to replace it. Indie chefs loose cqc, because it's an ability fit for fending off tiders in a station environment - not on a six player ship. Booze-o-mat full of drink-making ingredients is a large step to getting rid of bar dispensers completely. <!-- Please add a short description of why you think these changes would benefit the game. If you can't justify it in words, it might not be worth adding. --> :cl: add: Added the Sunskipper-class Culinary Vessel. add: Every drink and booze dispenser drink is in the booze-o-mat now. add: Added bottles for triple sec, lemon juice, creme de cacao and creme de menthe. del: Removed the Boyardee-class Entertainment Vessel. del: Independent chefs no longer get kitchen cqc. tweak: The "syndicate waiter's outfit" renamed to just "waiter's outfit". tweak: Indpenedent bartenders now start with a waistcoat. tweak: Edited waiter's outfit. Also, the role now gets kitchen access. tweak: Independent cooks now start with laceup shoes. /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. -->
About The Pull Request
TODO
ethics of passengersmore sofasrework the main room 💀Seeds in the locker (if something important is missing here, slap me with a large stick pls):
Meds in the locker:
Stuff in the safe:
New bottles! Absolutely not peak sprite design, but I needed something, alright? Lemon Juice, Triple Sec, Creme de Cacao, Creme de Menthe:
Also, as the bar doesn't have any dispensers, I have added all of dispenser reagents to the booze-o-mat instead.
Also also, wiki page (WIP): https://shiptest.net/wiki/User:Sadhorizon#Sunskipper-class_Culinary_Vessel
Why It's Good For The
Boyardee is old and really really bad at it's job as a restaurant ship. Sunskipper is meant to replace it.
Indie chefs loose cqc, because it's an ability fit for fending off tiders in a station environment - not on a six player ship.
Booze-o-mat full of drink-making ingredients is a large step to getting rid of bar dispensers completely.
Changelog
🆑
add: Added the Sunskipper-class Culinary Vessel.
add: Every drink and booze dispenser drink is in the booze-o-mat now.
add: Added bottles for triple sec, lemon juice, creme de cacao and creme de menthe.
del: Removed the Boyardee-class Entertainment Vessel.
del: Independent chefs no longer get kitchen cqc.
tweak: The "syndicate waiter's outfit" renamed to just "waiter's outfit".
tweak: Indpenedent bartenders now start with a waistcoat.
tweak: Edited waiter's outfit. Also, the role now gets kitchen access.
tweak: Independent cooks now start with laceup shoes.
/:cl: