penguin-teal / gnome-openweather

A GNOME Shell extension to show the weather of any location on Earth.
GNU General Public License v3.0
52 stars 17 forks source link

Translation issues #61

Closed lev741 closed 2 months ago

lev741 commented 2 months ago

Description

Not translatable strings: Section Locations/Provider don`t have strings to translate exept string "Geolocation Provider"

All translated forecast expressions (like Storm) are comented now and not used for OpenWeatherMap. So can`t be translated now and are on english forecast expressions only. Not czech. Is it end of translation of forecast expressions?

My Version

138.0.g6367a11 with the latest openweather.pot

penguin-teal commented 2 months ago

I believe the issue is now because of the way the code is written it cannot auto-generate the translation keys. I think the solution will be to just write a shell script to extract the necessary keys and throw it in the Makefile. Will fix soon.

penguin-teal commented 2 months ago

Ended up essentially using a no-op function to denote that that string will be eventually passed to gettext and there is a special --keyword option in xgettext for that. All weather conditions should appear in the .pot now.