-
Notifications
You must be signed in to change notification settings - Fork 360
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
Italian translation of the hub #1290
base: 0.8
Are you sure you want to change the base?
Conversation
Co-authored-by: Sebastiano Ballotta <sebeballotta@outlook.com>
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@rinodrummer Apologies for the late review. This looks good, but I can see 6 TODOs outstanding in the code that will need addressing before this can be merged. Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolve TODOs
Hi @glennjacobs, I'll give a look at them in the weekend! |
Hi @rinodrummer, I'm going to change this PR to draft until we have it ready for review again. |
I will look to merge translations once we've hit a beta release, as there may be some changes required. |
I've just realised I assumed this was for v1.0, but it's actually targeting 0.8. Would you have any interest in doing v1.0? |
To improve the usability, me and @sebhd1 decided to dive in the Italian translation of the hub.
We translated all the entries in the localization files, but noticed that most of the UI components have un-wrapped texts that make them hard to translate.
I think that a guide line must be discussed to define which namespaces must be attributed to some components and must enforce to use
__()
whenever possible, even in component labels, placeholders and so on when new hub section are introduced.We are willing to help and always open to discussions (which will handle here)!
References #1152.