-
Notifications
You must be signed in to change notification settings - Fork 27
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: make learn more links clickable for automation v2 [WPB-5888] #3064
Conversation
Build 5117 failed. |
Build 5118 succeeded. The build produced the following APK's: |
Build 5256 failed. |
Built wire-android-staging-release-pr-3064.apk is available for download |
Built wire-android-dev-debug-pr-3064.apk is available for download |
Quality Gate passedIssues Measures |
Built wire-android-staging-release-pr-3064.apk is available for download |
Built wire-android-dev-debug-pr-3064.apk is available for download |
Build 5269 succeeded. The build produced the following APK's: |
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764
The PR Description
What's new in this PR?
This is a next approach to fix this issue: https://wearezeta.atlassian.net/browse/WPB-5888
Issues
Previous fix, although looking fine on devices and layout inspector, was not working for automation, because it can't use
testTags
and it can't find inline content text composable.Solutions
In this PR, link text composable is extracted out of the inline content of the main text - inline content only contains a placeholder box to position the link correctly inside the text and to get valid size and position of that link to create the proper link text composable. Inline content is still used to make it work for all locales, no matter if the link is inside the text or at the end or if it's LTR or RTL.
PR Post Submission Checklist for internal contributors (Optional)
PR Post Merge Checklist for internal contributors
References
feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764
.