Skip to content
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

feat(view): ✨ 增加透明高斯模糊开关 #177

Merged
merged 1 commit into from
Jan 11, 2025
Merged

Conversation

nongyehong
Copy link
Member

@nongyehong nongyehong commented Jan 11, 2025

💻 变更类型 | Change Type

  • ✨ feat | 新增功能
  • 🐛 fix | 修复缺陷
  • ♻️ refactor | 代码重构(不包括 bug 修复、功能新增)
  • 💄 style | 代码格式(不影响功能,例如空格、分号等格式修正)
  • 📦️ build | 构建流程、外部依赖变更(如升级 npm 包、修改 vite 配置等)
  • 🚀 perf | 性能优化
  • 📝 docs | 文档变更
  • 🧪 test | 添加疏漏测试或已有测试改动
  • ⚙️ ci | 修改 CI 配置、脚本
  • ↩️ revert | 回滚 commit
  • 🛠️ chore | 对构建过程或辅助工具和库的更改(不影响源文件、测试用例)

🔀 变更说明 | Description of Change

📝 补充信息 | Additional Information

@github-actions github-actions bot added 前端 关于前端的代码修改 样式 labels Jan 11, 2025
@nongyehong nongyehong linked an issue Jan 11, 2025 that may be closed by this pull request
Copy link

codecov bot commented Jan 11, 2025

Copy link

PR 代码分析

### 代码变更分析

1. 代码逻辑的改动

  • 样式调整:多个组件(ApplyList.vueChatMain.vueChatSidebar.vue)中删除了 backdrop-filter: blur(10px) 样式。这表明开发人员可能决定移除或暂时禁用高斯模糊效果。
  • 设置项增加:在 setting.ts 中增加了 blur 字段,默认值为 true,用于控制是否开启透明高斯模糊效果。同时在 General.vue 中添加了相应的开关控件。

2. 潜在的问题或优化空间

  • 一致性问题:删除 backdrop-filter 样式的同时,在设置中增加了 blur 选项,可能存在前后端样式不一致的情况。建议确保所有相关组件根据设置动态应用样式。
  • 性能影响:高斯模糊效果对性能有一定影响,尤其是在移动设备上。虽然目前只是提供了开关,但建议在启用时进行性能测试,确保用户体验不受影响。

3. TypeScript 类型定义的准确性

  • 类型扩展:在 stores.d.ts 中正确地扩展了 page 对象的类型定义,增加了 blur 字段,并且注释清晰明确,符合 TypeScript 的最佳实践。

4. Vue 组件的性能影响

  • 渲染效率:新增的 n-switch 组件和样式调整不会显著影响性能,但在大规模使用时仍需关注。特别是 backdrop-filter 效果的启用和禁用,可能会对 GPU 渲染带来额外负担。

5. Rust 代码的安全性和性能

  • 无关变更:本次变更不涉及 Rust 代码,因此无需评估 Rust 代码的安全性和性能。

总结

本次变更加强了用户界面的可配置性,通过引入 blur 设置项,允许用户自定义界面效果。然而,需要注意样式调整的一致性和潜在的性能影响。建议在后续版本中进一步优化和测试,确保用户体验和性能的最佳平衡。

*这是由通义千问 AI 自动生成的 PR 分析,仅供参考。*

@nongyehong nongyehong merged commit cefe53d into master Jan 11, 2025
12 of 14 checks passed
@nongyehong nongyehong deleted the feat/blur-switch branch January 11, 2025 04:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
前端 关于前端的代码修改 样式
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 增加透明高斯模糊开关
1 participant