Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

如何销毁已注册的自定义注册菜单功能 #5940

Open
1353723767 opened this issue Aug 26, 2024 · 8 comments
Open

如何销毁已注册的自定义注册菜单功能 #5940

1353723767 opened this issue Aug 26, 2024 · 8 comments

Comments

@1353723767
Copy link

用boot.registerModule注册的自定义菜单ClearAll, 通过editor.destory() 无法销毁。导致关闭编辑器再重新打开编辑器(编辑器是封装了组件引入,并设置了v-if做销毁),提示 ClearAll 重复注册。
第一次打开是没问题的,第二次打开时就报错了

image

image

image

image

@cycleccc
Copy link

是否可以用沙盒做一个最小复现 demo,有最小复现 demo 的话方便找出问题原因。沙盒 demo 在 wangeditor 官网有。

@1353723767
Copy link
Author

是否可以用沙盒做一个最小复现 demo,有最小复现 demo 的话方便找出问题原因。沙盒 demo 在 wangeditor 官网有。

类似这样子,通过触发dialog的显示隐藏,弹层中引入编辑器组件,组件中去注册自定义菜单功能,第一次触发dialog的显示是没问题的,然后关闭之后再触发显示dialog,就会提示自定义菜单重复注册

image

@cycleccc
Copy link

dialog 内没有正确 destory,具体原因得复现再看。以前有人反馈过,问了后没有后续了。

@1353723767
Copy link
Author

dialog 内没有正确 destory,具体原因得复现再看。以前有人反馈过,问了后没有后续了。

好的,那你有空了可以试试看,然后可以看我最早发的截图中的第三张截图,编辑器是有调用destory()的,但第二次显示dialog时,就会报错,就很奇怪

@xb1357686
Copy link

xb1357686 commented Sep 9, 2024

同问,react useEffect中 destroy 调用了,再次打开后 提示组件key 已存在

@cycleccc
Copy link

cycleccc commented Sep 9, 2024

@xb1357686 能做个最小复现吗,有这个比较方便看。

@xb1357686
Copy link

xb1357686 commented Sep 9, 2024

https://stackblitz.com/edit/react-wzavmy?file=demo.tsx,package.json,edit.tsx,myMemu.ts
@cycleccc 大概是这样的 点击按钮切换就会报错,我看了些其他的issues,也是说需要全局注册,但是我需要传入动态的参数或者方法到自定义事件中,
顺便问一下,全屏是否可以改写它的方法?因为一些特殊原因,全屏跑不出父容器,我需要用父容器调用浏览器的全屏
非常感谢!!

@cycleccc
Copy link

@xb1357686 针对第一个,给 key 加个 random 后缀吧,每次创建换一个 key,wangeditor destroy 不会去销毁这个 集合
https://github.com/cycleccc/wangEditor-next/blob/66388295c83a4bba7712e4c35790185a58b401e8/packages/core/src/menus/register.ts#L29

针对第二个 wangeditor 有 fullscreen 方法,你可以试试调用这个设置全屏
https://cycleccc.github.io/docs/guide/API#fullscreen

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

No branches or pull requests

3 participants