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: add label prop to ConnectButton #379
Conversation
nicoglennon
commented
May 13, 2022
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
site/data/docs/connect-button.mdx
Outdated
@@ -59,6 +66,16 @@ The `ConnectButton` component exposes several props to customize its appearance | |||
|
|||
### Examples | |||
|
|||
#### Label | |||
|
|||
By default your Connect Button will say "Connect Wallet". |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in order to keep the doc copy as minimal as possible, I think we could remove this line since it's already documented in the props table above
site/data/docs/connect-button.mdx
Outdated
|
||
By default your Connect Button will say "Connect Wallet". | ||
|
||
For a custom label, pass your string to the `label` prop. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could we use an assertive tone, for consistency with the rest of the docs?
something along the lines of: Use the label
prop to set a custom ConnectButton
text.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for sure, sorry i know nothing about this kind of stuff