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

Updates Health Analyzer UI #1183

Merged
merged 5 commits into from
Nov 7, 2024
Merged

Conversation

gluesniffler
Copy link
Contributor

Description

Does what it says on the can, ports over the updates to the Health Analyzer made in Wizden's #30834, #31879, and #32636 because I need them for newmed

Credit to Floof #169 too since I yoinked it due to lazyness :trollface:


Changelog

🆑 Mocho

  • tweak: Ported over Wizden's updated Health Analyzer UI.

FoxxoTrystan and others added 5 commits November 7, 2024 00:10
…yzerUIWizden

Updated Health Analyzer UI From Latest Wizden Update
* hide spriteview from health analyzer while inactive

* add out of range indicator if analyzer becomes inactive

* hide out of range icon if there is no patient data
* Add health analyzer unrevivability warning

* Remove errornous comment
@github-actions github-actions bot added Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: UI Changes any XAML files Changes: YML Changes any yml files labels Nov 7, 2024
@github-actions github-actions bot added Status: Needs Review Someone please review this labels Nov 7, 2024
@FoxxoTrystan FoxxoTrystan added Size: 4-Small For small issues/PRs Priority: 3-Medium Needs to be resolved at some point Type: Port Brings something to here from another codebase Status: Awaiting Merge 2 or more maintainers has approved this PR and is now awaiting merge and removed Status: Needs Review Someone please review this labels Nov 7, 2024
@VMSolidus VMSolidus merged commit ffdaa84 into Simple-Station:master Nov 7, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: UI Changes any XAML files Changes: YML Changes any yml files Priority: 3-Medium Needs to be resolved at some point Size: 4-Small For small issues/PRs Status: Awaiting Merge 2 or more maintainers has approved this PR and is now awaiting merge Type: Port Brings something to here from another codebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants