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

Add support for overworld sprite gender differences + add all the sprites #5394

Merged
merged 20 commits into from
Oct 4, 2024

Conversation

kittenchilly
Copy link

Description

Allows overworld sprites and followers to have gender differences defined.
Adds sprites for all current pokemon gender differences.

Issue(s) that this PR fixes

Fixes #5124

People who collaborated with me in this PR

All the sprites for gender differences that weren't from GF came from here: https://www.deviantart.com/larryturbo/art/OWs-of-Missing-Gender-Variants-Normal-Shiny-878691887

Discord contact info

kittenchilly

@hedara90 hedara90 added category: pokemon Pertains to Pokémon data and functionality category: overworld Pertains to out-of-battle mechanics labels Sep 19, 2024
graphics/pokemon/pikachu/overworldf.png Outdated Show resolved Hide resolved
graphics/pokemon/gloom/overworldf.png Outdated Show resolved Hide resolved
@hedara90 hedara90 self-assigned this Sep 27, 2024
@hedara90 hedara90 added the status: on hold delayed until some decision later label Sep 27, 2024
@hedara90
Copy link
Collaborator

Poke me when this is ready

@kittenchilly
Copy link
Author

On hold as I am currently waiting for a master merge into upcoming

@kittenchilly
Copy link
Author

Master has been merged and I have updated the branch, so this PR is no longer on hold and can be reviewed

@hedara90 hedara90 removed the status: on hold delayed until some decision later label Sep 30, 2024
Copy link
Collaborator

@hedara90 hedara90 left a comment

Choose a reason for hiding this comment

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

I'll look at the code changes later, only sprite comments right now.

graphics/pokemon/alakazam/overworldf.png Outdated Show resolved Hide resolved
graphics/pokemon/bibarel/overworld.png Outdated Show resolved Hide resolved
graphics/pokemon/blaziken/overworldf.png Outdated Show resolved Hide resolved
graphics/pokemon/dodrio/overworldf.png Outdated Show resolved Hide resolved
graphics/pokemon/doduo/overworldf.png Outdated Show resolved Hide resolved
graphics/pokemon/roserade/overworldf.png Outdated Show resolved Hide resolved
graphics/pokemon/scyther/overworldf.png Outdated Show resolved Hide resolved
graphics/pokemon/sneasel/overworldf.png Outdated Show resolved Hide resolved
graphics/pokemon/staraptor/overworldf.png Outdated Show resolved Hide resolved
graphics/pokemon/wooper/overworldf.png Outdated Show resolved Hide resolved
@hedara90
Copy link
Collaborator

When setting OW_POKEMON_OBJECT_EVENTS to FALSE compilation fails. This is not the case on regular upcoming.
20240930_11h03m42s_grim

@kittenchilly
Copy link
Author

Addressed the sprite reviews, and fixed the OW_POKEMON_OBJECT_EVENTS compilation fail

Copy link
Collaborator

@hedara90 hedara90 left a comment

Choose a reason for hiding this comment

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

Some more blinking.
Now with pictures instead of bad descriptions of locations!

graphics/pokemon/alakazam/overworldf.png Outdated Show resolved Hide resolved
graphics/pokemon/donphan/overworldf.png Outdated Show resolved Hide resolved
graphics/pokemon/mamoswine/overworldf.png Outdated Show resolved Hide resolved
graphics/pokemon/meganium/overworldf.png Outdated Show resolved Hide resolved
graphics/pokemon/wooper/overworldf.png Outdated Show resolved Hide resolved
@kittenchilly
Copy link
Author

Addressed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: overworld Pertains to out-of-battle mechanics category: pokemon Pertains to Pokémon data and functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants