diff options
author | Sergei Kozelko <KozelkoS@yandex.ru> | 2022-11-03 01:47:10 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-11-03 01:47:10 +0000 |
commit | 78496e080f362def8265221301ab36a79947dd34 (patch) | |
tree | 6f51d700fca863cf27e25049093ac849fcf48a0e /app/src/main/res/layout/fragment_font_preview.xml | |
parent | 191df66a3bae86bab0ca8cb31cef0e005900d31d (diff) | |
download | infinity-for-reddit-78496e080f362def8265221301ab36a79947dd34.tar infinity-for-reddit-78496e080f362def8265221301ab36a79947dd34.tar.gz infinity-for-reddit-78496e080f362def8265221301ab36a79947dd34.tar.bz2 infinity-for-reddit-78496e080f362def8265221301ab36a79947dd34.tar.lz infinity-for-reddit-78496e080f362def8265221301ab36a79947dd34.tar.xz infinity-for-reddit-78496e080f362def8265221301ab36a79947dd34.tar.zst infinity-for-reddit-78496e080f362def8265221301ab36a79947dd34.zip |
Display comment avatar only if it is the right user (#1191)
Condition in callback for loading avatar url is almost always true[1]. So it would load avatar even if the viewholder got bound to another comment.
Ideally the solution would be to update the comment just like now, then find current position of the comment and call onItemChanged. However you cannot call onItemChanged from onBindViewHolder and that is a problem because callback can be executed synchronously.
So instead we just check that viewholder is bound to some comment and that bound comment's author matches initially requested author.
[1] The only case I know when it is false is when that comment got deleted and its author got replaced with "[deleted]" before the callback got executed
Diffstat (limited to 'app/src/main/res/layout/fragment_font_preview.xml')
0 files changed, 0 insertions, 0 deletions