-
Notifications
You must be signed in to change notification settings - Fork 21
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
Refactor/modal #646
Refactor/modal #646
Conversation
- rename ReceiveOnChain to ReceiveAddress to refelct usage
8483587
to
aaa26b2
Compare
Will try to bring this to a presentable state first ;) |
bc58a3c
to
393d693
Compare
@cstenglein still just a draft, but could be applied to "all" modals. No rush, any feedback welcome. |
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.
Thank you for the work on this :) 👍
|
||
return createPortal( |
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.
the transition doesn't really work with a portal. need to google how to make it work
Co-authored-by: Christoph Stenglein <9399034+cstenglein@users.noreply.github.com>
Co-authored-by: Christoph Stenglein <9399034+cstenglein@users.noreply.github.com>
Co-authored-by: Christoph Stenglein <9399034+cstenglein@users.noreply.github.com>
Co-authored-by: Christoph Stenglein <9399034+cstenglein@users.noreply.github.com>
Will rework this. Not happy with it. Should be simpler. |
Learned some things. Will tackle this again after the UI has been updated or during the update. |
Fixes #413
First draft
Only took care of the receive modal for now to test this.
Happy for any feedback.
Screenshots
Open points
master
too