-
Notifications
You must be signed in to change notification settings - Fork 490
Rapid Test Profile shows mixed languages after change in locale #3972
Comments
@MikeMcC399 Thanks. Internal Tracking ID: EXPOSUREAPP-9151 @Ein-Tim Actually I can observe something similar on iOS. The country list when checking a DCC is in English, although the phone locale is set to DE. No way to get it to German back. iPhone 6s. |
That is an entirely different problem. Perhaps you should submit an issue in the iOS repository? It doesn't belong here at all. I checked by the way on Android, and Android correctly provides the list of countries in German, but this is not related to the Rapid Test Profile in any way. |
I will open an issue regarding this! |
This seems to be an issue of TextInputLayout in the Material Components, see: material-components/material-components-android#2075 |
Thanks for uncovering that issue (material-components/material-components-android#2075)! The standard Android elements Edit: Android issue material-components/material-components-android#2075 is open with label "Needs investigation". CWA is using cwa-app-android/Corona-Warn-App/build.gradle Line 332 in 9d6525b
|
Closing as low impact and unsolved, due to the external component Material Components for Android (MDC-Android). |
@MikeMcC399 thanks for your comment, forwarded to internal ticket. |
material-components/material-components-android#2075 is now closed with a fix, however there is not yet a new release of Material Components for Android (MDC-Android) containing this fix. The upcoming CWA release 2.24.x is configured with Judging by the progress in the https://github.com/material-components/material-components-android/ repository, the fix looks like it might be released with a |
Material Components for Android 1.7.0-alpha03 addresses this issue. 1.7.0-alpha03 requires a minimum Updating Edit: Superseded by 1.7.0-rc01 |
@MikeMcC399 Thanks, forwarded to internal ticket. |
Re-opening this issue, since it still occurs and an official fix is now available. Material Components for Android (MDC-Android) has now made Release 1.7.0 available which enables this CWA issue to be resolved without using an Perhaps there is scope in a future CWA version to incorporate this fix? The official prerequisites however still include: This is not currently met by the 2.29 branch: cwa-app-android/Corona-Warn-App/build.gradle Lines 30 to 35 in f13b188
|
Good morning @MikeMcC399! I will re-open the internal ticket and forward your findings! Thanks! |
@MikeMcC399 FYI PR #5668 |
|
Fix is included in pre-release version v2.29.0-rc.0. |
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
The label
Fix 3.0
Please check and correct in the https://github.com/corona-warn-app/cwa-app-android/labels list. |
The label description has been changed now, thanks for the hint |
Avoid duplicates
Technical details
Describe the bug
If the device language is changed, the Rapid Test Profile input view does not refresh the field prompts.
Steps to reproduce the issue
(Updated for 2.29.x on Pixel 3a API 33 (Android 13))
App at step 6:
Expected behavior
Android apps are expected to immediately respond to language changes. At step 6 in the steps to reproduce, the app should show only German text as in the following screen shot.
Additional information
Thanks to feedback from @PhilippNowak96 who identified that this is an issue of TextInputLayout in the Material Components, see: issue material-components/material-components-android#2075.
The Material Components issue is now closed through material-components/material-components-android@051410b. Material Components for Android 1.7.0 is supposed to fix this issue.
Internal Tracking ID: EXPOSUREAPP-9151
The text was updated successfully, but these errors were encountered: