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

[MIRROR] Adjusts BS Miner with a new updated overlay, slightly better code, and less annoying buzzing/beeping #648

Merged
merged 1 commit into from
Nov 16, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Original PR: Skyrat-SS13/Skyrat-tg#25008

About The Pull Request

Mechanically, nothing has changed. It works exactly the same as before, except now it also has lights to show you why it's not working. And the sound is no longer obnoxiously loud with a range of 17 and passing walls

The first light lights up red when something is wrong.
The second and third light signal if atmos is not working, being either crimson (Pressure high) or teal (Pressure low)
Fourth light lights up orange when temp is high
All lights will light up red when too close to another miner

How This Contributes To The Skyrat Roleplay Experience

You will no longer have to hear an earpiercing buzz when standing near engineering or atmos.

Proof of Testing

Have tested this already to prove that mechanically nothing has changed, images of extra stuff provided

Screenshots/Videos Here it is complaining about low pressure and high heat

image

Functioning state stayed untouched
image

High pressure
image

Too close
image

Changelog

🆑 Majkl-J
qol: Bluespace miner now has visual indication to its status and problem
sound: The sound on the bluespace miner is no longer earpiercingly loud
image: Added lights to the bluespace miner
refactor: Refactored bluespace miner code
/:cl:

…d less annoying buzzing/beeping (#25008)

* this

* oop

* I do actually miss the sounds ever so slightly

* Still too much, silence you fool

* Moved this here

* Update modular_skyrat/modules/bluespace_miner/code/bluespace_miner.dm

Co-authored-by: Bloop <[email protected]>

* Update modular_skyrat/modules/bluespace_miner/code/bluespace_miner.dm

Co-authored-by: Bloop <[email protected]>

* Update modular_skyrat/modules/bluespace_miner/code/bluespace_miner.dm

Co-authored-by: Bloop <[email protected]>

* Update modular_skyrat/modules/bluespace_miner/code/bluespace_miner.dm

Co-authored-by: Bloop <[email protected]>

* Update modular_skyrat/modules/bluespace_miner/code/bluespace_miner.dm

Co-authored-by: Bloop <[email protected]>

---------

Co-authored-by: Bloop <[email protected]>
@Iajret Iajret merged commit aae2b26 into master Nov 16, 2023
24 checks passed
@Iajret Iajret deleted the upstream-mirror-25008 branch November 16, 2023 09:06
AnywayFarus added a commit that referenced this pull request Nov 16, 2023
Iajret pushed a commit that referenced this pull request Jan 28, 2024
* Remove unneeded 'a' from swabbing message. (#81118)

## About The Pull Request

When using the biopsy tool on something that doesn't have a swabbing
result it would say:
> "You do not manage to find a anything on [target]!".

The "find a anything on [target]" is incorrect and reads off, so this pr
just changes the respective statement in
`code/datums/components/swabbing.dm` to remove that a, becoming:
> "You do not manage to find anything on [target]!".
## Why It's Good For The Game

Minor grammar fix.
## Changelog
:cl:
spellcheck: You no longer fail to find "a anything" when swabbing
something for cytology that doesn't have swabbing results.
/:cl:

* Remove unneeded 'a' from swabbing message.

---------

Co-authored-by: _0Steven <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants