-
Notifications
You must be signed in to change notification settings - Fork 812
引入 SCSS 支持 #5093
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
base: main
Are you sure you want to change the base?
引入 SCSS 支持 #5093
Conversation
|
看起来本 PR 会让 HMCL 在 Linux LoongArch64 平台上无法构建,需要谨慎调查。 |
若确有必要为 Linux LoongArch64 平台提供构建支持,可考虑使用 sass npm 来完成 scss 文件的编译。
|
在不支持 EmbeddedSass(Dart Sass) 的平台上,现在可以通过设置环境变量或构建参数的方式使用外部命令行工具构建 SASS |
|
我还是感觉把这个导入构建流程不是特别合适。如果可以的话,我希望同时在代码里保留 css 和 scss,仅在开发者修改了 scss 的情况下才需要重建 css。 |
现在会在源码编译前计算 scss 文件夹的 sha256 然后比对 _sha256.scss 的内容判断 scss 是否更新,若 scss 未更新则跳过 build css 任务。 贡献者在提交 css/scss 相关变更时应确保:
此外可能需要引入一项新的检查流程,用于判断 CSS 文件与 SCSS 文件是否相匹配,以及 SCSS 文件夹的 sha256 是否与 _sha256.scss 文件一致。 |
引入 SCSS 支持
当前项目使用 CSS 维护 JavaFX 样式,随着样式规模扩大,逐渐暴露出维护成本高、复用能力弱、结构松散等问题。本次 PR 引入 SCSS 作为样式预处理方案,旨在解决现有 CSS 的局限。
本次 PR 未引入新的样式也未修改旧的样式,理论上 SCSS 编译产生的 CSS 应与主线基本保持一致。