feat(i18n): add Brazilian Portuguese localization - #142
Conversation
LeVraiArdox
left a comment
There was a problem hiding this comment.
Hello, thanks for the PR !
Although the translation is a very good improvement for Sleex, I don't think this is the right way to do it.
Firstly, we can't just use AI for this type of task. An 8B LLM is unpredictable and we can't let it rewrite the source code. Also, it's not a clean way to generate translations.
Then, instead of using QTranslator, it would probably be better to write a custom service that search for a string to translate (wrapped in a function, for example Translations.tr("stuff")), and translate it with a translation dictionary. The dictionary can be hand written or AI written if reviewed afterward, but it's on the translator's end. By translator I mean the person that translates.
If you want to take the challenge, go ahead. Else, you can close this PR. Thanks anyway :)
|
Hi there! I totally agree with you. Using AI to translate everything isn't the best way neither the safest. It was more of an example of possibility than a real implementation 😅 I liked your idea and I'll try to do it. I'm not so good and coding, that's why I vibe code most of it (you clearly noticed it 🤣). If that's okay, I'll be glad to help. |
|
I have no problem with vibe coding, as long as it respects the code style and have a human review behind. Also, AI sometimes makes useless comments, that can be avoided. Tho, I recommend you to try to understand the code that is generated, AI can be a good learning tool ;) |
Summary
Adds the initial internationalization infrastructure and Brazilian Portuguese
localization for Sleex.
Changes
pt_BR) translationsTesting
pt_BRcatalog is installed at runtimeqsTranslate()privacy options, interface options, sound settings and behavior settings
Refs #105