-
Notifications
You must be signed in to change notification settings - Fork 565
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
Phone Do not Disturb #4427
Phone Do not Disturb #4427
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code good
Currently, when the button is pressed it doesn't change its color or update the button text or have any visible state change in tgui, It feels like it should though... |
The hover-text and the icon on the button do update when the state is changed |
I didn't notice it in-game when I first commented and didn't look at the code, I see it in the code now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dunno why this has sat so long
About the pull request
Allows phones to be put on Do Not Disturb, preventing them from being called. This is not possible to toggle on certain phones such as Overwatch.
Explain why it's good for the game
Sometimes phones get spammed for trolls, other times the relevant people (like CL) are not in their office. This makes it easy to stop wasted calls on empty rooms or busy people.
Testing Photographs and Procedure
Screenshots & Videos
Put screenshots and videos here with an empty line between the screenshots and the
<details>
tags.Changelog
🆑
add: Added a Do Not Disturb feature for MOST phones. (Overwatch and a couple others excluded).
/:cl: