-
Notifications
You must be signed in to change notification settings - Fork 3
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
Replace rebranded superhero tip button to this repo #45
Comments
I didn't get the idea of this. What interface the rebranded button will have? I can't get how it is connected to Vuex. Do you know that the button in the current |
The idea is to have reusable component for any Vue project. As the interface It can use the same state and mapState helpers. Also maybe it can be separated from the Vuex and reused without it. In any case the main idea is to separate some parts and make superhero-utils used by the superhero-ui as a dependency. |
|
There is already one, we did it in #38
The current version of tip button just opens a deeplink to the wallet, don't see reasons to connect it to the store somehow.
This is a branch in another repository. |
I know, but it's rebranded one and the idea is to use the same for the superhero-ui and any other project.
Why? It can be useful for others. Helps make a nice tip button with the extension or iframe wallet. The current tip button version is works only with "deeplink".
Right, that's what I plan to solve – make it works with sdk also. the same sdk that uses the project that want to integrate the tip button.
Yes, but you asked why this idea comes to me. That's why – I start to rebranding the button for the |
They look different. Making buttons in superhero-utils to look like in superhero-ui is a breaking change.
SDK is too heavy (about 500kb) for a such use case. Making it optional will make documentation more complicated. I'm against to work on this. |
@davidyuk but it's rebranding now in any case.
I mean make an instruction how to use the component with own SDK instance but not to have the SDK itself at this repo as a dependency. |
Can you post a link to the actual designs of the tip button in superhero-utils?
I don't think the package users need it. |
Move rebranded superhero-ui button to this repo, after merging this branch. It will allow to reuse it in any Vue projects that has Vuex and at the superhero-ui as well. maybe it can be used also without Vuex
The text was updated successfully, but these errors were encountered: