-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[FIX] Spaces' thumbnails not loaded the first time #4084
[FIX] Spaces' thumbnails not loaded the first time #4084
Conversation
39cb720
to
c96d7f1
Compare
efeef44
to
da945ca
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.
Some comments here @manuelplazaspalacio! Nice job! 👍
owncloudApp/src/main/java/com/owncloud/android/presentation/files/filelist/FileListAdapter.kt
Outdated
Show resolved
Hide resolved
owncloudApp/src/main/java/com/owncloud/android/presentation/files/filelist/FileListAdapter.kt
Show resolved
Hide resolved
...udApp/src/main/java/com/owncloud/android/presentation/files/filelist/MainFileListFragment.kt
Outdated
Show resolved
Hide resolved
owncloudApp/src/main/java/com/owncloud/android/presentation/thumbnails/ThumbnailsRequester.kt
Show resolved
Hide resolved
9b10d01
to
4beaef6
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.
Good job @manuelplazaspalacio! Let's move this to QA 🚀
(1) [FIXED]thumbnails' shape has changed, is this intended @manuelplazaspalacio?
|
(2) [FIXED]Check the following steps out:
Current: Thumbnail not updated. If the image would be downloaded, thumbnail would have updated. Pixel 2 Android 11 |
631d893
to
86ad085
Compare
- Solving loading images errors.
- Adding name to modified files
Solving lint errors
5c6e03d
to
885ed79
Compare
f271deb
to
f31526a
Compare
This is approved on my side. Work to do: apply changes to uploads view to make these changes visible there as well. |
Related Issues
App: #3959
Spaces
Files list
Library PR (if needed):
QA
Reports and suggestions