-
Notifications
You must be signed in to change notification settings - Fork 99
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
fdo menu section translation. #751
Comments
Thank you for bringing this to our attention. As far as I can tell the translations should be retrieved from files like |
The top level menu titles are mostly translated using our icewm MO data, not via desktop files, since those additional *.directory files were not always available when the code was implemented. Even now, those *.directory files are part of the "gnome-menus" package on Debian, and their content is incomplete. But I found more issues and fishy code, worth overhauling some parts of data handling in fdomenu.c . Updates will follow soon. |
I think fdo do not use MO at all, when ret->title == nullptr, fdo will use MO data,
but, from below code, I think 'ret->title != nullptr' always, when I change
when I change 'if (ret->title == nullptr)' to 'if (true)', MO data is used. |
I think ret->title alway is a string.
|
@tumashu Yes, that's the location where the translation is missing. However, you also don't want to apply gettext() multiple times, but the code is too messy to track this as of now (there were actually preparations to track the translation but it's IMHO buggy). In the meantime I have decided to reimplement the whole thing from inside, at least the major parts of data collection. It's just too messy, back in 2015 when I decided to use GLib containers (just "because", to avoid further STL or libice framework), that made the code way too awkward. Also this focus on the building of the tree from top to bottom maybe was not the best idea. You can have a look, if you want: https://github.com/Code7R/icewm/tree/ftr_rebuilding_fdomenu (still a snapshot, the code between lines ~350 and ~900 is supposed to be gone, removed or replaced with compacter/better version, changing paradigm to loosely coupled dictionaries, some tree based, some unordered, separating the menu output creation completely from menu filling). ATM I am not sure whether the deeper menu structuring should be performed, and I tend to say: NO, use only main and one sub category level. Otherwise it would become creepy, lots of submenus with just one or two entries in each on a typical user system. |
@Code7R wow, good job, when this feature will be merged? |
@tumashu Merged on icewm-1-4-BRANCH now. Considering to release it as soon as enough translations have arrived. |
good news. |
I use 'includeprog icewm-menu-fdo --no-sep-others --seps --no-sub-cats -t x-terminal-emulator', but some menu section can not be translated to Chinese properly, for example: Audio, AudioVideo,Newwork,Utility, Video, I have checked zh_CN.po file, translate can be found. any suggestion?
The text was updated successfully, but these errors were encountered: