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

Improve some i18n things #13899

Merged
merged 1 commit into from
Jan 9, 2021
Merged

Improve some i18n things #13899

merged 1 commit into from
Jan 9, 2021

Conversation

vnctdj
Copy link
Contributor

@vnctdj vnctdj commented Jan 9, 2021

  • reuse some translations
  • move some strings to a more suitable category

I will adapt the lang .ini files accordingly.

@@ -1720,7 +1720,7 @@ void __KernelLoadReset() {
bool __KernelLoadExec(const char *filename, u32 paramPtr, std::string *error_string) {
SceKernelLoadExecParam param;

PSP_SetLoading("Loading exec...");
PSP_SetLoading("Loading executable...");
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we don't actually show these messages anymore. Personally I think "exec" and "executable" wouldn't make much sense to users, but it doesn't matter since it's no longer shown.

-[Unknown]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, this reminded me something... And I found it: I prepared an issue a looong time ago but never posted it, I don't know why...
Here it is:


With this pull request, new strings have been created for game launch information.
Now we have the following strings (if I haven't missed any):

Compiling shaders...
Generating code...
Loading executable...
Loading game...
Loading modules...
Loading shader cache...
Starting graphics...
Starting modules...

But this pull request replaced the display of these strings by a spinner.
We can also read a comment in this code by @unknownbrackets which says the following:

// Don't really need this, and it creates a lot of strings to translate...
// Maybe just show "Loading game..." only?

And he also commented this:

The text was intentionally removed because it causes a lot of extra strings to translate for 1.6.0.

But the strings mentioned above are still present in the code as of now.
So...
Is the spinner permanent and shown in all cases ? Therefore these strings would be useless ?
Or should we translate all these strings anyway because they will be reused ?

Copy link
Owner

Choose a reason for hiding this comment

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

The strings are still logged I think? But don't bother translating them, it's not useful. Won't show these to the user again for the beforementioned reasons.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, thanks, I'll just delete them then.

- reuse some translations
- move some strings to a more suitable category

I will adapt the lang .ini files accordingly.
@hrydgard hrydgard merged commit 55b5c6f into hrydgard:master Jan 9, 2021
@unknownbrackets unknownbrackets added this to the v1.11.0 milestone Jan 9, 2021
@vnctdj vnctdj deleted the i18nImprovements branch January 9, 2021 21:30
vnctdj added a commit to vnctdj/ppsspp-lang that referenced this pull request Jan 10, 2021
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