Skip to content

VSP1566 - Mobile - Changed the thumbnails to 256 size#555

Merged
luciancerbu-vsp merged 3 commits into
Developmentfrom
feature/VSP-1566-Mobile-iOS-Update-mobile-apps-to-handle-new-thumbnail-size
Apr 23, 2026
Merged

VSP1566 - Mobile - Changed the thumbnails to 256 size#555
luciancerbu-vsp merged 3 commits into
Developmentfrom
feature/VSP-1566-Mobile-iOS-Update-mobile-apps-to-handle-new-thumbnail-size

Conversation

@luciancerbu-vsp
Copy link
Copy Markdown
Contributor

No description provided.

Implement SDWebImage two-stage progressive loading (256px thumbnail, then
full-res) with cache configuration (300MB disk, 50MB memory) and cache
clearing on logout.

Fix thumbnail disappearing in share management when V2 API returns empty
strings instead of null. Added nonEmpty() filtering to all
preferredThumbnailURL properties across V1 and V2 models, plus guards
in ShareItemViewModel and FileModel.
Added 9 unit tests for empty string handling.
Copy link
Copy Markdown

@radusabau-tech radusabau-tech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

before merge can we get a product to test to decide if we go with the 256 ?

@luciancerbu-vsp luciancerbu-vsp merged commit 038c063 into Development Apr 23, 2026
3 checks passed
@luciancerbu-vsp luciancerbu-vsp deleted the feature/VSP-1566-Mobile-iOS-Update-mobile-apps-to-handle-new-thumbnail-size branch April 23, 2026 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants