-
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
Address design feedback on Profile share tab Screen #16257
Conversation
6a56384
to
ab86e22
Compare
Jenkins BuildsClick to see older builds (86)
|
ab86e22
to
6f127fc
Compare
I Will wait to hear from @mariocnovoa on the design review as well before merging this in. |
Nice, I'm also planning to check with figma this evening 👌 |
36% of end-end tests have passed
Failed tests (21)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Passed tests (12)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityOneDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityMultipleDevicePR:
|
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
7617bbc
to
be50fac
Compare
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.
Non blocking comment
5e26706
to
80e1e27
Compare
|
80e1e27
to
edc9e69
Compare
Hey @siddarthkay ! Is PR ready for test? |
edc9e69
to
b20d15c
Compare
Hi @pavloburykh : Sorry not yet, I've just now requested a re-review by Design Team. |
b20d15c
to
3e0527f
Compare
69fc219
to
a0b433a
Compare
@qoqobolo @pavloburykh @churik : I've fixed all the issues. Apologies for the delay. |
37% of end-end tests have passed
Failed tests (27)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Passed tests (16)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
|
647c0f7
to
e291be3
Compare
79% of end-end tests have passed
Failed tests (9)Click to expandClass TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (34)Click to expandClass TestActivityMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePRTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
|
@siddarthkay thanks for the fixes! I see you have updated the link format on the IMG_1708.MP4 |
Hi @qoqobolo : you're right, I missed this, I'll revert the links back to old format. |
9ff9588
to
d1f0254
Compare
Thanks for your work @siddarthkay! |
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 to me ✨
0bbb7b4
to
50682c2
Compare
This commit adds UI Touchups to profile share screen according to Design Review.
50682c2
to
9620179
Compare
fixes #16080
Testing notes
No functionality was changed, only UI Touchups
Platforms
status: ready