feat: add Help Icon to "Setup Wallet Connection" #106
Conversation
- Show an info toast with instructions on how to connect to Alby Hub
| const [isScanning, setIsScanning] = React.useState(false); | ||
| const [isConnecting, setIsConnecting] = React.useState(false); |
There was a problem hiding this comment.
this fixes react warning useState call is not destructured into value + setter pair
There was a problem hiding this comment.
I haven't seen this warning, weird. Do you know what plugin this is coming from?
| import { errorToast } from "~/lib/errorToast"; | ||
|
|
||
| import { nwc } from "@getalby/sdk"; | ||
| import { Nip47Capability } from "@getalby/sdk/dist/NWCClient"; |
There was a problem hiding this comment.
Sorry about these import order changes - I have prettier enabled by default.
Maybe we should add a .prettierrc in a separate PR?
There was a problem hiding this comment.
There already is one, but it's pretty empty
There was a problem hiding this comment.
It shouldn't matter if it's pretty empty - it should use prettier sane defaults. It was added recently though, and we didn't run a full format. We should do that. I will create a separate issue.
| info: (props: BaseToastProps) => ( | ||
| <BaseToast | ||
| {...props} | ||
| text2NumberOfLines={2} | ||
| text1Style={{ fontSize: 16 }} | ||
| text2Style={{ fontSize: 14 }} | ||
| /> | ||
| ), |
There was a problem hiding this comment.
I found the default text size for the info toast was unreadable, so I made it a little bit bigger.
|
Thanks for the PR! Closing in favor like we discussed in #110 🙌 |
Description
Show an info button that triggers a toast with instructions on how to connect to Alby Hub.
Context
Inspired because of #104
Screenshots