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

refactor: move from styled-components to vanilla extract #425

Merged
merged 166 commits into from
May 21, 2024

Conversation

ggdaltoso
Copy link
Member

@ggdaltoso ggdaltoso commented Feb 9, 2024

I changed everything. That's it. But basically removed the styled-component dependency to use vanilla-extract.

Why? Better support.

styled-components is great and I appreciate it. But, do we need a CSS-in-JS library for React95?? 🤔

No. We don't.

With vanilla-extract we can rely on great TS support + CSS output.

It has been fun to maintain this library. You should give it a try 👀

Notable changes

Well, with no more styled-components dependency, it becomes easier to install the library. You now need to install just the lib itself. There is no need to install missing types and/or dependencies.

npm install @react95/core
# or
yarn add @react95/core
# or
pnpm add @react95/core

And that's it.

Visual changes

I also did a look around and updated some components. There are several pixel dances but they should look pretty much the same.

The Tree component was refactored to not use images and to have a more precise layout as we have in the original w95.

old new
image image

More options in TitleBar options with new icons

image

 - css
 - vite-plugin
just the bootstrap
…es, such as:

 - png
 - svg
 - ttf
 - eot
 - woff
 - woff2
Each cursor has now its own class! 🎉
themes need to be global since global theme make use of CSS variables from normal themes
 - border widths
 - border styles
AND NOW WE"RE TALKING
 - drop `size` one. It isn't from Avatar anymore. We should have it on Frame instead
@ggdaltoso ggdaltoso linked an issue Mar 29, 2024 that may be closed by this pull request
@ggdaltoso ggdaltoso changed the title 👀 Refactor/vanilla extract refactor: move from styled-components to vanilla extract May 9, 2024
@ggdaltoso ggdaltoso marked this pull request as ready for review May 9, 2024 01:18
@ggdaltoso ggdaltoso merged commit bc9325b into master May 21, 2024
7 checks passed
@ggdaltoso ggdaltoso deleted the refactor/vanilla-extract branch May 21, 2024 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Storybook failing
1 participant