-
Notifications
You must be signed in to change notification settings - Fork 757
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
Transfer: hide PaymentId and Description by default #1855
Conversation
Looks reasonable to me. A good compromise between slowly phasing out payment ids but still supporting them for the time being. |
I like this feature but I’m unhappy about how “Payment IDs” is now not aligned with the other labels. Maybe it’s possible to add the + to the left of the Payment ID label and align it like it was before. |
We should go even further and say (optional, will be deprecated March/April 2019). See Monero issue 3772. Of course, this is contingent on agreeing to remove them in the next upgrade. |
9644ae5
to
8489763
Compare
Updated |
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.
Looking good now! And thanks for the @2x graphics :)
3fff976
to
e10e85a
Compare
Updated to hide Description field too. |
e10e85a
to
4121232
Compare
I'd remove the |
4121232
to
55c3d07
Compare
Yep. Updated the PR. |
55c3d07 Transfer: hide PaymentId and Description by default (xiphon)
Requires #1853 #1854
Feedback and comments on proposed change are welcome.