Skip to content
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

Confirm box should be internationalized - Cancel and OK buttons #849

Closed
yunohost-bot opened this issue Mar 20, 2017 · 6 comments
Closed
Labels
🎏 Translations 👾 bug Something isn't working
Milestone

Comments

@yunohost-bot
Copy link

Original Redmine Issue: 849

Author Name: jibec


In web admin, in screen upgrade/packages, the confirm box has "OK" or "Cancel" in French. It should be "Confirmer" ou "Annuler".
Please see screenshot.

I assume it should bring back the "cancel" from this string: https://github.com/YunoHost/yunohost-admin/blob/unstable/src/locales/fr.json#L55
but I can't find an "OK" string.

The box looks like to be build from this class:
https://github.com/YunoHost/yunohost-admin/blob/83127a5692fa6a8c25cb4599b613d6c6f05b574f/src/js/yunohost/helpers.js#L251

thanks for your help,

@yunohost-bot
Copy link
Author

File Attachment

Author: jibec

File: YNH-bouton Cancel 2017-03-20 06-52-47.png (7.32 KB)

@yunohost-bot
Copy link
Author

Properties Change

Author: moul

@yunohost-bot
Copy link
Author

Original Redmine Comment

Author: Bram


It's actually here https://github.com/YunoHost/yunohost-admin/blob/4f28ba44268de77527c7ae280b876ade4ca529b9/src/index.html#L79-L80

I don't know how we are supposed to make things translatable in the webadmin html.

@yunohost-bot
Copy link
Author

Original Redmine Comment

Author: alexAubin


Isn't this supposed to be a view/mustache thing like the others ?

@yunohost-bot
Copy link
Author

Original Redmine Comment

Author: Bram


The mustache stuff is only used for the dynamic content of the admin, the "body", the modal is stored as a global hidden html, that's probably a limitation of this current design.

I guess it could be extracted as a mustache thinghy, opi should be able to tell us more about that.

@yunohost-bot
Copy link
Author

Original Redmine Comment

Author: alexAubin

Status Changed: Resolved


We just tested it and couldn't reproduce : the string is translated (when loading the admin) using the key in data-y18n field. The only minor bug is that 'ok' is not a key, so the key is used and you get 'ok' (lowercase).

This PR fixes it : YunoHost/yunohost-admin#171

@yunohost-bot yunohost-bot added 👾 bug Something isn't working 🎏 Translations labels May 24, 2018
@yunohost-bot yunohost-bot added this to the 2.7.x milestone May 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎏 Translations 👾 bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant