Skip to content

Conversation

@18202781743
Copy link
Contributor

Remove it because of consuming performance.

pms: BUG-365879

Remove it because of consuming performance.

pms: BUG-365879
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • blendColorAlpha 函数在两个地方被定义,建议合并为一个函数定义以避免重复代码。
  • opacity 属性的计算在两个地方重复,建议提取为一个公共函数或变量。
  • 注释掉的 D.BoxShadow 代码被注释掉了,但没有移除,可能会引起混淆。如果不再需要,应该完全删除。
  • Loader 组件的 idaniamtionRoot 改为 animationRoot,这是一个拼写错误,应该修正。

是否建议立即修改:

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, mhduiy

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@18202781743 18202781743 merged commit 51e8e3d into linuxdeepin:master Jan 9, 2025
9 of 10 checks passed
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.

3 participants