Skip to content

Medical GUI - Add customizable message when opening player's medical menu#10883

Open
DartRuffian wants to merge 4 commits intoacemod:masterfrom
DartRuffian:medical-gui/menu-message
Open

Medical GUI - Add customizable message when opening player's medical menu#10883
DartRuffian wants to merge 4 commits intoacemod:masterfrom
DartRuffian:medical-gui/menu-message

Conversation

@DartRuffian
Copy link
Contributor

When merged this pull request will:

  • Add a customizable message to display on patient's screen when opening their medical menu
    • Optionally shows a configurable icon if the medic is a medic / doctor

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}.

"EDITBOX",
[LSTRING(medicalHintMedicIcon_DisplayName), LSTRING(medicalHintMedicIcon_Description)],
[ELSTRING(medical,Interface_Category), LSTRING(SubCategory)],
"",
Copy link
Contributor Author

@DartRuffian DartRuffian Apr 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Default icon would be good, not sure if there's one from vanilla / ACE

Co-authored-by: Will/KJW <100206101+SpicyBagpipes@users.noreply.github.com>
Co-authored-by: PabstMirror <pabstmirror@gmail.com>
@DartRuffian
Copy link
Contributor Author

DartRuffian commented Jun 16, 2025

Note to self, need to pass unit to the display Text so opening a dead player's menu doesn’t display a message

@SpicyBagpipes
Copy link
Contributor

Is this PR getting any further movement? Would like to see it implemented

@DartRuffian
Copy link
Contributor Author

Nothing else needed as far as I'm concerned. Default image would be nice but I couldn't find anything that seemed fitting in vanilla / ACE

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.

3 participants