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

Add the ability to apply hacks #214

Merged
merged 17 commits into from
Jun 18, 2023

Conversation

jonko0493
Copy link
Member

This adds a little "Apply Hacks" dialog that provides a list of common hacks that can be applied. After checking one or more off and hitting save, the hacks are assembled and inserted into the ARM9/overlays. This PR only implements a hack to skip the OP as it was simple to implement.

Closes #108.

@jonko0493 jonko0493 requested a review from WiIIiam278 June 18, 2023 19:20
@jonko0493 jonko0493 self-assigned this Jun 18, 2023
Copy link
Member

@WiIIiam278 WiIIiam278 left a comment

Choose a reason for hiding this comment

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

Big fan of this -- especially the modular nature that lets users make their own hacks (might want to document that format). A few changes to make, but happy to unblock this tbh now that you've fixed the duplication issue.

I'll prepare some icons for this new stuff perhaps just before release. Only major bit of feedback is to do with the adjustments made to the preferences dialogue, but I think a quick fix for the box sizes would be fine for 0.2 :)

src/SerialLoops/Dialogs/RomHacksDialog.cs Show resolved Hide resolved
src/SerialLoops/Dialogs/PreferencesDialog.cs Outdated Show resolved Hide resolved
src/SerialLoops/MainForm.eto.cs Outdated Show resolved Hide resolved
src/SerialLoops/Dialogs/RomHacksDialog.cs Outdated Show resolved Hide resolved
src/SerialLoops/Dialogs/RomHacksDialog.cs Outdated Show resolved Hide resolved
src/SerialLoops/Dialogs/RomHacksDialog.cs Show resolved Hide resolved
@jonko0493 jonko0493 merged commit 93c41de into haroohie-club:main Jun 18, 2023
@jonko0493 jonko0493 deleted the AddApplyingHacks branch June 18, 2023 22:50
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.

Add "apply hacks" feature
2 participants