fix(mobile): blurry memory photos #6734
Conversation
Deploying with
|
| Latest commit: |
c3a650d
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://62bf68fc.immich.pages.dev |
| Branch Preview URL: | https://mobile-fix-blurry-memory.immich.pages.dev |
| @@ -49,7 +49,7 @@ class ImmichImage extends StatelessWidget { | |||
| final Asset asset = this.asset!; | |||
| if (useLocal(asset)) { | |||
| return Image( | |||
| image: localThumbnailProvider(asset), | |||
| image: localThumbnailProvider(asset, dimension: 2048), | |||
There was a problem hiding this comment.
This should probably fix it. I think I'll make it a point to create a new localImageProvider which doesn't need a dimension, and just uses the full asset. Because using the thumbnail provider to show the full image feels a bit dirty.
|
I can confirm that this fixes the issue! Great work, Alex! I can refactor this later to use a non-thumbnail image provider, or else you can do it in this PR. Just copy / paste the thumbnail image provider and use it in place. Don't specify a In any case, I'll approve this, so merge it or add my suggestion as you like! |
|
Huh, weird. I can't approve the PR. |
|
@martyfuhry @shenlong-tanwen, My previous attempt caused a performance drop while browsing the timeline since it gets a large thumbnail size. So I refactored it to use the previous size 250px by default unless we pass in the |
Fixes #6423