-
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
Collectible counter component #18632 #18653
Conversation
Jenkins BuildsClick to see older builds (16)
|
cd96479
to
16ae7b7
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.
Hey @Rende11
Thanks for the PR, LGTM.
16ae7b7
to
8e2a452
Compare
@Francesca-G - can we get a design review of this component? 🙏 |
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.
Here's the design review
I left a comment about a color issue in the Error state, besides that it looks good :)
62% of end-end tests have passed
Failed tests (16)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestCommunityOneDeviceMerged:
Class TestDeepLinksOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Expected to fail tests (2)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Class TestCommunityOneDeviceMerged:
Passed tests (30)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityMultipleDevicePRTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
|
Fix issues Remove docs Remove unused bindings Fix color
8e2a452
to
9bf10e4
Compare
@Francesca-G - could you please check this component one more time - I believe @Rende11 addressed the ui issues. @pavloburykh - can I merge this pr afterwards? I ran the e2e tests above ^^ - it's also just a quo component btw |
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.
@J-Son89 all good now 👍
Fixes #18632
Summary
This PR introduces new component - Collectible counter (design)
It's expects only blured background
Design
Should there be the same border color forNoerror
status for dark and light themes?Before and after screenshots comparison
Simulator.Screen.Recording.-.iPhone.13.-.2024-01-27.at.21.22.31.mp4
iOS screenshots
status: ready