Skip to content

Replace Tailwind with Styled components#1

Open
jennysky wants to merge 4 commits intosharonamr:mainfrom
jennysky:styled-components
Open

Replace Tailwind with Styled components#1
jennysky wants to merge 4 commits intosharonamr:mainfrom
jennysky:styled-components

Conversation

@jennysky
Copy link
Copy Markdown

@jennysky jennysky commented Dec 8, 2022

  • Replace tailwind with Styled components

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Author

Choose a reason for hiding this comment

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

removed

Copy link
Copy Markdown
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
Copy Markdown
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