Skip to content

Fix copyright tex for GC JP #5324

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

Merged
merged 2 commits into from
Apr 8, 2025

Conversation

Archez
Copy link
Contributor

@Archez Archez commented Apr 4, 2025

The GC JP versions (not CE), only have a texture for the copyright matching the year 2002.
This adjusts the name of the texture extracted and updates the title screen logo to display 2002 as appropriately and matching decomp (when only the GC JP version is loaded, or language is set to JPN with any GC version).

Will require regen for these specific versions, which we already are expecting for 9.0.1 from the boss title card fix.

Build Artifacts

@Archez
Copy link
Contributor Author

Archez commented Apr 5, 2025

Updated to also include changes for uniquely naming the copyright textures in the JP CE version as 2004.
There are two in here, both 2004 but ever so slightly different. One displays when the language is JPN and the other when ENG so I named them as such.

@briaguya-ai
Copy link
Contributor

is there any more nuance to add to the width calculation or is

#define COPYRIGHT_TEX_WIDTH (isGC ? 160 : 128)
#define COPYRIGHT_TEX_LEFT (isGC ? 78 : 94)

still accurate?

@Archez
Copy link
Contributor Author

Archez commented Apr 6, 2025

Yes, all GC copyright tex have the same width/position. The only one that has something different per decomp is iQue which we don't support.

@Malkierian Malkierian merged commit 8866974 into HarbourMasters:develop-blair Apr 8, 2025
6 checks passed
Varuuna pushed a commit to Varuuna/Shipwright that referenced this pull request Apr 12, 2025
* Fix copyright tex for GC JP

* add 2004 texture for jp ce
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