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

[Skyrat Mirror] [SEMI-MODULAR] Adds a small icon/examine blurb for people with DNR to med/sec huds #12

Closed
wants to merge 1 commit into from

Conversation

ffmirrorbot[bot]
Copy link

@ffmirrorbot ffmirrorbot bot commented Oct 12, 2023

Original PR: ${this.url}

About The Pull Request

Title. Its just a tiny lil red/white exclamation mark and a blurb in warning span in the guy isnt already dead.

How This Contributes To The Skyrat Roleplay Experience

A common complaint about people in this game is how people always say "ahahah nobody can die ahhah just kill them and revive them ahaha"

Theyre not WRONG, but it does cause some issues in scenarios where its NOT true, namely in the DNR quirk. Like with my temp ftext icon, having this info available on-a-glance and clarifying when curious people examine will allow a lot more people to recognize this little thing.

Its locked to huds over concerns of cluttering the space around people with icons.

Finally, and I mean finally, we can actually genuinely yell at security who goes "im just gonna shoot the hostage taker" in the MOMENT and PREVENT IT FROM HAPPENING - because we can all see the lil !

Proof of Testing

Screenshots/Videos

![image]((original url)

Changelog

🆑 nikothedude
add: Sec/Medhuds can now see a small ! if a person has the DNR quick, as well as see a blurb in examine about it
/:cl:

… med/sec huds (#23524)

* sertdyuio

* uju

* actually i like this more

* i want to die

* Apply suggestions from code review

Co-authored-by: Bloop <13398309+vinylspiders@users.noreply.github.com>

* awddaw

* adwsawd

* Apply suggestions from code review

Co-authored-by: GoldenAlpharex <58045821+GoldenAlpharex@users.noreply.github.com>

* ok

---------

Co-authored-by: Bloop <13398309+vinylspiders@users.noreply.github.com>
Co-authored-by: GoldenAlpharex <58045821+GoldenAlpharex@users.noreply.github.com>
@Iajret Iajret closed this Oct 17, 2023
@Iajret Iajret deleted the upstream-mirror-23524 branch October 17, 2023 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants