Skip to content

fix: correct mouse/touchpad enable/disable button visibility#640

Merged
wyu71 merged 1 commit intolinuxdeepin:masterfrom
wyu71:master
Apr 10, 2026
Merged

fix: correct mouse/touchpad enable/disable button visibility#640
wyu71 merged 1 commit intolinuxdeepin:masterfrom
wyu71:master

Conversation

@wyu71
Copy link
Copy Markdown
Contributor

@wyu71 wyu71 commented Apr 10, 2026

Move PS/2 mouse canEnable check after getMouseInfoFromBusDevice() to avoid misidentifying bluetooth mice. Add validateCanEnableForMouse() to check sysfs authorized/remove files. Fix v25 touchpad disable by using DBus Enable method. Refactor TableWidget to use per-row canEnable from item data instead of global m_Enable flag.

修复鼠标/触摸板禁用按钮显示逻辑:将PS/2鼠标canEnable检查移到
getMouseInfoFromBusDevice之后避免误判蓝牙鼠标;新增sysfs文件
校验;v25触摸板禁用改用DBus Enable方法;TableWidget改为逐行
读取canEnable而非全局m_Enable标志。

Log: 修复鼠标触摸板禁用按钮显示和触控板禁用功能
PMS: BUG-355975
Influence: 蓝牙鼠标、I2C触控笔等无法禁用的设备不再显示禁用按钮;v25触摸板禁用功能恢复正常

Copy link
Copy Markdown

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

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

Please try again later or upgrade to continue using Sourcery

Move PS/2 mouse canEnable check after getMouseInfoFromBusDevice() to
avoid misidentifying bluetooth mice. Add validateCanEnableForMouse()
to check sysfs authorized/remove files. Fix v25 touchpad disable by
using DBus Enable method. Refactor TableWidget to use per-row
canEnable from item data instead of global m_Enable flag.

修复鼠标/触摸板禁用按钮显示逻辑:将PS/2鼠标canEnable检查移到
getMouseInfoFromBusDevice之后避免误判蓝牙鼠标;新增sysfs文件
校验;v25触摸板禁用改用DBus Enable方法;TableWidget改为逐行
读取canEnable而非全局m_Enable标志。

Log: 修复鼠标触摸板禁用按钮显示和触控板禁用功能
PMS: BUG-355975
Influence: 蓝牙鼠标、I2C触控笔等无法禁用的设备不再显示禁用按钮;v25触摸板禁用功能恢复正常
@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind, wyu71

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

@wyu71 wyu71 merged commit 8e86c3e into linuxdeepin:master Apr 10, 2026
20 of 22 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.

3 participants