-
Notifications
You must be signed in to change notification settings - Fork 0
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
Migrate to chakra #73
base: master
Are you sure you want to change the base?
Conversation
@a-saran kindly let me know when this is done and ready for review. |
@a-saran you will also need to write docs for your components. explain what they do . Use command yarn docz:dev When you write docs you will understand why you need those components. |
please use the design system with taste and right levels of abstraction. if you are spilling out too many props you probably need a well named component |
@kalidasm can you review this ? |
f922b17
to
658f95a
Compare
height={'1px'} | ||
bg={'black.3'} | ||
/> | ||
) | ||
} | ||
</div> | ||
))} |
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.
- Mobile View is not responsive.
- Headers are not looking as intended
Seems styles are missing in /blog
Page
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.
yes, blog pages are loading from @codebrahma/gatsby-theme-blog package, theme is hardcoded in that package. I am working on it
No description provided.