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] Venus Human Traps have an orbit menu category, recieve antag datum on spawn #1954

Merged
merged 1 commit into from
Feb 13, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#935
Original PR: tgstation/tgstation#81411

About The Pull Request

Venus Human Traps now have their own category in the orbit menu. It is a pleasant shade of green, just like vines are.

image

Venus Human Traps are also now given their antagonist datum upon spawning from a flower. The datum isn't really anything special but it's consistent with the other antag spawners.

This also makes the spawner piece of the VHT flower bud invisible. It's an abstract object meant to handle the logic for spawning, and only the structure should be visible. This currently manifests as there being two, identical flower buds on the same tile when a flower bud spawns. I did not want to get into untangling the two objects in this PR.

Why It's Good For The Game

Venus human traps being lumped in with the humans in the "Alive" menu is WEIRD. Now ghosts can give a gigantic vine infestation the attention it truly deserves.

Fixes a minor visual incongruity found while doing all this.

Changelog

🆑 Rhials
qol: Venus Human Traps are now visible in the orbit menu. Cool!
fix: Spawning as a Venus Human Trap now properly gives you an antag datum/objective.
fix: Venus human trap flowers no longer have a second, identical flower under themselves.
/:cl:

… datum on spawn (#935)

* Venus Human Traps have an orbit menu category, recieve antag datum on spawn (#81411)

## About The Pull Request

Venus Human Traps now have their own category in the orbit menu. It is a
pleasant shade of green, just like vines are.


![image](https://github.com/tgstation/tgstation/assets/28870487/90615559-a45b-44a6-8fb4-39a9172a8e3a)

Venus Human Traps are also now given their antagonist datum upon
spawning from a flower. The datum isn't really anything special but it's
consistent with the other antag spawners.

This also makes the spawner piece of the VHT flower bud invisible. It's
an abstract object meant to handle the logic for spawning, and only the
structure should be visible. This currently manifests as there being
two, identical flower buds on the same tile when a flower bud spawns. I
did not want to get into untangling the two objects in this PR.
## Why It's Good For The Game

Venus human traps being lumped in with the humans in the "Alive" menu is
WEIRD. Now ghosts can give a gigantic vine infestation the attention it
truly deserves.

Fixes a minor visual incongruity found while doing all this.
## Changelog
:cl: Rhials
qol: Venus Human Traps are now visible in the orbit menu. Cool!
fix: Spawning as a Venus Human Trap now properly gives you an antag
datum/objective.
fix: Venus human trap flowers no longer have a second, identical flower
under themselves.
/:cl:

* Venus Human Traps have an orbit menu category, recieve antag datum on spawn

---------

Co-authored-by: Rhials <[email protected]>
@ReezeBL ReezeBL merged commit 01f54f6 into master Feb 13, 2024
24 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-935 branch February 13, 2024 10:38
AnywayFarus added a commit that referenced this pull request Feb 13, 2024
Iajret pushed a commit that referenced this pull request May 8, 2024
…ks, gives taur legs names, etc. (#1954)

* atomize

* dwa'

* why isnt it publishing

* womp

* i am stupid

* suffix, to avoid confusion

* Apply suggestions from code review

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

* manual review

---------

Co-authored-by: GoldenAlpharex <[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