search

opensumi / core

🚀 A framework helps you quickly build AI Native IDE products. https://preview.opensumi.com
https://opensumi.com
MIT License
2.85k stars 369 forks source link

[BUG] Keyboard shortcut 快捷键面板中快捷键设置无效 #3820

Open chcarmen opened 2 months ago

chcarmen commented 2 months ago

Keyboard shortcut 快捷键设置无效

image

复现路径（To Reproduce

  1. 如上，修改任意快捷键
  2. IDE 重启后，配置失效

预期表现（Expected behavior IDE 重启后，自定义配置仍保留

环境信息（Environment）

erha19 commented 2 months ago

@chcarmen 修改完的配置文件会写入 ~/.sumi/keymaps.json 内，检查一下写入是否正常，重启后是否存在修改内容

chcarmen commented 2 months ago 
[
  {
    "when": "dialogViewVisible",
    "command": "dialog.ensure",
    "key": "O"
  }
]

这是我的 keymaps.json, 有写入我更新的快捷键。但是重启 IDE 后，还是默认的。

截屏2024-07-03 上午11 40 09
erha19 commented 2 months ago

@chcarmen 没有复现你的问题，是否能在 ide-electron 项目中复现该问题？

chcarmen commented 2 months ago

就是 ide-electron , 基于框架 3.1.3

chensimeng commented 2 months ago

@chcarmen 没有复现你的问题，是否能在 ide-electron 项目中复现该问题？

下载最新的 ide-electron 项目，能浮现快捷键保存失败的问题！

chensimeng commented 2 months ago

D3638F22-5911-4BD2-B141-EC96036E9FFA 更新快捷键cursorUp为command+K后，不紧没有修改成功，还会自动多一个cursorUp的项出来，如上图

erha19 commented 2 months ago

@chensimeng 设置及保存快捷键的相关逻辑在这里 https://github.com/opensumi/core/blob/5a6654240f74eaecb29e9d1bbea17430c1c2f0b8/packages/keymaps/src/browser/keymaps.service.ts#L329

恢复快捷键的逻辑在 https://github.com/opensumi/core/blob/5a6654240f74eaecb29e9d1bbea17430c1c2f0b8/packages/keymaps/src/browser/keymaps.service.ts#L183

可以尝试调试定位一下，暂时没时间排查这个问题