-
Notifications
You must be signed in to change notification settings - Fork 43
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
fix: generate typed css module #534
base: master
Are you sure you want to change the base?
Conversation
@trim21 this pull request is now in conflict 😩 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #534 +/- ##
=======================================
Coverage 82.43% 82.43%
=======================================
Files 98 98
Lines 5942 5942
Branches 459 459
=======================================
Hits 4898 4898
Misses 1038 1038
Partials 6 6 ☔ View full report in Codecov by Sentry. |
@@ -9,27 +9,27 @@ import style from './SubMenu.module.less'; | |||
const buildMenuBottom = (verb: '看' | '读' | '听' | '玩'): MenuItemType[] => [ | |||
{ | |||
key: 'do', | |||
className: style['subMenuItem--do'], |
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.
这里之前写的也是错的,目前vite module的设置只会生成camalCase的class name
Preview Deployment
|
@trim21 this pull request is now in conflict 😩 |
@trim21 this pull request is now in conflict 😩 |
@trim21 this pull request is now in conflict 😩 |
@trim21 this pull request is now in conflict 😩 |
@trim21 this pull request is now in conflict 😩 |
@trim21 this pull request is now in conflict 😩 |
@trim21 this pull request is now in conflict 😩 |
#473 (comment)
不太确定是不是需要
vite dev
的情况下会watch*.module.less
然后生成对应的.d.ts
.试了一下似乎还是得生成
*.module.less.d.ts
,生成*.module.d.less.ts
并不能被正确识别...这里构建失败是因为之前写的有bug... 这个
editorItemCursor
在runtime是不存在的。需要 @FoundTheWOUT 修一下
看来还是有必要的(