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

Review and revise some maps' script and text labels #420

Open
3 tasks
Rangi42 opened this issue Jul 14, 2023 · 0 comments
Open
3 tasks

Review and revise some maps' script and text labels #420

Rangi42 opened this issue Jul 14, 2023 · 0 comments
Labels

Comments

@Rangi42
Copy link
Member

Rangi42 commented Jul 14, 2023

Follow-up to PR #367.

  • Revise the Seafoam Islands script labels, like SeafoamIslandsB3FObjectMoving1Script/SCRIPT_SEAFOAMISLANDSB3F_OBJECT_MOVING1. "It all has to do with moving the boulders around. ... They were also very, very redundant here. Many routines that do practically the same thing."
  • Revise the Silph Co text labels. "For all the Silph workers whose text just varies depending on whether you saved Silph, I don't think they need dialog-specific labels; just things like SilphCo7FWorkerM1BeforeText and SilphCo7FWorkerM1AfterText would do. Likewise for other NPCs who just switch between two texts based on an event."
  • Name trainers' labels and constants according to their class, not their sprite. Special characters like gym leaders already do this. There's just one non-special case that already does so: in PokemonMansionB1F.asm, an OPP_BURGLAR has TEXT_POKEMONMANSIONB1F_BURGLAR, despite using SPRITE_SUPER_NERD. The rest are named after their sprite.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant