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] formula assistant: localize argument separator #3790
[FIX] formula assistant: localize argument separator #3790
Conversation
The argument separator in the formula assistant was not localized, it was always a comma no matter the locale. This commit fixes it. Task: 3789860
@@ -73,6 +73,10 @@ export class FunctionDescriptionProvider extends Component<Props> { | |||
this.assistantState.allowCellSelectionBehind = false; | |||
}, 2000) as unknown as number; | |||
} | |||
|
|||
get formulaArgSeparator() { | |||
return this.env.model.getters.getLocale().formulaArgSeparator + " "; |
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.
return this.env.model.getters.getLocale().formulaArgSeparator + " "; | |
return this.env.model.getters.getLocale().formulaArgSeparator + "\u00A0"; |
Not sure if the unbreakable space was necessary though
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.
I didn't put an unbreaking space because it looks like simple spaces are preserved with t-esc
. And it makes it harder to read the code if you don't know by heart unicode characters ;)
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.
robodoo r+
The argument separator in the formula assistant was not localized, it was always a comma no matter the locale. This commit fixes it. closes #3790 Task: 3789860 Signed-off-by: Lucas Lefèvre (lul) <lul@odoo.com>
Forward-porting to 'saas-17.2' (from #3882). |
Description
The argument separator in the formula assistant was not localized, it was always a comma no matter the locale. This commit fixes it.
Task: : 3789860
review checklist