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

Jak 2: Finnish translations #3533

Merged
merged 22 commits into from
Aug 11, 2024
Merged

Jak 2: Finnish translations #3533

merged 22 commits into from
Aug 11, 2024

Conversation

Aloqas
Copy link
Contributor

@Aloqas Aloqas commented May 24, 2024

Finnish translations for Jak 2. These include cutscenes and all game text.

All subtitle timings for cutscenes as well as non-cutscenes have been edited for a better flow and to fit the 4x3 ratio.
I've been working on these solo for the most part so any input from other finns would be appreciated.

A few issues in the progress menu I mentioned in #3504 still persist

I couldn't figure out how to add Finnish to the options menu, so I'm gonna need someone else to do that part. 💀
But I was able to add them to the debug menu.

I also increased subtitle heap so hopefully that doesn't break anything.

Fixes #3620

@OpenGOALBot
Copy link
Collaborator

Can one of the admins verify this patch?

@@ -8774,6 +8774,7 @@ Unlike [[engine]], users can use [[engine-pers]] as a parent class."
(progress-graphics-ps2-lod-dist #x1337)
(progress-graphics-msaa-off #x1338)
(progress-graphics-display #x1339)
(language-name-finnish #x133a)
Copy link
Member

Choose a reason for hiding this comment

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

you need to also add this to decompiler/config/jak2/all-types.gc and the failing builds will pass.

@Aloqas Aloqas marked this pull request as draft May 25, 2024 17:58
@545u
Copy link

545u commented Jun 3, 2024

I've been working on these solo for the most part so any input from other finns would be appreciated.

LGTM!

@xTVaser
Copy link
Member

xTVaser commented Jun 29, 2024

Conflicts need to be resolved before this can be merged

@xTVaser
Copy link
Member

xTVaser commented Aug 1, 2024

Added to menus and tested, id encourage double checking my work though:
image
Screenshot 2024-07-31 223417

@Aloqas
Copy link
Contributor Author

Aloqas commented Aug 4, 2024

Added the country flag
2024-08-04T16-34-23

@Aloqas Aloqas marked this pull request as ready for review August 4, 2024 15:12
@xTVaser
Copy link
Member

xTVaser commented Aug 9, 2024

I think you can ignore the failing build, there's a mistake in it that I need to fix.

Is this ready for review now or are you still working on it?

@Aloqas
Copy link
Contributor Author

Aloqas commented Aug 9, 2024

I think it's ready now

@xTVaser
Copy link
Member

xTVaser commented Aug 11, 2024

ok to test

@xTVaser xTVaser self-assigned this Aug 11, 2024
@xTVaser xTVaser added this to the August 2024 milestone Aug 11, 2024
@xTVaser xTVaser merged commit 9d2a23e into open-goal:master Aug 11, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[jak2] [progress] do not use option index as language selection
4 participants