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

Medical GUI - Reformat injury list to clarify global vs. limb-specific information #9468

Merged
merged 5 commits into from Oct 7, 2023

Conversation

amsteadrayle
Copy link
Contributor

When merged this pull request will:

  • Change the layout of the injury list to separate global and limb-specific information
  • Add new non-issue text to positively indicate lack of conditions
  • Deemphasize non-issues with darkened text color

IMPORTANT

  • If the contribution affects the documentation, please include your changes in this pull request so the documentation will appear on the website.
  • Development Guidelines are read, understood and applied.
  • Title of this PR uses our standard template Component - Add|Fix|Improve|Change|Make|Remove {changes}.

@amsteadrayle
Copy link
Contributor Author

This will improve the readability of the overview/injury list in multiple ways:

  • Visually separating information that applies to the whole body and to the selected body part
  • Positively confirming the lack of certain conditions
  • Keeping certain information in the same location regardless of the presence or absence of other conditions

image

arma3_x64_6e2G3a4PPm.mp4

@mharis001
Copy link
Member

This is nice UX improvement. 👍

@LinkIsGrim
Copy link
Contributor

LinkIsGrim commented Oct 7, 2023

Should be some indication of which limb the IV is attached to, due to tourniquets. I misremembered how we display IVs in that menu.

Good to go.

@LinkIsGrim LinkIsGrim added the kind/enhancement Release Notes: **IMPROVED:** label Oct 7, 2023
@LinkIsGrim LinkIsGrim added this to the 3.16.1 milestone Oct 7, 2023
@amsteadrayle
Copy link
Contributor Author

Should be some indication of which limb the IV is attached to, due to tourniquets. I misremembered how we display IVs in that menu.

That would be nice and not too much work. But yeah, it's a separate thing. I might give it a try now thinking about it.

And I agree with putting bleeding above blood loss. 👍

@LinkIsGrim LinkIsGrim modified the milestones: 3.16.1, 3.16.0 Oct 7, 2023
@LinkIsGrim LinkIsGrim merged commit 78bb051 into acemod:master Oct 7, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Release Notes: **IMPROVED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants