Skip to content

fix: clear DComboBox popup highlight on mouse leave#734

Merged
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
electricface:swt/fix-bug304991
Mar 23, 2026
Merged

fix: clear DComboBox popup highlight on mouse leave#734
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
electricface:swt/fix-bug304991

Conversation

@electricface
Copy link
Copy Markdown
Member

  • Add eventFilter to handle Leave events, clearing highlight when
    mouse exits popup
  • Remember the previous index so keyboard navigation can continue
    from where it left off
  • Update example code to use DComboBox consistently
  • Add unit tests for popup leave and keyboard navigation behavior

Log: fix popup highlight not cleared when mouse leaves dropdown
Influence: DTK Widget DComboBox
PMS: BUG-304991

fix: DComboBox 下拉弹窗鼠标离开时清除高亮

  • 添加 eventFilter 处理 Leave 事件,鼠标移出弹窗时清除高亮
  • 记住之前的索引位置,使键盘导航可以从离开处继续
  • 更新示例代码统一使用 DComboBox
  • 添加单元测试覆盖弹窗离开和键盘导航行为

Log: 修复下拉框弹窗鼠标离开时未清除高亮问题
Influence: DTK Widget DComboBox
PMS: BUG-304991

@deepin-ci-robot
Copy link
Copy Markdown
Contributor

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

@electricface electricface marked this pull request as ready for review March 23, 2026 09:34
18202781743
18202781743 previously approved these changes Mar 23, 2026
- Add eventFilter to handle Leave events, clearing highlight when
mouse exits popup
- Remember the previous index so keyboard navigation can continue
from where it left off
- Update example code to use DComboBox consistently
- Add unit tests for popup leave and keyboard navigation behavior

Log: fix popup highlight not cleared when mouse leaves dropdown
Influence: DTK Widget DComboBox
PMS: BUG-304991

fix: DComboBox 下拉弹窗鼠标离开时清除高亮

- 添加 eventFilter 处理 Leave 事件,鼠标移出弹窗时清除高亮
- 记住之前的索引位置,使键盘导航可以从离开处继续
- 更新示例代码统一使用 DComboBox
- 添加单元测试覆盖弹窗离开和键盘导航行为

Log: 修复下拉框弹窗鼠标离开时未清除高亮问题
Influence: DTK Widget DComboBox
PMS: BUG-304991
@deepin-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

@electricface
Copy link
Copy Markdown
Member Author

/forcemerge

@deepin-bot
Copy link
Copy Markdown
Contributor

deepin-bot bot commented Mar 23, 2026

This pr force merged! (status: blocked)

@deepin-bot deepin-bot bot merged commit f9cb55e into linuxdeepin:master Mar 23, 2026
20 of 23 checks passed
@electricface electricface deleted the swt/fix-bug304991 branch March 23, 2026 10: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.

3 participants