fix: update default shortcut settings - #139
Conversation
Reviewer's GuideUpdates default window/workspace shortcut configurations and refreshes translation source catalogs to replace the old 'Cancel Maximize Window' action with a new 'Restore window' action and to align TS metadata with Qt expectations. Sequence diagram for updated Restore_window shortcut actionsequenceDiagram
actor User
participant ShortcutConfig as Shortcut_config_JSON
participant DConfig as DConfig_service
participant ShortcutEngine as Shortcut_engine
participant WindowManager as Window_manager
User->>ShortcutConfig: press Restore_window shortcut
ShortcutConfig->>DConfig: lookup keybinding Restore_window
DConfig-->>ShortcutEngine: return QKeySequence
ShortcutEngine->>WindowManager: trigger Restore_window
WindowManager-->>User: window restored from maximized state
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Align the built-in shortcut defaults and presentation with the revised control center layout. - Update workspace navigation and window-move key combinations with matching DConfig serial increments. - Move task switching shortcuts to the Window category and multitask view to the top of Workspace. - Rename the restore-window source label and refresh all dde-app translation catalogs without adding new localized text. 按照调整后的控制中心布局更新内置快捷键默认值和展示信息。 - 更新工作区切换及窗口移动组合键,并同步递增对应的 DConfig serial。 - 将窗口切换快捷键移至窗口分类,并将多任务视图置于工作区分类首位。 - 更新恢复窗口的源文案并刷新全部 dde-app 翻译目录,暂不新增具体翻译文本。 Log: update default shortcut settings Pms: TASK-393897 Change-Id: I23b0db51f7eb2ef0f7717c011217e1abd18894a3
e91de12 to
062a5d5
Compare
deepin pr auto review★ 总体评分:100分■ 【总体评价】
■ 【详细分析】
■ 【改进建议代码示例】 <?xml version="1.0" encoding="utf-8"?>
<!DOCTYPE TS>
<TS version="2.1" language="zh_CN">
<context>
<name>org.deepin.dde.shortcut.dde-app</name>
<message>
<source>Restore window</source>
- <translation type="unfinished"></translation>
+ <translation>恢复窗口</translation>
</message>
</context>
</TS> |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: robertkill, yixinshark The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/forcemerge |
|
This pr force merged! (status: blocked) |
Summary
Translation note
Test plan
cmake --build build --parallel 4ctest --test-dir build --output-on-failure -R shortcut-(physicalkeyalias|triggeractioncatalog|qkeysequenceconverter|x11shortcutpolicy|x11wmaccelconverter)Summary by Sourcery
Update default shortcut configurations and translation catalogs to reflect revised window management and workspace navigation actions.
New Features:
Enhancements:
Chores: