-
Notifications
You must be signed in to change notification settings - Fork 49
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
antd theme customization broken with next-plugin-antd-less versions >= 1.0.0 #36
Comments
this file https://github.com/rriski/next-plugin-antd-less/blob/main/assets/antd-custom.less will change to @import '~antd/lib/style/themes/default.less'; <-- You need to import it at once in your project
@primary-color: #52c41a;
//
//@table-bg: "red"; <-- 'red' is not a valid css value.
//@table-header-bg: "blue";
@table-bg: red;
@table-header-bg: blue; The previous version used 'less-vars-to-js' to help with the conversion and the problem may have been hidden, but now I've removed 'less-vars-to-js'. The problem came up. |
Of course, all of these changes were made just so that the changes to |
It would be better to specify in the README that the default Ant variables file must be imported 😃 |
Confirming that adding |
Doesn't work for Antd 4.15+ |
Any solution for this? My antd version is 4.16.5 using latest next.js and tailwindcss |
Steps to reproduce:
git clone https://github.com/rriski/next-plugin-antd-less
yarn && yarn dev
next-plugin-antd-less
to version 1.0.5.yarn && yarn dev
The repo follows https://github.com/vercel/next.js/tree/canary/examples/with-ant-design-less with next-plugin-antd-less added. This plugin supported customizing the antd theme (https://ant.design/docs/react/customize-theme) prior to v1.0.0.
I suppose this commit is to blame: 418ec5c.
The text was updated successfully, but these errors were encountered: