fix: resolve QML Drag.active binding loop warnings in windowed launcher

[Ivy233]

Fix the binding loop detected for property "active" in the windowed launcher. Both IconItemDelegate and FreeSortListView bound Drag.active to the very MouseArea.drag.active property that drives it while reading Drag.active back on the same item (opacity/states/delayRemove), which QML reports as a binding loop. Set Drag.active imperatively from the MouseArea's drag.onActiveChanged handler instead of a binding.

修复窗口启动器的 Drag.active 属性绑定循环告警。
IconItemDelegate 与 FreeSortListView 都将 Drag.active 绑定到驱动它自身的 MouseArea.drag.active，同时又在同一控件上反向读取 Drag.active
（用于 opacity/states/delayRemove），触发 QML 绑定循环告警。
现改为在 mouseArea 的 drag.onActiveChanged 处理器中命令式赋值
Drag.active，从而消除该绑定环。

PMS: TASK-394335

Summary by Sourcery

Remove circular Drag.active bindings from the windowed launcher to prevent QML binding loop warnings during drag interactions.

Bug Fixes:

• Eliminate QML binding loop warnings in the windowed launcher's draggable icon delegates and list view.

Enhancements:

• Synchronize drag state imperatively to preserve drag behavior without circular property bindings.

[deepin-ci-robot]

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Ivy233

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

[sourcery-ai]

Hey - I've reviewed your changes and they look great!

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[deepin-ci-robot]

deepin pr auto review

AI 代码审查报告

PR: linuxdeepin/dde-launchpad#802
分支: fix/drag-binding-loop-warnings → master
作者: Ivy233
平台: GitHub
分析模式: 全量分析

---

总体评分

维度	评分	状态
语法逻辑	25/25	✓
代码质量	24/25	✓
代码性能	20/20	✓
代码安全	30/30	✓
总分	99/100	优秀

总体评价结论：代码审查通过

评价原因：本次提交修复了 QML Drag.active 绑定循环警告问题，采用命令式赋值替代属性绑定的方式打破了循环依赖。代码改动最小化，注释清晰解释了问题原因和修复方案，未发现安全漏洞，实际提升了运行时性能。

---

提交信息

• 标题: fix: resolve QML Drag.active binding loop warnings in windowed launcher
• 修改文件: qml/windowed/FreeSortListView.qml, qml/windowed/IconItemDelegate.qml
• 改动行数: +18 / -2

---

维度1：语法逻辑（25/25）✓

语法正确，逻辑清晰

审查内容：

1. 两个文件中均将 Drag.active: mouseArea.drag.active（属性绑定）修改为 Drag.active: false（初始值），并在 MouseArea 的 drag.onActiveChanged 回调中命令式设置 Drag.active = drag.active，语法完全正确。
2. drag.onActiveChanged 是 MouseArea drag 属性组的标准信号处理器，使用方式正确。
3. 在 FreeSortListView.qml 中，itemDelegate.Drag.active = drag.active 通过 id 引用正确设置父级 Item 的 Drag.active 属性。
4. 在 IconItemDelegate.qml 中，root.Drag.active = drag.active 通过 root id 引用正确设置 Control 的 Drag.active 属性。
5. 边界条件处理完善：Drag.active 初始值为 false，在拖拽开始时通过回调设为 true，拖拽结束时设为 false，状态转换完整。
6. 现有的 Drag.onActiveChanged 处理器（FreeSortListView.qml 第323行）在 Drag.active 变为 false 时调用 listViewDragScroller.stopScroll()，与新的命令式赋值方式兼容，逻辑无冲突。

问题列表：无

---

维度2：代码质量（24/25）✓

代码结构清晰，注释完整

审查内容：

1. 注释完整性（5/5）：两个文件中新增的注释详细解释了绑定循环的原因（"Binding Drag.active to mouseArea.drag.active would make Drag.active read back the very property it drives"）以及修复方案（"Set it imperatively from the MouseArea instead"），注释质量高，有助于后续维护者理解。
2. 代码重复（4/5）：两个文件中的修复模式和注释内容相似度较高，均描述了相同的绑定循环问题。虽然各自针对上下文做了适当调整（FreeSortListView 提到 opacity/delayRemove/onActiveChanged，IconItemDelegate 提到 opacity/states），但核心注释文本几乎一致，轻微冗余。
3. 结构合理性（5/5）：drag.onActiveChanged 处理器放置在 drag.target 附近，位置合理，与现有代码结构一致。
4. 调试信息清理（5/5）：无残留调试代码，无敏感注释。
5. 命名规范（5/5）：使用标准的 QML 命名约定，id 命名清晰（itemDelegate、root、mouseArea）。

问题列表：

1. 两个文件中的注释文本高度相似，可考虑在项目文档中统一记录此绑定循环问题的修复模式，减少注释重复。

---

维度3：代码性能（20/20）✓

性能良好，资源使用合理

审查内容：

1. 本次修复直接消除了 QML 绑定循环（binding loop），绑定循环会导致 QML 引擎反复求值并输出警告日志，消耗 CPU 资源和 I/O。修复后性能实际得到提升。
2. 命令式赋值（drag.onActiveChanged 中设置 Drag.active）仅在拖拽状态变化时触发一次，相比属性绑定的持续监听，开销更低。
3. 无不必要的计算、无频繁系统调用、无资源泄漏。
4. 无魔法数字或硬编码配置引入。

问题列表：无

---

维度4：代码安全（30/30）✓

存在0个安全漏洞

漏洞对比统计：新增漏洞 0 个，减少漏洞 0 个，持平 0 个

审查内容：

1. 本次改动为 QML 前端属性绑定方式调整，不涉及用户输入处理、网络请求、文件系统操作、命令执行等安全敏感操作。
2. 无硬编码密钥、无敏感信息泄露。
3. 无命令注入、SQL 注入、路径遍历等风险。
4. 无缓冲区操作、无反序列化操作。
5. 安全漏洞扫描工具（security_scanner.py）扫描结果：0 个漏洞。
6. OCR 专业代码审查结果：无问题发现。

安全漏洞清单：无

---

改进建议

虽然本次提交质量很高，以下为可选的改进建议：

1. 注释去重：两个文件中关于绑定循环的注释文本高度相似，可考虑在项目级 QML 编码规范文档中记录此模式，在各文件中引用即可。
2. 风格一致性：FreeSortListView.qml 新增的 drag.onActiveChanged: function () 使用了空格（function ()），而同文件中已有的 Drag.onActiveChanged: function() 未使用空格（第323行），建议统一函数表达式风格。

---

漏洞统计

指标	数值
当前漏洞总数	0
新增漏洞	0
修复漏洞	0
持平漏洞	0
风险等级	None

---

本报告由 AI 代码审查系统自动生成
扫描时间: 2026-08-31 17:52