-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
[RNmobile] Trim feature image copy #37956
Conversation
Size Change: +640 B (0%) Total Size: 1.13 MB
ℹ️ View Unchanged
|
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 noticed that on iOS, these strings are untranslated when testing the editor with this change. However, this makes sense as the strings have changed, so LGTM 🎊 !
Tested on Samsung Galaxy S20 FE 5G (Android 10).
NOTE: Since this is a user-facing fix, I'm wondering if we should update the react-native-editor
changelog, wdyt?
Agreed, thanks for the suggestion! updated in ac10f8392f |
## 1.69.0 | ||
|
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 noticed that an extra line has been introduced to all sections of the changelog, @jhnstn is this expected?
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.
yeah I just noticed that, it was auto formatted by VScode. But I just noticed half of the file has the extra whitespace already so I'm thinking keep the extra space. Happy to revert that if you think it's better to have no space.
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 saw that other changelog files have this format (example reference), so looks great to me leaving it this way 👍 .
Description
This is a partial fix for wordpress-mobile/gutenberg-mobile#4432
Leaving whitespace at the end of a string causes the translation look up to fail on Android.
While this is not the most robust solution it does fix the two known problem strings.
How has this been tested?
Since this issue only occurs on Android, testing should be done with an Android device or emulator
Types of changes
Checklist:
*.native.js
files for terms that need renaming or removal).