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

Added back GBA sprites via config #5206

Merged

Conversation

AsparagusEduardo
Copy link
Collaborator

@AsparagusEduardo AsparagusEduardo commented Aug 19, 2024

Description

Restores Emerald's Pokémon sprites via new P_GBA_SPECIES_GFX config (off by default).

Images

image image

People who collaborated with me in this PR

@AlexOn1ine for their help with script to migrate data from vanilla to our current gSpeciesInfo.

Feature(s) this PR does NOT handle:

We don't add new GBA-style sprites for any Pokémon that doesn't exist in Gen 3, so families will potentially look inconsistent.
Contributors are welcome to PR Gen3-style sprites, but they are low priority to me at least.

Discord contact info

AsparagusEduardo

@AsparagusEduardo AsparagusEduardo added category: pokemon Pertains to Pokémon data and functionality new-feature Adds a feature labels Aug 19, 2024
@AsparagusEduardo AsparagusEduardo force-pushed the _RHH/pr/upcoming/oldSprites branch from 0a14e7e to 8b351fa Compare August 19, 2024 00:31
@AsparagusEduardo AsparagusEduardo marked this pull request as ready for review August 29, 2024 16:57
@AsparagusEduardo
Copy link
Collaborator Author

AsparagusEduardo commented Aug 29, 2024

GBA icons and footprints restored. Ready for review!

@Bassoonian
Copy link
Collaborator

Can you update the images in the PR description to reflect the now accounted for footprints and icons?

@AsparagusEduardo
Copy link
Collaborator Author

Can you update the images in the PR description to reflect the now accounted for footprints and icons?

Done.

@kittenchilly
Copy link

I think it should be referred to as something like P_OLD_SPECIES_GFX since P_GBA_SPECIES_GFX I feel is too vague. Also technically all gfx on the gba is gba gfx lol

@AsparagusEduardo
Copy link
Collaborator Author

I think it should be referred to as something like P_OLD_SPECIES_GFX since P_GBA_SPECIES_GFX I feel is too vague. Also technically all gfx on the gba is gba gfx lol

I called them "GBA" instead of "OLD" because if people were to contribute to add Gen3-style sprites for new Pokémon, then it would be an actual misnomer imo, as it's about the style instead of time difference (GBA vs DS)

@AsparagusEduardo
Copy link
Collaborator Author

As requested by @Bassoonian, separated icons and footprints into their own configs.

@kittenchilly
Copy link

I think it should be referred to as something like P_OLD_SPECIES_GFX since P_GBA_SPECIES_GFX I feel is too vague. Also technically all gfx on the gba is gba gfx lol

I called them "GBA" instead of "OLD" because if people were to contribute to add Gen3-style sprites for new Pokémon, then it would be an actual misnomer imo, as it's about the style instead of time difference (GBA vs DS)

Hm maybe then it can be P_GBA_STYLE_SPECIES_GFX

@Bassoonian
Copy link
Collaborator

Their actual path in the repo still refers to them as "old", so even if the config has a different name, it's still referred to as old somewhere. Do they truly need to be in a separate folder anyway? What's preventing us from doing eg front_gba.png?

@AsparagusEduardo
Copy link
Collaborator Author

Their actual path in the repo still refers to them as "old", so even if the config has a different name, it's still referred to as old somewhere. Do they truly need to be in a separate folder anyway? What's preventing us from doing eg front_gba.png?

You're right, I'll rename them too.

@AsparagusEduardo
Copy link
Collaborator Author

Moved files to the graphics/pokemon folder with the _gba suffix.
Also, renamed the configs.

Ready for re-review.

@Bassoonian
Copy link
Collaborator

There are still a handful of files in the pokemon_old folder

@AsparagusEduardo
Copy link
Collaborator Author

There are still a handful of files in the pokemon_old folder

Done

@Bassoonian Bassoonian merged commit 358c0d0 into rh-hideout:upcoming Oct 18, 2024
1 check passed
@AsparagusEduardo AsparagusEduardo deleted the _RHH/pr/upcoming/oldSprites branch October 19, 2024 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: pokemon Pertains to Pokémon data and functionality new-feature Adds a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants