-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
add cloud version #5198
add cloud version #5198
Conversation
@jamakase when can we merge this. I know not everything is done but maybe we can do other features and bug fixes in a separate PR? |
const Title = styled(H5)` | ||
padding: 25px 25px 22px; | ||
border-bottom: 1px solid ${({ theme }) => theme.greyColor20}; | ||
//border-bottom: 1px solid ${({ theme }) => theme.greyColor20}; |
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.
@jamakase should this be removed?
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.
Actually no, this is part of new UI where we update borders and shadows. Decided to leave it as a comment to check all places more properly.
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 think I spotted one typo, otherwise I did a very cursory glance through.
What
Adds some new concepts into oss version:
Adds cloud version