Skip to content
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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jennysky
Copy link

@jennysky jennysky commented Dec 8, 2022

  • Replace tailwind with Styled components

Copy link
Owner

@sharonamr sharonamr left a 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;
Copy link
Owner

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.

Copy link
Author

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 ?

Copy link
Owner

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?

Copy link
Author

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 @@
{
Copy link
Owner

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

Copy link
Author

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

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed

Copy link
Owner

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

Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants