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
Transaction row details and layout improvements #175
Conversation
c5380e0
to
655c9f0
Compare
Restricting export to not more than 12 months
Fix last year CSV export
Reorganize footer, add version number
c2a0382
to
c3436c2
Compare
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.
ok, respect, I love this
Have you thought of using the ellipsis component that we have in the desktop wallet, where the ...
is added in the middle? Have a look at HashEllipsed
. (in branch token-list
)
Some missing padding:
Regarding the "?" stuff, I would highly recommend you check out my proposals in the desktop wallet PR, I think they are nice! Looking forward for your input!
fa33256
to
7ae9798
Compare
7ae9798
to
a550fe6
Compare
Everything has been addressed. Ready for the final review :) |
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.
Nice job! 👏
Is this effect intentional? My guess is that it's not.
Kapture.2023-03-20.at.08.34.58.mp4
Kapture.2023-03-20.at.08.36.58.mp4
address: 1EckT33Znpgimrwo4XbCSyybuUmGZSAc8cE27BUe6uQTC
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.
Good to merge after the above issue is fixed, unless you want me to do another test afterwards
The issue is fixed in the follow-up PR! Thanks for your review, merging. |
No description provided.