Use singleton ImageLoader

wordpress-mobile · Oct 7, 2024 · ae3db67 · ae3db67
1 parent 8122d32
commit ae3db67
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/WordPress/src/main/java/org/wordpress/android/WordPress.kt b/WordPress/src/main/java/org/wordpress/android/WordPress.kt
@@ -4,6 +4,7 @@ import android.app.Application
 import android.content.Context
 import coil.ImageLoader
 import coil.ImageLoaderFactory
+import coil.decode.VideoFrameDecoder
 import com.android.volley.RequestQueue
 import dagger.hilt.EntryPoints
 import org.wordpress.android.fluxc.tools.FluxCImageLoader
@@ -25,6 +26,9 @@ abstract class WordPress : Application(), ImageLoaderFactory {
     override fun newImageLoader(): ImageLoader {
         return ImageLoader.Builder(this)
             .crossfade(true)
+            .components {
+                add(VideoFrameDecoder.Factory())
+            }
             .build()
     }
 

diff --git a/WordPress/src/main/java/org/wordpress/android/ui/compose/components/MediaUriPager.kt b/WordPress/src/main/java/org/wordpress/android/ui/compose/components/MediaUriPager.kt
@@ -30,9 +30,8 @@ import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.res.vectorResource
 import androidx.compose.ui.tooling.preview.Preview
 import androidx.compose.ui.unit.dp
-import coil.ImageLoader
 import coil.compose.AsyncImage
-import coil.decode.VideoFrameDecoder
+import coil.imageLoader
 import coil.request.ImageRequest
 import org.wordpress.android.R
 
@@ -81,18 +80,13 @@ private fun MediaUriImage(uri: Uri) {
     val context = LocalContext.current
     val mimeType = context.contentResolver.getType(uri)
     if (mimeType?.startsWith("video/") == true) {
-        val imageLoader = ImageLoader.Builder(LocalContext.current)
-            .components {
-                add(VideoFrameDecoder.Factory())
-            }
-            .build()
         Box {
             AsyncImage(
                 model = ImageRequest.Builder(context)
                     .data(uri)
                     .crossfade(true)
                     .build(),
-                imageLoader = imageLoader,
+                imageLoader = context.imageLoader,
                 contentScale = ContentScale.Crop,
                 contentDescription = null,
                 modifier = Modifier