-
Notifications
You must be signed in to change notification settings - Fork 205
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
Not localizable #185
Comments
Hello, what do you mean by "as it is into other languages" ? you have a Config file where you can redefine all literals to that form _("text") if you want. that not an issue. |
Look here for instance. ImGuiFileDialog/ImGuiFileDialog.cpp Line 3919 in a7847af
Given that this code uses preprocessor-based string concatenation, replacing it with a dynamic expression fails. I'm not sure, but maybe these ID concatenations could be replaced by |
maybe you need one config file per localisation, because you will have the same issue for many other ImGui based lib. theses funcs uses const char* event if i remove this "##tag" you will have the issue.. but what is your localisation system ? what language / framework ? |
My file
and the macro In fact, I can affirm that rewriting that "##tag" concatenation in a compatible way will fix the problem. |
you localization system seem to be a performance killer. you system seem ok with many classic gui like QT, who is not call these _() each time. but for ImGui its crazy... So you need to be smarter and pre calc the most of things for a fast display.. |
There isn't slowness. Texts will always be cached after the first access. |
LOL |
Hello. This package is impossible to localize as it is into other languages.
One may attempt to redefine the strings in a custom header and wrap these into gettext() calls.
There is an obstacle: the code relies on these strings being defined as literals, and build fails.
Besides, it would be more helpful if texts were already wrapped in
_("text")
fashion; then the translation tools can readily scan these strings, making the task a lot simpler.The text was updated successfully, but these errors were encountered: