Skip to content

fix: sync customize keymap when reverting invalid shortcut#435

Merged
pengfeixx merged 1 commit intolinuxdeepin:masterfrom
pengfeixx:fix-353289
Apr 13, 2026
Merged

fix: sync customize keymap when reverting invalid shortcut#435
pengfeixx merged 1 commit intolinuxdeepin:masterfrom
pengfeixx:fix-353289

Conversation

@pengfeixx
Copy link
Copy Markdown
Contributor

When a shortcut is set to an invalid key (e.g. Ctrl+Shift+L), the revert uses the visible option's current value and syncs it to the hidden customize keymap, preventing stale data from overwriting on keymap switch.

修复快捷键设为无效组合键后恢复值不正确的问题,同步更新customize
keymap避免后续keymap切换时用脏数据覆盖。

Log: 修复无效快捷键恢复时使用了旧值的问题
PMS: BUG-353289
Influence: 用户设置无效快捷键后,快捷键将正确恢复为当前有效值而非之前的自定义值。

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 @pengfeixx, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

When a shortcut is set to an invalid key (e.g. Ctrl+Shift+L), the revert
uses the visible option's current value and syncs it to the hidden
customize keymap, preventing stale data from overwriting on keymap switch.

修复快捷键设为无效组合键后恢复值不正确的问题,同步更新customize
keymap避免后续keymap切换时用脏数据覆盖。

Log: 修复无效快捷键恢复时使用了旧值的问题
PMS: BUG-353289
Influence: 用户设置无效快捷键后,快捷键将正确恢复为当前有效值而非之前的自定义值。
@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

@pengfeixx pengfeixx merged commit 303ace6 into linuxdeepin:master Apr 13, 2026
21 of 22 checks passed
@pengfeixx pengfeixx deleted the fix-353289 branch April 13, 2026 09:53
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