[VDG] UI Decoupling #38 #11478
[VDG] UI Decoupling #38 #11478
Conversation
| .CombineLatest(_wallet.Privacy.IsWalletPrivate) | ||
| .CombineLatest(_wallet.Coins.List.ToCollection()) | ||
| .Flatten() | ||
| .Throttle(TimeSpan.FromMilliseconds(250)) |
There was a problem hiding this comment.
I can see without the throttle it fires many many times... but I don't understand why. Can one of you explain? @ichthus1604 @SuperJMN
There was a problem hiding this comment.
I also see the tile is changing right after the wallet got logged in, probably caused by the throttle.
Can we avoid it somehow?
There was a problem hiding this comment.
@soosr the Sample() in WalletModel.TransactionProcessed was causing that. I've removed it in the latest commit. I don't think it makes any practical sense to keep it.
There was a problem hiding this comment.
It was added to UiTriggers here #10245, because the continous triggering caused UI freeze #10244.
If you remove it from WalletModel, it's fine because History still relies on UiTriggers.
Keep this in mind when you reach History with refactoring. #10245 (review)
Decouples
PrivacyTileControlViewModel