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
[VDG] [Fluent] Allow Coinjoins to be expanded in the TreeDataGrid #7510
[VDG] [Fluent] Allow Coinjoins to be expanded in the TreeDataGrid #7510
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.
tested 4f1bdf6
expand and collaps works well. cj tx details works.
When expanded, the included coinjoin has a tx icon, not the shield icon.
awesome work, this is really promissing!
Tested. This looks good. One thing I noticed that can maybe improve is that if you have only one coinjoin tx then there is no need to expand it. |
Will look into that. Should be possibile. |
@MaxHillebrand fixed icon |
@yahiheb added check and now only if more then one transaction is present expander is visible |
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.
tACK
...etWasabi.Fluent/ViewModels/Wallets/Home/History/HistoryItems/CoinJoinHistoryItemViewModel.cs
Outdated
Show resolved
Hide resolved
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.
tACK 743cc71
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.
tACK
IObservable<Unit> updateTrigger) | ||
: base(orderIndex, transactionSummary) | ||
{ | ||
Label = transactionSummary.Label.Take(1).FirstOrDefault(); |
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.
We might not need this line if #7604 gets accepted.
8630539
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.
nit
WalletWasabi.Fluent/ViewModels/Wallets/Home/History/HistoryItems/HistoryItemViewModelBase.cs
Outdated
Show resolved
Hide resolved
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.
tACK 559bae6
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
|
No description provided.