-
Notifications
You must be signed in to change notification settings - Fork 1
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
Replace Tailwind with Styled components #1
base: main
Are you sure you want to change the base?
Conversation
jennysky
commented
Dec 8, 2022
•
edited
Loading
edited
- Replace tailwind with Styled components
2238e73
to
c946772
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.
@jennysky Thank you so much for you PR (first one for this project!)
This PR introducing some bugs in the UI, also missing the ability to control the look and feel of the components under core.
@layer components { | ||
/* Docs styling */ | ||
.blox-code-editor { | ||
@apply rounded-lg mb-4 font-mono text-sm bg-gray-900 text-gray-300; |
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.
This what makes everything themeable , so when after you run create-blox-app
you'll be able to edit the index.css for your liking.
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.
class blox-code-editor
doesn't exists anymore,
I removed all the name classes and all the tailwind.
so I am not sure
do you want to keep empty index.css ?
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 meant that we need a way to control core components styles at docs level.
maybe with createGlobalStyle?
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.
added bloxStylesOverrides to createGlobalStyle, it allow to overrides all the styles in core
all the rest styles can be overrided in docs
.prettierrc
Outdated
@@ -0,0 +1,15 @@ | |||
{ |
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.
Adding prettier is out of scope for this PR.
I guess adding eslint is a good idea
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.
it was too many files , so it is hard to write without formating
I can remove it
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.
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.
Please also revert prettier changes - i plan to add eslint soon + validate it on ci
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.
reverted prettier of the most of the unchanged files
77ce332
to
5d7aa7c
Compare