No translation #214

Activity

1. phw commented on Sep 24, 2017

   

   phw

   on Sep 24, 2017

   Contributor

   More actions

   Same here, installed gradio via flatpak, but it isn't localized, even though German translation should be available. Works for me with other Flatpak apps, including my own.
2. julianrichen commented on Sep 24, 2017

   

   julianrichen

   on Sep 24, 2017

   More actions

   Doesn't the Locale extension provide translations?

   flatpak --user install flathub de.haeckerfelix.gradio.Locale
   
3. haecker-felix commented on Sep 24, 2017

   

   haecker-felix

   on Sep 24, 2017

   Last edited by haecker-felix

   Owner

   More actions

   I can confirm that the translation is not working. I don't know what's the problem. If you know it, feel free to open a pull request :-)

   This is my first project to be translated, so I don't have much experience.
4. phw commented on Sep 25, 2017

   

   phw

   on Sep 25, 2017

   Contributor

   More actions

   I have experience with app localization and also did some Flatpak packaging, so I try to debug this a bit.

   de.haeckerfelix.gradio.Locale is installed, it got installed alongside de.haeckerfelix.gradio as it should. When I launch the gradio flatpak the console reports

   ** Message: gradio-app.vala:167: Locale dir: /app/share/locale
   

   This looks right. Also when I open a shell into the app with

   flatpak run --command=sh de.haeckerfelix.gradio 
   

   I can clearly see the localization files in /app/share/locale, this looks Ok:

   sh-4.3$ ls /app/share/locale/de/LC_MESSAGES/gradio.mo -lah
   -rw-r--r-- 2 0 0 6,6K Jan  1  1970 /app/share/locale/de/LC_MESSAGES/gradio.mo
   

   Also the initialization of the locales in https://github.com/haecker-felix/gradio/blob/master/src/gradio-app.vala#L156 . I compared everything above to my own https://github.com/phw/peek/blob/master/src/main.vala but don't see a difference. But in my app it works with Flatpak, gradio simple shows default locale texts. Will try to debug further.

   Just to make sure I also installed gradio locally and there everything shows up localized.
5. phw commented on Sep 25, 2017

   

   phw

   on Sep 25, 2017

   Contributor

   More actions

   Just to make sure I also installed gradio locally and there everything shows up localized.

   Wrong, current 6.0.1 release has this issue, I tested with 6.0. Pull request incoming...

   👍React with 👍1Reacted by gogogogi
6. phw

   added a commit that references this issue on Sep 25, 2017

   Fixed wrong gettext domain

   ...

   This issue will close once commit ef3b946 is merged into the 'master' branch.

   ef3b946
7. haecker-felix

   closed this as completedin #224on Sep 25, 2017