-
Notifications
You must be signed in to change notification settings - Fork 987
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
Use scrollable multiline text-input in contact request form #19383
Conversation
Jenkins BuildsClick to see older builds (36)
|
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 fixing it @seanstrom !
src/status_im/contexts/profile/contact/contact_request/style.cljs
Outdated
Show resolved
Hide resolved
src/status_im/contexts/profile/contact/contact_request/view.cljs
Outdated
Show resolved
Hide resolved
94% of end-end tests have passed
Failed tests (2)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (45)Click to expandClass TestActivityMultipleDevicePRTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestActivityMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
Class TestDeepLinksOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMergedTwo:
|
src/status_im/contexts/profile/contact/contact_request/view.cljs
Outdated
Show resolved
Hide resolved
d9169ac
to
912de96
Compare
@status-im/mobile-qa can you please review these E2E results? 🙏 |
@seanstrom thanks for the PR. Failed e2e are not PR related. |
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.
LGTM
fee0d7e
to
a91a676
Compare
a91a676
to
b670872
Compare
56% of end-end tests have passed
Failed tests (20)Click to expandClass TestActivityMultipleDevicePRTwo:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestCommunityMultipleDeviceMerged:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (27)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityOneDeviceMerged:
Class TestDeepLinksOneDevice:
|
Hey @seanstrom , thanks for the fix! ISSUE 1: Long message preview overflows the app screenSteps to reproduce:
Expected result (nightly): video_2024-04-03_16-09-40.mp4Actual result (PR): video_2024-04-03_16-10-01.mp4ISSUE 2: Image/link preview is not visible at all or partially hidden behind the bottom sheetComparison of image preview behavior in release (it's equal to nightly in this case) and PR on iOS (iPhone SE): IMG_2013.MP4Another example of the image preview: on this Android device it's partially hidden (Huawei P40 lite) Link preview behavior in PR (Android and iOS): IMG_2014.MP4 |
@qoqobolo Great finds! 🙌 |
0% of end-end tests have passed
Failed tests (20)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePR:
Class TestActivityMultipleDevicePRTwo:
Class TestCommunityMultipleDeviceMerged:
|
b670872
to
251f63a
Compare
@qoqobolo I think I may have found a fix for those issues, can you test this again please? 🙏 |
88% of end-end tests have passed
Failed tests (5)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMergedTwo:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (42)Click to expandClass TestActivityMultipleDevicePRTwo:
Class TestDeepLinksOneDevice:
Class TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestActivityMultipleDevicePR:
|
Sorry for the delay @seanstrom! |
251f63a
to
76f224f
Compare
… the input container, and container
… reached for the bottom-sheet
…missed on android
76f224f
to
68409f5
Compare
fixes #19324
Summary
input-container-style
can now be passed to override the styles of the input component.Testing notes
Platforms
Areas that maybe impacted
Functional
Steps to test
Before and after screenshots comparison
Before
Here's a recording of the behaviour before this PR: #19324
After iOS
Screen.Recording.2024-03-25.at.10.30.15.mov
After Android
Screen.Recording.2024-03-25.at.10.49.23.mov
status: ready