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 window to customize specific controls #1122

Merged
merged 2 commits into from
Aug 16, 2022

Conversation

lilDavid
Copy link
Contributor

Eliminates the merge issues from #851 but otherwise makes the same changes.

This adds a window accessed from the Enhancements > Controls menu that contains options to change the way certain actions are mapped to the N64 controller, sorted using collapsing headers like the cosmetics menu. Some or all of the controls enhancements could be moved into this window, but that's outside the scope of this PR.

As for the ocarina control panel, there are options to change which play which notes as well as which buttons modify the note (originally L, R, and Z). Additionally, the option to use the D-pad for the ocarina and text was split with the latter being moved to the ocarina controls, and there's a new option to play the ocarina using the camera stick.

Currently only for the ocarina
@jbodner09 jbodner09 requested review from briaguya-ai and dcvz August 14, 2022 01:45
@dcvz
Copy link
Contributor

dcvz commented Aug 14, 2022

Have not forgotten this PR. It’s been a busy weekend but want to look this over soon 🙂

@dcvz dcvz merged commit 46a421f into HarbourMasters:develop-zhora Aug 16, 2022
dcvz added a commit to dcvz/Shipwright that referenced this pull request Aug 16, 2022
@lilDavid lilDavid deleted the customize_game_controls branch August 16, 2022 11:48
louist103 pushed a commit to louist103/Shipwright-1 that referenced this pull request Aug 19, 2022
Kenix3 pushed a commit to Kenix3/Shipwright that referenced this pull request Oct 19, 2022
dcvz pushed a commit to briaguya-ai/Shipwright that referenced this pull request Nov 12, 2022
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