-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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: use design system windicss config #21503
Changes from 4 commits
a5bd89b
c1a85bf
f3740d3
1945916
dd96433
d0f8114
b656eab
b637d94
cfd1cb3
4191ddf
3a19570
7c191a6
c326da6
dc62e8d
f2cb0d2
80002d9
51376db
c5dbba1
5570637
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -19,6 +19,7 @@ | |
}, | ||
"dependencies": {}, | ||
"devDependencies": { | ||
"@cypress-design/css": "0.1.0", | ||
"@graphql-typed-document-node/core": "^3.1.0", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not directly related but do you think we should consider changing to TailwindCSS at this point for future sustainability? It has way more contributors, and just looking at the repo we have 18 issues (Tailwind) vs 172(Windi). I feel like Tailwind will likely outlast and be a more sustainability option. I have no idea of the cost of migration - should be small, since Windi is "Tailwind compatible". I don't see a strong benefit to Windi since Tailwind now has a JIT compiler, which was the main issue Windi solved (speed). Thoughts? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Unless there is a compelling reason (bug, needed enhancement), I would prefer to avoid unnecessary changes. Among the differences, TailwindCSS is a PostCSS-based plugin instead of being independent in WindiCSS. It means that in WindiCSS the plugin controls the configuration. With TailwindCSS, we would have to export a TailwindCSS config. There are a few configurations as well that would need to migrate. Colors, safelist, plugins. |
||
"@headlessui/vue": "1.4.0", | ||
"@iconify/iconify": "2.1.2", | ||
|
@@ -34,7 +35,6 @@ | |
"@vitejs/plugin-vue": "2.2.4", | ||
"@vitejs/plugin-vue-jsx": "1.3.8", | ||
"@vueuse/core": "7.2.2", | ||
"@windicss/plugin-interaction-variants": "1.0.0", | ||
"ansi-to-html": "0.6.14", | ||
"bluebird": "3.5.3", | ||
"classnames": "2.3.1", | ||
|
@@ -68,7 +68,6 @@ | |
"vue-i18n": "9.2.0-beta.7", | ||
"vue-router": "4", | ||
"vue-tsc": "^0.3.0", | ||
"windicss": "3.1.4", | ||
"wonka": "^4.0.15" | ||
}, | ||
"files": [ | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
import { defaultConfig } from '@packages/frontend-shared/windi.config' | ||
import { defineConfig } from 'windicss/helpers' | ||
/** | ||
* This file is used for vscode plugin autocompletion of windi colors | ||
*/ | ||
|
||
export default defineConfig(defaultConfig) | ||
export { default } from '@packages/frontend-shared/windi.config' |
This file was deleted.
This file was deleted.
This file was deleted.
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.
Just learned that https://github.com/cypress-io/cypress-design exists!