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

feat: Refactor buttonProps & update document about ConnectButton #59

Closed
wants to merge 3 commits into from

Conversation

kiner-tang
Copy link
Member

@kiner-tang kiner-tang commented Oct 14, 2023

image
  • Update document about ConnectButton

@github-actions
Copy link

github-actions bot commented Oct 14, 2023

@yutingzhao1991
Copy link
Collaborator

我觉得不需要, 这个组件本来就叫 ConnectButton,上面再有一个 buttonProps 太奇怪了。倒是可以有一个 modalProps

冲突的问题感觉概率很小,即便是出现也可以换一个别名。

@alexsupa597
Copy link

我觉得不需要, 这个组件本来就叫 ConnectButton,上面再有一个 buttonProps 太奇怪了。倒是可以有一个 modalProps

冲突的问题感觉概率很小,即便是出现也可以换一个别名。

这样的维护成本会不会很高?

@yutingzhao1991
Copy link
Collaborator

我觉得不需要, 这个组件本来就叫 ConnectButton,上面再有一个 buttonProps 太奇怪了。倒是可以有一个 modalProps
冲突的问题感觉概率很小,即便是出现也可以换一个别名。

这样的维护成本会不会很高?

维护什么的成本?props 出现冲突的概率我觉得还是比较小的,只要现在不冲突,antd 的 Button 我理解应该不会有太多新增 props 了。我们本身命名 ConnectButton 的时候也考虑下语义,甚至如果是 Button 的熟悉都可以提 pr 到 antd Button 修改,这样其实不太会冲突的。

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

Successfully merging this pull request may close these issues.

3 participants