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

overhaul: Explorer Mode structure changes :: infrastructure setup #29

Merged
merged 6 commits into from
Nov 2, 2024

Conversation

mileswest07
Copy link
Owner

@mileswest07 mileswest07 commented Nov 2, 2024

  • Removed unused placeholder icons
  • Expanded space of placeholder spritesheet to make room for more variants, likely needed and will be expanded later
  • Refactored rawData to no longer reference old placeholders
  • Refactored some data in rawData for area names using unofficial sources
  • Refactored CSS definitions to be simplified, making Explorer Mode image extraction easier
  • Fixed offset issue with MZM Unified sprites on spritesheet
  • Added new icons to MZM Unified spritesheet: Item Orb, Randomizer
  • Fixed bug: if UseSprites is unset but UseMZMSprites is set, then MZM Unified sprites will still show. Code now checks both flags before deciding to show MZM Unified sprites or not.
  • Fixed bug: selecting Z1M1 option was selecting the wrong games. This has been fixed.
  • Fixed bug: when selecting AM2R for Randomizer Mode, the Extreme Labs hint icon was not loading properly. This has been fixed.
  • Fixed bug: when selecting AoL for Randomizer Mode, the Dash Boots hint icon was not loading properly. This has been rectified.
  • Refactored map rawData into mapTree structure: instead of listing all of the nodes as a flat array of hubs, all nodes are set up in a tree structure of parents, children, and vines tied via hubId-vineToHubId keys. Explorer code will be updated in a future PR; will not work if Explorer.html is reenabled.
  • Refactored rawData input to read from multiple JSON files instead of a .JS file import. This reduces load on system as it will only load required files and data struct. From this point on, code changes will need to require data and struct testing on each affected JSON file.
  • Added multiple JSON files to data/*.json

@mileswest07 mileswest07 merged commit 72c8946 into main Nov 2, 2024
3 checks passed
@mileswest07 mileswest07 deleted the map-overhaul branch November 2, 2024 07:17
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.

1 participant