Skip to content

fix: show touchscreen dialog on init#439

Merged
fly602 merged 1 commit into
linuxdeepin:masterfrom
fly602:master
Jul 10, 2026
Merged

fix: show touchscreen dialog on init#439
fly602 merged 1 commit into
linuxdeepin:masterfrom
fly602:master

Conversation

@fly602

@fly602 fly602 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor
  1. Remove the check that skips showing the touchscreen selection dialog when the device is already associated with a screen
  2. Pass the touch UUID directly to TouchscreenSetting instead of pre- checking the touch map
  3. Remove unused SYSTYPE variable and related build configuration in debian/rules
  4. Remove pbis-open dependency generation for Professional amd64 builds

Influence:

  1. Verify the touchscreen selection dialog appears during initialization even when the touchscreen is already associated
  2. Test that the dialog correctly displays when launching with a touch device UUID argument
  3. Verify the build succeeds without the removed SYSTYPE and pbis-open dependency configurations
  4. Test on both Server and Desktop system types to ensure no regression
  5. Verify that the touch association still works correctly after selecting a screen in the dialog

fix: 修复初始化不弹出触控屏选择框的问题

  1. 移除了当触摸设备已关联屏幕时跳过显示选择对话框的检查逻辑
  2. 直接将触摸 UUID 传递给 TouchscreenSetting,不再预先检查 touch map
  3. 移除 debian/rules 中无用的 SYSTYPE 变量及相关构建配置
  4. 移除 Professional amd64 构建中 pbis-open 依赖的生成逻辑

Influence:

  1. 验证初始化时即使触摸设备已关联屏幕,触控屏选择对话框也能正常弹出
  2. 测试使用触摸设备 UUID 参数启动时对话框能正确显示
  3. 验证移除 SYSTYPE 和 pbis-open 依赖配置后构建正常成功
  4. 在 Server 和 Desktop 系统类型上测试,确保无回归问题
  5. 验证在对话框中选择屏幕后触摸关联功能仍能正常工作

PMS: BUG-369775

1. Remove the check that skips showing the touchscreen selection dialog
when the device is already associated with a screen
2. Pass the touch UUID directly to TouchscreenSetting instead of pre-
checking the touch map
3. Remove unused SYSTYPE variable and related build configuration in
debian/rules
4. Remove pbis-open dependency generation for Professional amd64 builds

Influence:
1. Verify the touchscreen selection dialog appears during initialization
even when the touchscreen is already associated
2. Test that the dialog correctly displays when launching with a touch
device UUID argument
3. Verify the build succeeds without the removed SYSTYPE and pbis-open
dependency configurations
4. Test on both Server and Desktop system types to ensure no regression
5. Verify that the touch association still works correctly after
selecting a screen in the dialog

fix: 修复初始化不弹出触控屏选择框的问题

1. 移除了当触摸设备已关联屏幕时跳过显示选择对话框的检查逻辑
2. 直接将触摸 UUID 传递给 TouchscreenSetting,不再预先检查 touch map
3. 移除 debian/rules 中无用的 SYSTYPE 变量及相关构建配置
4. 移除 Professional amd64 构建中 pbis-open 依赖的生成逻辑

Influence:
1. 验证初始化时即使触摸设备已关联屏幕,触控屏选择对话框也能正常弹出
2. 测试使用触摸设备 UUID 参数启动时对话框能正确显示
3. 验证移除 SYSTYPE 和 pbis-open 依赖配置后构建正常成功
4. 在 Server 和 Desktop 系统类型上测试,确保无回归问题
5. 验证在对话框中选择屏幕后触摸关联功能仍能正常工作

PMS: BUG-369775

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @fly602, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@fly602
fly602 requested a review from deepin-ci-robot July 10, 2026 03:14
@fly602

fly602 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

/test all

@fly602
fly602 merged commit 36fcd27 into linuxdeepin:master Jul 10, 2026
16 checks passed
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.

2 participants