ant-design / pro-components

🏆 Use Ant Design like a Pro!
https://pro-components.antdigital.dev
MIT License
4.3k stars 1.36k forks source link

🐛[BUG]Layout 会自动给 http url 加上跳转事件 #8411

Open Yawenina opened 5 months ago

Yawenina commented 5 months ago

提问前先看看:

https://github.com/ryanhanwu/How-To-Ask-Questions-The-Smart-Way/blob/main/README-zh_CN.md

🐛 bug 描述

问题背景是我们想给 Menu 加 tooltip, 但是 antd 有一个 a::before 的样式覆盖了 a 标签,导致 tooltip 不生效,(antd issue)

为了解决这个问题,我们重设了 a::before 的样式,但是此时如果点击一个 http 链接,则会打开 2 个窗口。

这个原因是,ProLayout 内部会给 http 的链接绑定跳转事件(ProLayout 代码),当我们在 menuItemRender 包装 a 标签时,由于解决上面 tooltip 的问题去掉了蒙层,就会导致一个点击事件,会被 spana 都处理,就会导致打开 2 个窗口。

📷 复现步骤

https://stackblitz.com/edit/vitejs-vite-3bbuza?file=src%2FApp.tsx

🏞 期望结果

不要给 http url 注册 window.open, 交给使用方处理。

💻 复现代码

© 版本信息

🚑 其他信息