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

Rename the Move Relearner specials #684

Merged
merged 1 commit into from
Jan 19, 2025
Merged

Conversation

Zeturic
Copy link
Contributor

@Zeturic Zeturic commented Jan 13, 2025

This renames SelectMoveTutorMon and DisplayMoveTutorMenu to ChooseMonForMoveRelearner and TeachMoveRelearnerMove, respectively.

This is how they're named in pokeemerald, and it avoids SelectMoveTutorMon from being confused with the similar ChooseMonForMoveTutor.

@SatoMew
Copy link
Contributor

SatoMew commented Jan 19, 2025

The character is called Move Maniac.

@Zeturic
Copy link
Contributor Author

Zeturic commented Jan 19, 2025

Are you suggesting that these, plus all of the existing symbols that use the Move Relearner terminology (link) should be renamed to use the "Move Maniac" name instead?

If yes, I would argue that's a downgrade in clarity. ChooseMonForMoveRelearner is more clear than ChooseMonForMoveManiac because people don't actually refer to this guy (or these characters generally) as Move Maniac(s), regardless of what he happens to call himself in-game.

I actually considered going through and renaming everything to Move Reminder (the general term they officially stuck with, and how I personally refer to such characters), but it'd be a much larger change of questionable utility, and I figured consistency with pokeemerald was a more pressing concern, so I didn't.

@SatoMew
Copy link
Contributor

SatoMew commented Jan 19, 2025

Are you suggesting that these, plus all of the existing symbols that use the Move Relearner terminology (link) should be renamed to use the "Move Maniac" name instead?

Yes, or Move Reminder, but Move Maniac is more accurate in the context of these games.

@GriffinRichards
Copy link
Member

If yes, I would argue that's a downgrade in clarity. ChooseMonForMoveRelearner is more clear than ChooseMonForMoveManiac because people don't actually refer to this guy (or these characters generally) as Move Maniac(s), regardless of what he happens to call himself in-game.

it'd be a much larger change of questionable utility, and I figured consistency with pokeemerald was a more pressing concern, so I didn't.

Agreed on both counts.

@GriffinRichards GriffinRichards merged commit d844877 into pret:master Jan 19, 2025
1 check passed
github-actions bot pushed a commit that referenced this pull request Jan 19, 2025
gitMyPad added a commit to gitMyPad/firered that referenced this pull request Jan 27, 2025
commit d844877
Author: Kevin <[email protected]>
Date:   Sun Jan 19 14:14:55 2025 -0500
    Rename the Move Relearner specials. (pret#684)
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