Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Makes it more obvious in the lobby when there's a job overflow #4589

Merged
merged 2 commits into from
Dec 19, 2024

Conversation

Absolucy
Copy link
Member

@Absolucy Absolucy commented Dec 16, 2024

About The Pull Request

This adds some somewhat obvious notifications to the lobby - a chat notice, and an obvious doohickey on the menu itself showing the job icon and a flashing infinity symbol.

imo this is better than players not realizing there's an overflow, then just immediately cryoing when they end up in a job they dislike.

Also, I added a preference (disabled by default) to require the user to confirm that they want to ready up when there's an overflow.

just to clarify, you can figure out the overflow role in the lobby anyways if you manually check your job preferences, this just makes it practically impossible to miss

Screenshots

Chat notice

2024-12-16 (1734376468) ~ dreamseeker

Visual indicator

2024-12-16 (1734381431) ~ dreamseeker

Ready confirmation preference

REMINDER, YOU ONLY GET THIS PROMPT WHEN YOU EXPLICITLY ENABLE THE PREFERENCE

2024-12-16 (1734365458) ~ dreamseeker

2024-12-16 (1734365070) ~ dreamseeker

Why It's Good For The Game

Changelog

🆑
qol: There is now a chat notice and a visual indicator in the lobby for when there's a non-standard overflow.
qol: Added a preference (disabled by default) to confirm readying up when there's a non-standard overflow.
fix: Fixed a bug where the overflow station trait could roll assistants, which just did... nothing.
/:cl:

@DexeeXI
Copy link
Collaborator

DexeeXI commented Dec 17, 2024

This is currently on TM, especially given that issues are referencing this. This may be granted an exemption to the feature ban in place.

@DexeeXI DexeeXI added Major peak QoL improves quality of life labels Dec 17, 2024
@ThePooba ThePooba merged commit cee1824 into Monkestation:master Dec 19, 2024
24 checks passed
github-actions bot added a commit that referenced this pull request Dec 19, 2024
@Absolucy Absolucy deleted the job-overflow-notif branch December 19, 2024 22:55
Gw0sty pushed a commit to Gw0sty/Monkestation2.0 that referenced this pull request Jan 9, 2025
…station#4589)

* Makes it more obvious in the lobby when there's a job overflow

* fix a whoopsie there
Gw0sty pushed a commit to Gw0sty/Monkestation2.0 that referenced this pull request Jan 9, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Major peak QoL improves quality of life
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants