Skip to content

fix: enable QComboBox wheel event patch - #67

Merged
lzwind merged 1 commit into
deepin-community:masterfrom
wjyrich:fix-NoPatch
Sep 21, 2026
Merged

lzwind merged 1 commit into
deepin-community:masterfrom
wjyrich:fix-NoPatch

Conversation

@wjyrich

@wjyrich wjyrich commented Sep 21, 2026

Copy link
Copy Markdown
Contributor
  1. Add fix-ComboBox-Wheelevent.patch to debian/patches/series
  2. Update debian/changelog for version 6.8.0+dfsg-0deepin36
  3. The patch allows QComboBox to propagate wheel events to parent scroll widgets when the style hint disables wheel scrolling
  4. This restores expected scrolling behavior in embedded or container- based UI layouts

Influence:

  1. Test QComboBox inside a scrollable container (e.g. QScrollArea) and verify wheel events scroll the parent when combo box wheel scrolling is disabled via style hint
  2. Verify that QComboBox still scrolls its items when wheel scrolling is enabled
  3. Test wheel event behavior across different styles/platforms to ensure the style hint is respected
  4. Confirm no regression in normal combo box interactions (selection, popup display)
  5. Verify the patch applies cleanly during package build

fix: 启用 QComboBox 滚轮事件补丁

  1. 将 fix-ComboBox-Wheelevent.patch 添加到 debian/patches/series
  2. 更新 debian/changelog 至版本 6.8.0+dfsg-0deepin36
  3. 该补丁使 QComboBox 在样式提示禁用滚轮滚动时将滚轮事件传递给父级滚动 控件
  4. 恢复嵌入式或容器化 UI 布局中的预期滚动行为

Influence:

  1. 测试位于可滚动容器(如 QScrollArea)中的 QComboBox,验证在样式提示禁 用组合框滚轮滚动时滚轮事件能滚动父级
  2. 验证启用滚轮滚动时 QComboBox 仍能滚动其选项
  3. 在不同样式/平台下测试滚轮事件行为,确保样式提示被正确遵循
  4. 确认组合框的正常交互(选择、弹出显示)无回归
  5. 验证补丁在打包构建过程中能正确应用

PMS: BUG-339039

1. Add fix-ComboBox-Wheelevent.patch to debian/patches/series
2. Update debian/changelog for version 6.8.0+dfsg-0deepin36
3. The patch allows QComboBox to propagate wheel events to parent scroll
widgets when the style hint disables wheel scrolling
4. This restores expected scrolling behavior in embedded or container-
based UI layouts

Influence:
1. Test QComboBox inside a scrollable container (e.g. QScrollArea) and
verify wheel events scroll the parent when combo box wheel scrolling is
disabled via style hint
2. Verify that QComboBox still scrolls its items when wheel scrolling
is enabled
3. Test wheel event behavior across different styles/platforms to ensure
the style hint is respected
4. Confirm no regression in normal combo box interactions (selection,
popup display)
5. Verify the patch applies cleanly during package build

fix: 启用 QComboBox 滚轮事件补丁

1. 将 fix-ComboBox-Wheelevent.patch 添加到 debian/patches/series
2. 更新 debian/changelog 至版本 6.8.0+dfsg-0deepin36
3. 该补丁使 QComboBox 在样式提示禁用滚轮滚动时将滚轮事件传递给父级滚动
控件
4. 恢复嵌入式或容器化 UI 布局中的预期滚动行为

Influence:
1. 测试位于可滚动容器(如 QScrollArea)中的 QComboBox,验证在样式提示禁
用组合框滚轮滚动时滚轮事件能滚动父级
2. 验证启用滚轮滚动时 QComboBox 仍能滚动其选项
3. 在不同样式/平台下测试滚轮事件行为,确保样式提示被正确遵循
4. 确认组合框的正常交互(选择、弹出显示)无回归
5. 验证补丁在打包构建过程中能正确应用

PMS: BUG-339039
@github-actions

Copy link
Copy Markdown

TAG Bot

TAG: 6.8.0+dfsg-0deepin36
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot

Copy link
Copy Markdown
Contributor

Hi @wjyrich. Thanks for your PR.

I'm waiting for a deepin-community member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@deepin-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind
Once this PR has been reviewed and has the lgtm label, please assign utsweetyfish for approval. For more information see the Code Review Process.

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

@lzwind
lzwind merged commit 310a9a2 into deepin-community:master Sep 21, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants