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
fix: Translation related issues #11932
Conversation
To make description and HTML options with line breaks translatable
make onboard step and shortcut format translatable
make html options translatable
fix translation issue
fix translation issue
fix translation issue
fix Chinese language key
fix Chinese language key
fix translation
fix translation issue
fix from different path goes to list view, the sidebar menu appear multiple times issue
add translate
This comment has been minimized.
This comment has been minimized.
revert back
@surajshetty3416 @gavindsouza |
Co-authored-by: Suraj Shetty <13928957+surajshetty3416@users.noreply.github.com>
Co-authored-by: Suraj Shetty <13928957+surajshetty3416@users.noreply.github.com>
Co-authored-by: Suraj Shetty <13928957+surajshetty3416@users.noreply.github.com>
@surajshetty3416 |
@szufisher all checks need to have a green tick before we can merge the PR. |
fix the pr semantic issue
@surajshetty3416 |
@szufisher we are squashing commits in this PR since none of the commit messages (except one) is following conventional commit message guidelines. |
@@ -128,6 +128,7 @@ frappe.ui.keys.AltShortcutGroup = class AltShortcutGroup { | |||
return !this.is_taken(letter) && is_valid_char; | |||
}); | |||
if (!shortcut_letter) { | |||
$text_el.attr('data-label', text_content); |
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.
@szufisher @surajshetty3416 any idea why this was added here?
Mainly added the missing translation call __(), and fixed some translation related bugs.
related discussion thread
https://discuss.erpnext.com/t/html-field-as-formatted-description-is-not-translatable/67806
https://discuss.erpnext.com/t/question-why-line-break-return-has-been-replaced-by-empty-string-in-js-translate-function-which-caused-some-description-not-translatable/67803
issues
#11856