-
Notifications
You must be signed in to change notification settings - Fork 57
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
Upgrade React Native 0.71.11
- Android changes
#5872
Upgrade React Native 0.71.11
- Android changes
#5872
Conversation
09f2e5a
to
f726421
Compare
Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job! |
The result of running: ``` rm -rf ./node_modules && npm install ```
…0.71.8-android # Conflicts: # package-lock.json
…0.71.8-android # Conflicts: # gutenberg
@@ -12,6 +12,9 @@ const { | |||
} = e2eUtils; | |||
import { NESTED_COLUMNS_3_LEVELS } from './test-editor-data'; | |||
|
|||
const ANDROID_COLUMN_APPENDER_BUTTON_XPATH = |
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 haven't checked it further, but seems the native composition for Button
has changed in the new version of RN or dependencies. I experienced something similar in Gutenberg with other elements (example).
const cellId = isAndroid() | ||
? 'Column 1. Width is 50 Percent (%)., double-tap to change unit' | ||
: 'Column 1. Width is 50 Percent (%).'; | ||
const cellId = 'Column 1. Width is 50 Percent (%).'; |
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.
Similar to this change, the accessibility hint is no longer appended to the accessibility content description on Android.
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.
This changes seem sound to me, thanks for all the work debugging and fixing the tests @fluiddot! 👏 👏
Related PRs:
0.71.11
- Android changes WordPress/gutenberg#512890.71.11
WordPress-Android#18613To test:
0.71.11
- iOS changes #5881.0.71.11
WordPress/gutenberg#51303.PR submission checklist: