uiwjs / react-amap

基于 React 封装的高德地图组件,帮助你轻松的接入地图到 React 项目中。
https://uiwjs.github.io/react-amap
MIT License
418 stars 69 forks source link

fix(polygon): ignore error when map unmounted #321

Closed mashirozx closed 1 year ago

mashirozx commented 1 year ago

related #320

改成 remove 以后又发现在 antd 的 modal 上使用时,关闭弹窗会报错。

复现:https://github.com/MashiroArchive/react-amap-bug-repoort

codesandbox:https://codesandbox.io/p/github/MashiroArchive/react-amap-bug-repoort/main

可能是 amap sdk 内调用了polygon instance 的内部 remove 方法(这个时候 instance 已经被销毁了),导致的报错:

image

我暂时想不到既能确保polygon 能正常移除,同时又不会报错的方法了,虽然不优雅但 try catch 能解决问题

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-amap ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 12, 2023 10:34am
mashirozx commented 1 year ago

ping @jaywcjlove

jaywcjlove commented 1 year ago

@mashirozx Upgrade v5.0.6

indredK commented 1 year ago

确有其事,切换菜单的时候也会,我还打了补丁来着,现在好了