Add new text for connect wallet button tooltip
#1450
Conversation
| const label = isAppPage ? | ||
| <span>Connect once to use your wallet with<br/>all apps in the DAppscout marketplace!</span> : | ||
| defaultLabel; | ||
| const localStorageKey = `${ isAppPage ? 'dapp-' : '' }wallet-connect-tooltip-shown-${ isMobile ? 'mobile' : 'desktop' }`; |
There was a problem hiding this comment.
I wonder why there are separate keys for mobile and desktop devices. It seems unnecessary since user sessions and data cannot be shared between the two.
There was a problem hiding this comment.
The problem is that both components are rendered at the same time, and if you use 1 key, only the tooltip for one of the device types will be displayed. I tried using Show/Hide components, but the result was even worse, because the button itself appeared with a delay. If you have an idea, I'm ready to redo it, because I don't really like this solution myself
There was a problem hiding this comment.
Oh, I see now. Then, maybe the parent component should pass isDisabled=true prop if viewport size doesn't match. Will it work?
Resolves #1431