sync: from linuxdeepin/dtkdeclarative

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtkdeclarative.

Source-pull-request: linuxdeepin/dtkdeclarative#395

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: deepin-ci-robot

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

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

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

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. QML文件中的可见性逻辑：

   • 在CheckDelegate.qml文件中，Rectangle的visible属性被修改为!control.checked && !D.DTK.hasAnimation && control.hovered。这个修改可能会影响控件的显示逻辑，需要确认这个逻辑是否符合设计要求。如果D.DTK.hasAnimation和control.hovered是控制显示的关键因素，那么这个修改是合理的。否则，应该重新评估这个逻辑。

2. ComboBox的属性添加：

   • 在ComboBox.qml文件中，添加了horizontalAlignment属性，并在TextField和Text组件中使用。这个修改是合理的，因为它增加了对文本对齐的控制，但需要确保这个属性在所有情况下都能正确工作，并且与现有的布局和样式保持一致。

3. ItemDelegate的激活逻辑：

   • 在ItemDelegate.qml文件中，Loader的active属性被修改为!checked && control.normalBackgroundVisible && !D.DTK.hasAnimation && control.hovered。这个修改与CheckDelegate.qml中的修改类似，需要确认这个逻辑是否符合设计要求。如果D.DTK.hasAnimation和control.hovered是控制激活的关键因素，那么这个修改是合理的。否则，应该重新评估这个逻辑。

4. 代码风格和一致性：

   • 在ComboBox.qml和ItemDelegate.qml文件中，添加了新的属性和逻辑，但并没有更新相关的文档或注释。建议添加相应的文档或注释，以便其他开发者理解这些修改的目的和影响。

5. 性能考虑：

   • 在CheckDelegate.qml和ItemDelegate.qml文件中，添加了对D.DTK.hasAnimation和control.hovered的检查。这些检查可能会影响性能，特别是在动画频繁发生的情况下。建议评估这些检查对性能的影响，并考虑是否有更高效的方法来实现相同的功能。

6. 安全性：

   • 代码中没有明显的安全问题，但需要确保所有用户输入都经过适当的验证和清理，以防止潜在的安全漏洞，如XSS攻击。

总体来说，这些修改提高了代码的可读性和灵活性，但需要进一步确认逻辑的正确性和性能影响。同时，建议添加必要的文档和注释，以便其他开发者更好地理解这些修改。