-
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: increase numbered keyboard touchable area #18330
fix: increase numbered keyboard touchable area #18330
Conversation
Jenkins BuildsClick to see older builds (8)
|
75% of end-end tests have passed
Failed tests (8)Click to expandClass TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Expected to fail tests (4)Click to expandClass TestCommunityOneDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Passed tests (36)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestCommunityOneDeviceMerged:
Class TestDeepLinksOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityMultipleDevicePR:
|
66e3b1f
to
3eeaba3
Compare
Hi @briansztamfater ! Nice to meet you and thanks for your PR:) I am faced with a crush that blocks me in my testing. Please, take a look on it's details here: ISSUE 1: [Android/iOS] app crushes when user tries to select any asset Steps:
Actual result: app crushed |
40% of end-end tests have passed
Failed tests (26)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Expected to fail tests (3)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Passed tests (19)Click to expandClass TestActivityMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestDeepLinksOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
|
Hi @mariia-skrypnyk! Thanks for assigning this issue. About the error you mentioned, there is another PR that tries to fix the error: #18326 So probably we can give that PR priority once approved. |
Thanks Brian! |
94% of end-end tests have passed
Not executed tests (32)Failed tests (1)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Passed tests (15)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
|
3eeaba3
to
f99a4af
Compare
@mariia-skrypnyk The fix was merged and I just rebased this PR so hopefully we can continue testing this one 🙏 |
73% of end-end tests have passed
Failed tests (10)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePRTwo:
Expected to fail tests (3)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestCommunityOneDeviceMerged:
Passed tests (35)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestDeepLinksOneDevice:
|
Thanks @briansztamfater for merge and rebase! |
Signed-off-by: Brian Sztamfater <[email protected]>
f99a4af
to
65f0e40
Compare
Signed-off-by: Brian Sztamfater <[email protected]>
fixes #18205
Summary
This PR increases touchable area of keys in numebered keyboard to improve responsiveness of touches
Platforms
Steps to test
status: ready