-
Notifications
You must be signed in to change notification settings - Fork 985
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
[#18929] fix: no bottom space under the join button #18987
Conversation
Jenkins BuildsClick to see older builds (15)
|
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.
Looks good! I left one small comment about consistency between iOS and Android.
src/status_im/contexts/communities/actions/accounts_selection/style.cljs
Outdated
Show resolved
Hide resolved
12% of end-end tests have passed
Failed tests (41)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePR:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (6)Click to expandClass TestDeepLinksOneDevice:
Class TestCommunityOneDeviceMerged:
|
ec4cabb
to
f83250d
Compare
92% of end-end tests have passed
Failed tests (3)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMergedTwo:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (44)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestDeepLinksOneDevice:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityCenterContactRequestMultipleDevicePR:
|
f83250d
to
0a3a2ae
Compare
0a3a2ae
to
c0c2070
Compare
Hi @mohsen-ghafouri ! Issue on SE is fixed but new one appeared on bigger models (too big margin) iPhone 13: |
c0c2070
to
0cb971b
Compare
Thank you @mariia-skrypnyk for testing this PR, please check again. |
Hi @mohsen-ghafouri ! iPhone is fixed thanks! But I've compared 2 android builds: last one and previous (March 6) and on the last one I found such new issue: ISSUE 2: Avatar is too close to status bar ISSUE 3: bottom space under the join button on Android *it was not only at the last android build but I found it only now (sorry for this) while using dark theme. |
@mariia-skrypnyk for issue 3, in iOS there are more spaces above handler, as you can see the white area below the green section is safe area, we added the same padding(12px) in both iOS and android. for issue 2 i don't think it's related to my PR, I only apply changes to the bottom of "Request to join" screen, the screenshot doesn't seem to be related to this screen. |
Hi @mohsen-ghafouri ! Yes, I was not sure in issue 3. |
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.
Bottom margin doesn't match design, Figma comment here
Hey @Francesca-G, thank you for reviewing this. please check this comment #18987 (comment) this is safe area that provide by OS. we have this space also in other modals |
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.
@mohsen-ghafouri apologies and thanks for the clarification 🙏
Looks good to me!
0cb971b
to
728ae92
Compare
fixes #18929
fixes #18997
Summary
No bottom space under the 'Slide to request to join' button
Platforms
Result
status: ready