sync: from linuxdeepin/qt5integration

[deepin-ci-robot]

Synchronize source files from linuxdeepin/qt5integration.

Source-pull-request: linuxdeepin/qt5integration#286

[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. 代码质量改进：

• 在drawSliderHandleFocus函数中，将重复的路径绘制逻辑提取为createPath匿名函数是一个很好的改进，提高了代码复用性和可维护性。
• 在polish函数中，对窗口半径和边框颜色的设置逻辑更加清晰，增加了主题切换时的动态更新。

2. 性能优化：

• 去掉了frameRadius的重复计算，直接使用height/2，减少了不必要的计算。
• 优化了绘制顺序，将clipPath的设置提前，避免了重复设置。

3. 安全性改进：

• 在多处增加了对styleObject的空指针检查，如：

if (opt->styleObject && opt->styleObject->property("_d_slider_spaceLeft").isValid())

这增加了代码的健壮性。

4. 视觉效果改进：

• 对焦点框的绘制进行了优化，增加了内外两层效果，使焦点更明显：

// 绘制focus的外圈
p->drawRoundedRect(rectHandle.adjusted(outerRectOffect, outerRectOffect, -outerRectOffect, -outerRectOffect), ...);

// 绘制focus的内高亮圈
pen.setColor(getThemTypeColor(QColor("#F7F7F7"), QColor("#323232")));

5. 建议改进：

• 在drawControl中的按钮绘制部分，可以考虑将颜色设置逻辑提取为单独的函数，提高代码复用性。
• 在polish函数中，建议将setWindowNoEffect的调用统一管理，避免重复代码。
• 对于getThemTypeColor函数的使用，建议增加参数验证，确保传入的颜色值有效。

6. 其他建议：

• 建议为新增的函数添加适当的注释说明，特别是createPath匿名函数的参数含义。
• 考虑将一些魔法数字（如0.07, 0.05等）定义为常量，便于维护。
• 在处理主题切换时，建议增加错误处理机制，防止因主题切换失败导致的显示异常。

总体来说，这次改动主要优化了UI绘制效果和代码结构，提高了代码的可维护性和健壮性。建议在后续开发中继续保持这些良好的编程实践。

[18202781743]

/topic dtk-cpv20

[deepin-ci-robot]

Add topic: dtk-cpv20 successed.