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

Fixed some gba sprites #5607

Merged
merged 1 commit into from
Oct 28, 2024

Conversation

SubzeroEclipse
Copy link

Description

Some of the newly re-added GBA sprites had their positions (the back of Chimecho and Deoxys-N, namely) or their elevation (a few mons that float in Emerald, but their sprites in the expansion were touching the ground) wrong. This PR fixes this.

Feature(s) this PR does NOT handle:

For Deoxys, I fixed the position of its back sprite, but i didn't address some other things going on with it: namely, the front sprite that flip flops between Normal and Speed form and the icons that were not updated for its other forms (Gen 6+ icons were not added for them)

Discord contact info

subzeroeclipse

Copy link
Collaborator

@AsparagusEduardo AsparagusEduardo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Besides 1 comment, everything else lgtm :)

@@ -11433,7 +11436,7 @@ const struct SpeciesInfo gSpeciesInfoGen3[] =
.frontAnimId = ANIM_GROW_VIBRATE,
.backPic = gMonBackPic_DeoxysNormal,
.backPicSize = MON_COORDS_SIZE(64, 56),
.backPicYOffset = 6,
.backPicYOffset = P_GBA_STYLE_SPECIES_GFX ? 2 : 6,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Author

@SubzeroEclipse SubzeroEclipse Oct 28, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That sprite for Deoxys-N wasn't supposed to show up at all in vanilla Emerald, the 8 was used by the back sprite of Speed Forme :)
pokeemerald-0 (2)
I got the 2 by testing manually, there wasn't much I could refer to for placing the Normal form and that's why I left it alone besides the back that was easily observable (tried to look up Pokeruby's code, but the elevations there looks different from Emerald, and I couldn't find the section regarding the other placements)

@AsparagusEduardo AsparagusEduardo merged commit 5b5ecaa into rh-hideout:upcoming Oct 28, 2024
1 check passed
@AsparagusEduardo AsparagusEduardo added bugfix Bugfixes category: sprite-issue Issues with a Pokémon's sprite/palette labels Oct 28, 2024
@AsparagusEduardo AsparagusEduardo added the category: pokemon Pertains to Pokémon data and functionality label Dec 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Bugfixes category: pokemon Pertains to Pokémon data and functionality category: sprite-issue Issues with a Pokémon's sprite/palette
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants