-
Notifications
You must be signed in to change notification settings - Fork 984
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: address input multiline #19962
fix: address input multiline #19962
Conversation
Jenkins BuildsClick to see older builds (25)
|
88% of end-end tests have passed
Failed tests (5)Click to expandClass TestWalletMultipleDevice:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestDeepLinksOneDevice:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (46)Click to expandClass TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePR:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestWalletOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityMultipleDevicePRTwo:
|
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.
Thanks for the PR @OmarBasem !
247aba9
to
d1e3dbe
Compare
Hi @OmarBasem, thanks for PR! ISSUE 1: Address is not aligned with the check icon on AndroidDesign |
85% of end-end tests have passed
Failed tests (6)Click to expandClass TestDeepLinksOneDevice:
Class TestWalletMultipleDevice:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Expected to fail tests (2)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Passed tests (44)Click to expandClass TestWalletOneDevice:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestActivityMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityMultipleDevicePRTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
|
d1e3dbe
to
cf42a79
Compare
Thanks for your testing @qoqobolo, fixed the issue! |
Thanks @OmarBasem , but it's not aligned on iOS now 🥲 Figma https://www.figma.com/design/HKncH4wnDwZDAhc4AryK8Y/Wallet-for-Mobile?node-id=689-192841&m=dev |
66cb217
to
01e2d32
Compare
Thanks for your testing @qoqobolo! Please check now and let me know |
Thanks @OmarBasem, PR can be merged. |
01e2d32
to
c3d598c
Compare
c3d598c
to
33b8ac7
Compare
fixes: #19788
Summary
This PR fix addresses input to support multiline.
Demo
Screen_Recording_20240510_120614_Status.mp4