search

opensumi / core

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

[BUG] registerMenuItem 注册 ComponentMenuItem 类型的 item 时，对非`EditorTitle`的场景不生效 #2571

Open fanweiren1217 opened 1 year ago

fanweiren1217 commented 1 year ago

描述你的问题（Describe the bug） 期望将底部栏的这个菜单里的图标都换成自定义类型的，但是调用registerMenuItem注册不生效，发现下图代码中做了过滤 image

erha19 commented 1 year ago

@vagusX 这里的代码有啥实现背景吗？是不是可以开放到所有菜单注册上面？

vagusX commented 1 year ago

是可以考虑开放的，当时一开始只是想拿 EditorTitle 做试点开放

另外开放之后，因为是 ComponentMenuItem 应该是自定义组件类型，开放后，定制行为需要统一，容易有些高有些矮，协调性由定制者来把握哈~

erha19 commented 1 year ago

@fanweiren1217 可以提个实验性的 PR 上来，试试

fanweiren1217 commented 1 year ago

@erha19 在本地测试了下，放开后自定义组件未渲染

  1. 开始 https://github.com/opensumi/core/blob/main/packages/main-layout/src/browser/tabbar/panel.view.tsx image
  2. 进入 https://github.com/opensumi/core/blob/main/packages/core-browser/src/components/actions/index.tsx image
  3. 发现调用useContextMenus后返回两个空数组 image
erha19 commented 1 year ago

@fanweiren1217 相关代码可以提交上来看一下，可能写法上有些问题

fanweiren1217 commented 1 year ago

@erha19 提了一个 pr 哈，可以参考一下 https://github.com/opensumi/core/pull/2595