pret / pokeemerald

Decompilation of Pokémon Emerald
2.2k stars 2.36k forks source link

BUGFIX for the aspect ratio of the Pokémon logo on the title screen #2010

Closed pkmnsnfrn closed 1 month ago

pkmnsnfrn commented 2 months ago

Before                                                After

I am pkmnsnfrn on Discord.

GriffinRichards commented 1 month ago

This seems to me too subjective a change to include by default, our BUGFIXs should be unambiguous solutions (I'm not even sure I agree this is a bug, but that's beside the point). Anyone who cares about this might care equally about the other minor changes it makes to the appearance, like the highlight inside the "P" disappearing or the changes to the black outline.

pkmnsnfrn commented 1 month ago

When I think of BUGFIX I think

the developers did X, but they almost certainly meant to do Y

and I cannot imagine the world's most valuable brand intentionally used the wrong aspect ratio for the logo.

I don't feel that strongly about this feature, and I'm fine if you close it

but

if you agree "yeah, this was probably a mistake" then I can work on fixing the highlight and the black outline and updating this PR.

GriffinRichards commented 1 month ago

That's a sensible definition of BUGFIX. I don't agree that they almost certainly meant to use the logo in this PR as opposed to the one in-game. I grabbed an official logo online and compared it to the in-game asset; they seem reasonably the same to me:

pokemon_logo

But either way, I think we should avoid including fixes that involve making subjective decisions like for graphic design.

pkmnsnfrn commented 1 month ago

That gif is fascinating and very helpful! And I think "no graphic decisions" is more than fair.

cc @meejle