ant-design / ant-design-web3

🥳 Efficient react components for building dapps easier | Connect crypto wallets and more Web3 UI components | Web3 icons | Supports Ethereum, Solana, Bitcoin and others.

Home Page:https://web3.ant.design

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[Bug] WalletConnect showQrModal 为 true 时,出现两个弹窗

jeasonstudio opened this issue · comments

What happens?

当 showQrModal 配置不为 false 时,会使用 WallectConnect 自带的 web3modal 弹窗,但是我们自己的弹窗不会消失。

screenshot-2ZczslGv@2x

How To Reproduce

https://web3.ant.design/components/ethereum-cn#%E4%BD%BF%E7%94%A8-web3modal-%E8%BF%9E%E6%8E%A5-walletconnect

我无法从createConfig 配置出来的 wagmi config 里面拿到东西判断 showQrModal 的配置,有点坑,我提一个 pr 上来用 dom 上面的 className 判断,看看大家有没有更好的。

我无法从createConfig 配置出来的 wagmi config 里面拿到东西判断 showQrModal 的配置,有点坑,我提一个 pr 上来用 dom 上面的 className 判断,看看大家有没有更好的。

不需要这样判断,有一个属性 useWalletConnectOfficialModal ,用户手动指定就行,只是现在这个属性没生效。

感觉 #919 这个 issue 可以推进下了,不直接暴露 wagmi 的配置给开发者,这样就不会有这样的问题了,现在开发者需要同时维护 wagmi 和 web3 的配置,两边配置需要匹配,确实容易出错。

感觉 #919 这个 issue 可以推进下了,不直接暴露 wagmi 的配置给开发者,这样就不会有这样的问题了,现在开发者需要同时维护 wagmi 和 web3 的配置,两边配置需要匹配,确实容易出错。

正在做重构