-
Notifications
You must be signed in to change notification settings - Fork 76
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
chore[UI - Wallet Stability] Rename SendModalFooter to TransactionModalFooter #15497
chore[UI - Wallet Stability] Rename SendModalFooter to TransactionModalFooter #15497
Conversation
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.
This was a rename... No idea what's going wrong with GitHub
On branch 15212-rename-sendmodalfooter-to-transactionmodalfooter
Your branch is ahead of 'origin/15212-rename-sendmodalfooter-to-transactionmodalfooter' by 1 commit.
(use "git push" to publish your local commits)
Changes to be committed:
(use "git restore --staged <file>..." to unstage)
modified: ui/imports/shared/popups/send/SendModal.qml
renamed: ui/imports/shared/popups/send/views/SendModalFooter.qml -> ui/imports/shared/popups/send/views/TransactionModalFooter.qml
modified: ui/imports/shared/popups/send/views/qmldir
Jenkins Builds
|
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.
LGTM!
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.
Here some comments added!
…alFooter ... and remove particular code for send modal - derive the footer from the standard `StatusDialogFooter` to reduce code duplication (drop shadow, divider, margins, etc) - rename the default button text - expose the icon name and make it used the correct login/auth type icon - some minor fixes and cleanups Fixes #15212
c51ab1e
to
19157ee
Compare
What does the PR do
... and remove particular code for send modal
StatusDialogFooter
to reduce code duplication (drop shadow, divider, margins, etc)Fixes #15212
Affected areas
SendModal
Screenshot of functionality (including design for comparison)