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] TRAIT_DEAF now works on non-carbon mobs #2163

Merged
merged 1 commit into from
Feb 29, 2024
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1165
Original PR: tgstation/tgstation#81624

About The Pull Request

This PR makes it so that the can_hear proc will return FALSE if the proc is called on a non-carbon that has the deaf trait.

Why It's Good For The Game

This makes it so that non-carbon mobs can have the TRAIT_DEAF added to them and it will now work. There isn't anything that would currently utilize this on here. But, I do have a project downstream where TRAIT_DEAF is added to a non-carbon mob.

Changelog

🆑 softcerv
code: TRAIT_DEAF now works on non-carbon mobs
/:cl:

* TRAIT_DEAF now works on non-carbon mobs (#81624)

## About The Pull Request
This PR makes it so that the `can_hear` proc will return `FALSE` if the
proc is called on a non-carbon that has the deaf trait.
## Why It's Good For The Game
This makes it so that non-carbon mobs can have the TRAIT_DEAF added to
them and it will now work. There isn't anything that would currently
utilize this on here. But, I do have a project downstream where
TRAIT_DEAF is added to a non-carbon mob.
## Changelog
:cl:
code: TRAIT_DEAF now works on non-carbon mobs
/:cl:

---------

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

* TRAIT_DEAF now works on non-carbon mobs

---------

Co-authored-by: Pinta <[email protected]>
Co-authored-by: MrMelbert <[email protected]>
@Iajret Iajret merged commit 6e7ab31 into master Feb 29, 2024
24 checks passed
@Iajret Iajret deleted the upstream-mirror-1165 branch February 29, 2024 18:27
AnywayFarus added a commit that referenced this pull request Feb 29, 2024
Iajret pushed a commit that referenced this pull request Apr 24, 2024
…2163)

* Adds sorting options and a search bar to the crew monitor (#82732)

## About The Pull Request
This PR adds buttons in the crew monitor UI to sort by name, position,
or vitals, as well as a search bar for searching for specific people.
For vitals, people are sorted first by life status (living, crit, dead,
etc), then by damage. Also TS-ifies the UI.



https://github.com/tgstation/tgstation/assets/42454181/a7c0c262-8ea7-4e9f-8de6-6b9cf7481b23


## Why It's Good For The Game
Currently, finding a specific person, or a person in a specific place,
is a pain in the butt with the crew monitor. You have to go through an
unsorted list of people grouped only by department. Don't know the
person's job? Tough luck, you're going down the list one by one. It is
not at all a friendly user experience. With this PR, it becomes a
breeze, and makes the crew monitor much easier to use.
## Changelog
:cl:
add: The crew monitor now has sorting options and a search bar.
/:cl:

* Adds sorting options and a search bar to the crew monitor

---------

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