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
remove trailing whitespace #17903
remove trailing whitespace #17903
Conversation
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.
Looks good to me. I think the failing UI tests are unrelated?
@Findus23 Thanks for doing that. We might maybe need to wait with merging that until we migrated to weblate. If we merge that now Transifex would remove all already done translations for that. Not sure if weblate has a better handling for whitespace changes. |
Indeed I would wait for a bit with merging this. I also kept it a draft so I can add more things I might notice during the weblate migration. I don't know how weblate handles whitespace changes, but I have already seen that if a source string changes, the translators will see a diff of the string and can apply the changes to the translation. |
The trailing whitespace were fixed with the Weblate migration |
A few fixes noticed on weblate.
I hope those whitespace are not required for properly spacing texts on the site. But if they are, things are already broken as they are missing in all translations.
Review