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: wrong color stroke in profile setting #19815
fix: wrong color stroke in profile setting #19815
Conversation
Jenkins BuildsClick to see older builds (40)
|
be012d7
to
0786f96
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.
Thank you for the great work!
96% of end-end tests have passed
Expected to fail tests (2)Click to expandClass TestCommunityOneDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (50)Click to expandClass TestCommunityOneDeviceMerged:
Class TestDeepLinksOneDevice:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestWalletMultipleDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
Class TestWalletOneDevice:
Class TestActivityMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
|
29e52c5
to
c2bb94e
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.
I understand it's a bit tricky because the border needs to be both blurred and transparent, which can be challenging to achieve. Simulating the border color to match the background also seems difficult. Overall, LGTM :) nice work
c2bb94e
to
f8647a8
Compare
Hey @codemaster115 . Thank you for PR. Take a look at the found issue PR_ISSUE 1: Wrong UI on shell screens when light mode is enabledActual result:Activity Center Expected result:Activity Center |
4357344
to
a6f1fc7
Compare
90% of end-end tests have passed
Failed tests (3)Click to expandClass TestWalletMultipleDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Expected to fail tests (2)Click to expandClass TestCommunityOneDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (47)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestDeepLinksOneDevice:
Class TestCommunityOneDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestWalletMultipleDevice:
Class TestWalletOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
|
@VolodLytvynenko Please check again. Thanks |
5923c84
to
94fc8bc
Compare
100% of end-end tests have passed
Passed tests (1)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
|
Hi @codemaster115 thank you for quick issue fix :) No issues from my side @Francesca-G could you please take a look current PR? |
2362c34
to
e4f8d59
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.
e4f8d59
to
67fc0eb
Compare
67682d6
to
c724220
Compare
fixes #19300
Summary
Stroke around the avatar in profile setting is wrong.
Platforms
Areas that maybe impacted
Functional
Steps to test
Before and after screenshots comparison
status: ready