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

Updating the zelda3.ini in the switch area #231

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

Conversation

chris-lansman
Copy link

Description

[zelda3.ini] Updated the file with some cleanup, added back the missing comments, turned on the proper key mappings for controller based input

Will this Pull Request break anything?

No

Suggested Testing Steps

Run on the switch. I am doing so now.

chris-lansman and others added 2 commits February 21, 2023 23:42
…ng comments, turned on the proper key mappings for controller based input
@@ -93,10 +111,10 @@ CancelBirdTravel = 0

[KeyMap]
# Change what keyboard keys map to the joypad
# Order: Up, Down, Left, Right, Select, Start, A, B, X, Y, L, R
Order: Up, Down, Left, Right, Select, Start, A, B, X, Y, L, R
Copy link
Owner

Choose a reason for hiding this comment

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

Why did you remove the #

Copy link
Author

@chris-lansman chris-lansman Mar 6, 2023

Choose a reason for hiding this comment

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

I removed the # to uncomment the keys that actually map to the switch instead of the windows keys that were mapped before. Either way it appears to have no affect as mapping these and then trying any of the menu toggles produced no results.

The inverse change (i.e commenting out) happens on line 117 to remove the windows key mapping which do not exist in the context of the switch console.

Copy link
Owner

Choose a reason for hiding this comment

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

Order: is not the right syntax, Order: is just a comment.

Copy link
Owner

Choose a reason for hiding this comment

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

The one you're looking for is Controls =

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.

2 participants