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

Adds ability to fly from the PokeNav Region Map #5672

Closed
wants to merge 5 commits into from

Conversation

khbsd
Copy link

@khbsd khbsd commented Nov 16, 2024

Lets the player press 'R' over locations they have visited to fly to them from the PokeNav Region Map.

Description

  • changed RegionMap struct to typedef RegionMap struct with the name RegionMap to allow more flexible use in other files
  • refactored some logic from CB_ExitFlyMap() to allow setting the fly destination correctly from other files
  • added OW_FLAG_USE_FLY_FROM_POKENAV to allow this feature to be disabled
  • added SYS_USE_POKENAV_FLY as a system flag to allow disabling the Show Mon animation when flying from the PokeNav
  • added new Looped Task in pokenav_region_map.c to handle flying
  • added new function UpdateRegionMapHelpBarText() so the HelpBarText can be easily synced with cursor movement and loading the map from the menu
  • added new inputs to both ProcessRegionMapInput_* functions
  • added new zoomed in and out text strings with the R button and the text 'FLY' for when the cursor is over a flyable location in the region map
  • adjusted some #includes and changed some static functions to non-static so they could be declared in the header files
  • uhhhhh probs more but i forgor

Feature(s) this PR does NOT handle:

Adding other interactions in the overworld to handle other HM moves, this is specifically for Fly

Discord contact info

@khbsd

hedara90 and others added 5 commits November 14, 2024 17:23
Co-authored-by: Hedara <hedara90@gmail.com>
i told it not to but it added this anyways
@khbsd khbsd closed this Nov 16, 2024
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