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

Update warning screen natives #1041

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

freedy69
Copy link
Contributor

@freedy69 freedy69 commented Mar 4, 2024

Corrected names and documented all the parameters

Copy link
Contributor

@PsychoShock PsychoShock left a comment

Choose a reason for hiding this comment

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

Hello,

I have left some comments. This is great work, nice job.

HUD/GetWarningScreenMessageTitleHash.md Outdated Show resolved Hide resolved
HUD/SetWarningMessage.md Outdated Show resolved Hide resolved
HUD/SetWarningMessage.md Outdated Show resolved Hide resolved
@PsychoShock
Copy link
Contributor

Can you squash everything btw please? :)

Update warning screen natives

Update SetWarningMessageWithHeader

Documented params and description

Add button list to SetWarningMessage

Update IsWarningMessageActive_2

Update SetWarningMessageWithHeaderAndSubstringFlags

Fix oversight in SetWarningMessageWithHeaderAndSubstringFlags

update native name in example

SET_WARNING_MESSAGE_WITH_ALERT

documented everything regarding this native

Update GetWarningScreenMessageTitleHash.md

added that this native returns 0 if warning message isnt active

Remove dead imgur link in SetWarningMessage

Update SetWarningMessage.md

Removed gist link and improved information about buttons to help make it easier for people to understand
Did some testing and this doesn't seem to be case

**Result of the example code:**
[https://i.imgur.com/imwoimm.png](https://i.imgur.com/imwoimm.png)
| Button Index | Button Text | Button Input | Notes |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please use an enum over a table.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Bump on this

@AvarianKnight AvarianKnight added the needs validation This looks good, but needs additional confirmation of suggested change. label Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs validation This looks good, but needs additional confirmation of suggested change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants