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-ons localization issues #26

Open
hessenfarmer opened this issue Mar 5, 2021 · 10 comments
Open

Add-ons localization issues #26

hessenfarmer opened this issue Mar 5, 2021 · 10 comments
Assignees
Labels
engine Needs C++ coding changes enhancement New feature or request i18n Internationalization & Localization

Comments

@hessenfarmer
Copy link
Contributor

hessenfarmer commented Mar 5, 2021

although all addons are translated to german the translation is not shown in the Addons screen in current widelands.
This is either a bug in the Github action or in the i18n code

@Noordfrees
Copy link
Member

Do you mean the Installed or the Browse tab?
For remotes, it is not possible to show them localized with the current implementation (which is fixed in my addons-webserver branch though).

@Noordfrees Noordfrees added the i18n Internationalization & Localization label Mar 5, 2021
@frankystone
Copy link
Contributor

The localized strings might only be applied if one restarts widelands.

Currently i have updated some addons and i saw localization files are downloaded, but the localization was shown only after restarting widelands. Not sure if reentering the Addons (by quitting the addons window and reopen it again) does also do the trick.

@Noordfrees
Copy link
Member

Noordfrees commented Mar 5, 2021

I cannot reproduce that, for me the name and description are show in the Installed tab correctly in Low German immediately after the download completed…

@Noordfrees Noordfrees added the needs information This issue seems incomplete label Mar 5, 2021
@frankystone
Copy link
Contributor

Just to be precise: The observation i made was for already downloaded addons. Thats why i talked about upgrading an addon which was already installed. In the tab 'installed' i have clicked 'Update all' but the localization was not applied immediately.

Let's wait for @hessenfarmer's response.

@Noordfrees
Copy link
Member

Sorry seems I had overread that. So the add-on's localized description is currently FOO, then you do an upgrade that changes the localized description to BAR (while not changing the unlocalized description), and the info in the installed tab keeps showing FOO?
I know gettext does some caching of translated strings, to improve performance, so this is not really a bug in Widelands. But I'll check whether there exists some magic to reset this cache after an upgrade to fix this glitch.

@hessenfarmer
Copy link
Contributor Author

I meant the Browse tab and I was confused as some Part are localized there e.g."Diese Erweiterung ist bekannt Abstürze zu verursachen. If this is normal then this is not a bug.

@hessenfarmer
Copy link
Contributor Author

I just downloaded the 2 mapsets (challenge and former official) the localization is shon in the addons manager however the maps are not translated when strting a game

@Noordfrees
Copy link
Member

I meant the Browse tab and I was confused as some Part are localized there e.g."Diese Erweiterung ist bekannt Abstürze zu verursachen. If this is normal then this is not a bug.

Name, description and author are currently never localized in the Browse tab. Everything else is. So all fine here.

I just downloaded the 2 mapsets (challenge and former official) the localization is shon in the addons manager however the maps are not translated when strting a game

Confirmed, that seems to be a general bug, will fix

@Noordfrees Noordfrees self-assigned this Mar 5, 2021
@Noordfrees Noordfrees added bug Something isn't working Map Set Category: Maps and removed needs information This issue seems incomplete labels Mar 5, 2021
@hessenfarmer
Copy link
Contributor Author

maybe a picture shows it better
this can be found in the browse tab of remote addons
grafik

@Noordfrees Noordfrees changed the title translation bot seems to be not working correctly Add-ons localization issues Mar 5, 2021
@Noordfrees
Copy link
Member

Noordfrees commented Mar 5, 2021

So I summarize the several different glitches which this bug report is about:

@Noordfrees Noordfrees added enhancement New feature or request engine Needs C++ coding changes and removed Map Set Category: Maps bug Something isn't working labels Mar 6, 2021
@Noordfrees Noordfrees modified the milestone: Test Milestone Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engine Needs C++ coding changes enhancement New feature or request i18n Internationalization & Localization
Projects
None yet
Development

No branches or pull requests

3 participants