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

UI: Rename Wild Arms hack #11356

Merged
merged 1 commit into from
Jun 14, 2024
Merged

Conversation

JordanTheToaster
Copy link
Member

Description of Changes

Renames the Wild Arms hack to something closer to what it actually does internally.

Rationale behind Changes

More logical naming and less game related hacks good.

Suggested Testing Steps

Make sure CI is happy the hack still functions and it loads properly from the DB in games that have it as a gamefix.

@lextra2
Copy link

lextra2 commented Jun 8, 2024

The tooltip should be more descriptive of what it does.

And if changed, it should also mention something like
Formerly called Wild Arms hack
because people will wonder where it went.

@JordanTheToaster JordanTheToaster force-pushed the hacky-whacky branch 3 times, most recently from 2e92cee to 9d083ab Compare June 9, 2024 12:58
Copy link
Contributor

@lightningterror lightningterror left a comment

Choose a reason for hiding this comment

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

Perhaps it would've been better to split the commit in several: gs changes, qt, fsui, config, doc removal, db changes.

@kamfretoz
Copy link
Contributor

Missed this during the testing, the text for the checkbox doesn't appear and the GameDB entry for it aren't getting loaded.
image
image

@JordanTheToaster JordanTheToaster force-pushed the hacky-whacky branch 3 times, most recently from 4570279 to 087dd0b Compare June 10, 2024 15:23
@JordanTheToaster
Copy link
Member Author

Missed this during the testing

All should be fixed now.

@JordanTheToaster JordanTheToaster force-pushed the hacky-whacky branch 2 times, most recently from c33861d to 955e9fd Compare June 13, 2024 16:53
@refractionpcsx2 refractionpcsx2 merged commit f3776fa into PCSX2:master Jun 14, 2024
13 checks passed
@JordanTheToaster JordanTheToaster deleted the hacky-whacky branch June 14, 2024 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants