-
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
fixed dark border around the community logo #18314
Conversation
Jenkins BuildsClick to see older builds (26)
|
50% of end-end tests have passed
Failed tests (20)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Expected to fail tests (4)Click to expandClass TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (24)Click to expandClass TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestDeepLinksOneDevice:
|
88% of end-end tests have passed
Failed tests (1)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Expected to fail tests (5)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Passed tests (42)Click to expandClass TestDeepLinksOneDevice:
Class TestActivityMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestCommunityOneDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
|
83c4990
to
9d3303d
Compare
a5b088c
to
63f9156
Compare
@Francesca-G I worked on this and looks now. You could take a look and maybe confirm |
looks good to me as well ✨ |
5b12fe2
to
51f5f14
Compare
51f5f14
to
2e9c95e
Compare
Hi @jo-mut! Thanks fro your PR! Can I ask you for a small comment like "design review is required" for your future PR's in case it is needed. |
@mariia-skrypnyk I may have just missed a step and involved the design review earlier. Just as a point of clarification you mean, the design review comes after pr is QA tested is that what you mean? |
2e9c95e
to
965aadb
Compare
Thanks @jo-mut for your comment. Yes, if you PR needs a design review this step comes after QA's testing. We involve designers by dragging the card to the Design review column. You can find all this described in this doc here. |
This PR removes the dark border around the community logo on communities page. The bug is only reproducible when the user has not yet joined the community
fixes #17565
Before & After screen shots