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

RCD, Airlock QoL #8195

Merged
merged 12 commits into from
Dec 14, 2023
Merged

RCD, Airlock QoL #8195

merged 12 commits into from
Dec 14, 2023

Conversation

Tsar-Salat
Copy link
Contributor

@Tsar-Salat Tsar-Salat commented Dec 10, 2022

Ports:

About The Pull Request

Airlock's are now possible to build in-game. (or atleast cyclelinks are).

RCD's can now print windoors, 1-dir windows, named airlocks, cyclelinked airlocks, and furnishings.
How RCD's handled matter was also significantly refactored.

As the plumberator is a subtype of RCDs, those received small cleanup, but nothing noteworthy or playerfacing.

TODO:

  • No conflicts
  • Radial sprites all appear
  • Can insert mats into RCD
  • Can insert compressed matter into RCD
  • Can name & link airlocks
  • Can place windows
  • Can place furnishings

Why It's Good For The Game

Theres no real reason not to have furnishings and directional windows on the RCD.
Plus the addition of making airlocks in game will assist greatly for those making custom shuttles.

Mat insertion is literally just a bunch of if statements, we should just be assigning a value to the mats able to be inserted, and checking for that value when inserting any mat. Then it's simple as adding one line of code under whatever /obj/item/stack/sheet we want to add in the future

Testing Photographs and Procedure

rcd1
rcd2
rcd3
RCD4
rcd5
rcd6
rcd7

Changelog

🆑 rkz, AlinaStarkova
tweak: You can now set an Airlocks name and cycle ID in the RCD airlock settings menu, or by clicking on an airlock control board and making an airlock manually, this allows you to make airlocks that can cycle with each other. examine an airlock to see what, if any, ID they are set to cycle on (linear 1x3 airlocks to space currently do not use cycleIDs)
add: The RCD's functionality for directional windows has been enabled.
add: Cyborgs can now install RCD upgrades, with the notable exception of silo link upgrade
add: The RCD's functionality for windowed doors has been enabled.
add: The RCD furnishing upgrade may now be installed
/:cl:

@github-actions github-actions bot added Sprites TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 labels Dec 10, 2022
@CydiaLamiales
Copy link
Contributor

Engineborg buffs, pogu

@PestoVerde322
Copy link
Contributor

do the "expansion" tag exist for the CL?

@itsmeow
Copy link
Member

itsmeow commented Dec 10, 2022

No they do not, remove them

@Tsar-Salat
Copy link
Contributor Author

changed to tweak

code/game/machinery/doors/airlock.dm Outdated Show resolved Hide resolved
code/game/machinery/doors/airlock.dm Outdated Show resolved Hide resolved
code/game/machinery/doors/airlock_electronics.dm Outdated Show resolved Hide resolved
code/game/turfs/open/floor.dm Show resolved Hide resolved
code/game/machinery/doors/windowdoor.dm Outdated Show resolved Hide resolved
code/game/objects/items/RCD.dm Outdated Show resolved Hide resolved
code/game/objects/items/RCD.dm Show resolved Hide resolved
code/game/objects/items/RCD.dm Outdated Show resolved Hide resolved
@Tsar-Salat Tsar-Salat marked this pull request as draft January 2, 2023 23:41
@github-actions
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Tsar-Salat Tsar-Salat closed this Jan 27, 2023
@Tsar-Salat Tsar-Salat deleted the airlocks-QOL branch March 5, 2023 22:56
@Tsar-Salat Tsar-Salat restored the airlocks-QOL branch October 18, 2023 16:38
@Tsar-Salat
Copy link
Contributor Author

I am planning on finishing this, im just atomizing what I can so this is more legible and less of a mess

@github-actions
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Tsar-Salat Tsar-Salat marked this pull request as ready for review October 25, 2023 08:24
@Tsar-Salat
Copy link
Contributor Author

I broke radial sprites for the new upgrade at some point, gotta fix em

@Tsar-Salat
Copy link
Contributor Author

Alright, Furnishing labels now use Bee chair/table/stool sprites instead of TG and actually appear. Yay.

This is now ready.

Copy link
Member

@PowerfulBacon PowerfulBacon left a comment

Choose a reason for hiding this comment

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

Kinda cool, I am still strongly against the RCD but at the same time its kind of needed for construction because of how long and convuluded contruction has been made.
If these are getting constantly used to build annoying nukie holdouts with glass doors on nukie rounds, then they will definitely be getting a nerf.

@PowerfulBacon PowerfulBacon added this pull request to the merge queue Dec 14, 2023
@Tsar-Salat
Copy link
Contributor Author

Kinda cool, I am still strongly against the RCD but at the same time its kind of needed for construction because of how long and convuluded contruction has been made.

Mhm, TG made it more expensive to use overall I believe, but cheaper if you are rebuilding or deconstructing roundstart structures, which may be a possible design avenue to explore in the future.

I like the idea of an RCD as a utility, not a replace-all construction type tool.

At the very least, it being on shuttles makes more sense now that you can make cyclelinks ingame.

Merged via the queue into BeeStation:master with commit 59712fa Dec 14, 2023
8 checks passed
@Tsar-Salat Tsar-Salat deleted the airlocks-QOL branch December 14, 2023 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Sprites TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 Tweak
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants