feat: 完整主题系统实现 #84
Open
feat: 完整主题系统实现 #84
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
hl128k
commented
Aug 22, 2025
hl128k
commented
- 添加外部主题加载功能，支持从自定义目录加载主题文件
- 实现ConfigStorage集成，统一主题设置持久化存储
- 新增多个内置主题：亮色、暗色、墨绿深邃主题
- 添加示例主题文件：蓝色、深紫色、森林绿色、海洋蓝色主题
- 完善主题切换体验，支持自动模式和手动切换
- 解决主题设置在应用重启后的持久化问题
- 优化ThemeProvider组件，避免加载顺序导致的竞态条件
- 添加主题设置UI界面，支持主题目录配置
- 更新国际化文件，支持多语言主题相关文案
- 统一代码风格，移除调试日志
- 添加外部主题加载功能，支持从自定义目录加载主题文件 - 实现ConfigStorage集成，统一主题设置持久化存储 - 新增多个内置主题：亮色、暗色、墨绿深邃主题 - 添加示例主题文件：蓝色、深紫色、森林绿色、海洋蓝色主题 - 完善主题切换体验，支持自动模式和手动切换 - 解决主题设置在应用重启后的持久化问题 - 优化ThemeProvider组件，避免加载顺序导致的竞态条件 - 添加主题设置UI界面，支持主题目录配置 - 更新国际化文件，支持多语言主题相关文案 - 统一代码风格，移除调试日志
|
合并了之后,提示了一堆eslint的错误一键修复了一下,可能影响挺多了,现在的主题只是有,用于测试的,自认为审美不行,主要是实现功能 |
|
由于近期我们也统一的格式问题，现在冲突较多，请先处理下； |
|
格式问题我这边来修复一下，但是因为主题涉及到东西很多，基本每个页面都会有一点改动的，我是以最新主分支做的改动； |
|
是的，我看到你代码有对所有的样式进行定义，不过这个跟我们想得有一些区别；在样式控制上我们设想的尽量还是通过css来控制，可以做成主题包，支持比较灵活的修改。由于前期使用了unocss 我们现在并不具备模块块的css控制能力，所以我们在规划对所有模块尽量的进行一个全面标准的css定义，并且支持导出和导入能力。 |
|
那看样子要进行大型更新了，等你们有定义了再说 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.