opensumi / core

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

[BUG] F2 快捷键「重命名符号」的 Tips 提示文案为 undefined #3089

Closed zmyiy closed 8 months ago

zmyiy commented 1 year ago

描述你的问题(Describe the bug)

image

复现路径(To Reproduce) 右键,点击 「重命名符号 F2」菜单选项

预期表现(Expected behavior) image

环境信息(Environment)

erha19 commented 1 year ago

@zmyiy 托管 monaco 默认快捷键的逻辑见:https://github.com/opensumi/core/blob/96477a517f05e47209e42aa3d53444baa4117b63/packages/monaco/src/browser/monaco.resolved-keybinding.ts

这里也可能是快捷键没注册,也有可能是 command 没注册

zmyiy commented 1 year ago

@erha19 有什么好的思路能给这个快捷键加回去吗? Debug 看了下,调用的地方 image 注册的地方实际上是把 acceptRenameInput 这个快捷键给过滤掉了,内置 defaultKeybindings 是有这个快捷键的。 image

erha19 commented 1 year ago

@zmyiy 这里确实是框架通过声明 -{command} 的方式卸载了的快捷键

image

相关代码见:

https://github.com/opensumi/core/blob/2bf2b3bf06f3398ce3527b3b3cd0439119ccca5a/packages/monaco/src/browser/monaco.contribution.ts#L493-L494

这里可能需要对部分不允许用户修改的快捷键做个白名单处理(即用户就算修改了 acceptRenameInput 命令快捷键,默认的快捷键依旧需要保留),否则后续用户如果在快捷键面板中修改了,同样也会造成问题