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

完善本地取色缺少的参数 #453

Merged
merged 7 commits into from
Dec 26, 2024
Merged

完善本地取色缺少的参数 #453

merged 7 commits into from
Dec 26, 2024

Conversation

Linux-qitong
Copy link
Contributor

@Linux-qitong Linux-qitong commented Dec 25, 2024

🔗 链接 issue

❓ 修改类型

  • [x ] 🐞 Bug fix (修复问题的连续性改)
  • [x ] 👌 增强功能(改进现有功能,如性能)
  • ✨ Feature(添加功能的连续性修改)
  • ⚠️ 中断修改(重大的修改,如配置文件调整、模块移除)

📚 描述

📝 清单

  • 我已链接问题或讨论。
  • 我已经添加了测试(如果可能的话)。
  • 我已相应地更新了文档

Linux-qitong and others added 4 commits December 26, 2024 00:05
Signed-off-by: 青稚 <[email protected]>
Signed-off-by: 青稚 <[email protected]>
经过测试,这个代码可以去除


Signed-off-by: 青稚 <[email protected]>
@Linux-qitong
Copy link
Contributor Author

Linux-qitong commented Dec 26, 2024

everfu and others added 2 commits December 26, 2024 12:23
Key Optimizations:
1.Minimized DOM Access: Using and for batch processing instead of repetitive .document.querySelectorAllforEachgetElementsByClassName
2.Reduced Repetitive Calculations: Cached values and made calculations inline where appropriate.
3.Modern JavaScript Features: Used destructuring, , and arrow functions for cleaner and more concise code.const

关键优化:
1.最小化 DOM 访问:使用 和 进行批处理,而不是重复 .document.querySelectorAllforEachgetElementsByClassName
2.减少重复计算:缓存值并在适当的情况下进行内联计算。
30现代 JavaScript 功能:使用解构、 和 箭头函数 来获得更简洁、更简洁的代码。const

Signed-off-by: 青稚 <[email protected]>
@Linux-qitong
Copy link
Contributor Author

最新提交
关键优化:
1.最小化 DOM 访问:使用 和 进行批处理,而不是重复 .document.querySelectorAllforEachgetElementsByClassName
2.减少重复计算:缓存值并在适当的情况下进行内联计算。
3.现代 JavaScript 功能:使用解构、 和 箭头函数 来获得更简洁、更简洁的代码。const

@everfu everfu merged commit ec8144a into everfu:dev Dec 26, 2024
2 checks passed
@Linux-qitong Linux-qitong deleted the patch-1 branch December 26, 2024 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants