Merge pull request #277 from SimonHalvdansson/codex/migrate-image-loading-from-picasso-to-coil

Replace Picasso with Coil

Author: SimonHalvdansson

app/build.gradle

@@ -66,6 +66,6 @@ dependencies {
     implementation(libs.volley)
     implementation(libs.bundles.okhttp)
     implementation(libs.shimmer)
-    implementation(libs.picasso)
+    implementation(libs.coil)
     implementation(libs.jsoup)
 }

app/src/main/java/com/simon/harmonichackernews/adapters/CommentsRecyclerViewAdapter.java

@@ -51,7 +51,8 @@
 import com.simon.harmonichackernews.utils.FontUtils;
 import com.simon.harmonichackernews.utils.ThemeUtils;
 import com.simon.harmonichackernews.utils.Utils;
-import com.squareup.picasso.Picasso;
+import coil.Coil;
+import coil.request.ImageRequest;
 
 import org.jetbrains.annotations.NotNull;
 import org.sufficientlysecure.htmltextview.HtmlTextView;
@@ -306,9 +307,11 @@ public void onClick(View v) {
                 if (story.nitterInfo.imgSrc != null) {
                     headerViewHolder.nitterImage.setVisibility(VISIBLE);
                     try {
-                        Picasso.get()
-                                .load(story.nitterInfo.imgSrc)
-                                .into(headerViewHolder.nitterImage);
+                        ImageRequest request = new ImageRequest.Builder(ctx)
+                                .data(story.nitterInfo.imgSrc)
+                                .target(headerViewHolder.nitterImage)
+                                .build();
+                        Coil.imageLoader(ctx).enqueue(request);
                     } catch (Exception e){
                         e.printStackTrace();
                     };

app/src/main/java/com/simon/harmonichackernews/network/FaviconLoader.java

@@ -7,7 +7,9 @@
 
 import com.simon.harmonichackernews.R;
 import com.simon.harmonichackernews.utils.Utils;
-import com.squareup.picasso.Picasso;
+
+import coil.Coil;
+import coil.request.ImageRequest;
 
 import java.util.Objects;
 
@@ -18,12 +20,14 @@ public static void loadFavicon(String url, ImageView into, Context ctx, String f
             String host = Utils.getDomainName(url);
             int faviconSize = Utils.pxFromDpInt(ctx.getResources(), 17);
 
-            Picasso.get()
-                    .load(getFaviconUrl(host, faviconProvider))
-                    .resize(faviconSize, faviconSize)
-                    .onlyScaleDown()
+            ImageRequest request = new ImageRequest.Builder(ctx)
+                    .data(getFaviconUrl(host, faviconProvider))
+                    .size(faviconSize, faviconSize)
                     .placeholder(Objects.requireNonNull(ContextCompat.getDrawable(ctx, R.drawable.ic_action_web)))
-                    .into(into);
+                    .target(into)
+                    .build();
+
+            Coil.imageLoader(ctx).enqueue(request);
         } catch (Exception ignored){};
     }
 

app/src/main/res/layout/activity_about.xml

@@ -118,7 +118,7 @@
         <TextView
             android:paddingTop="3dp"
             android:paddingBottom="3dp"
-            android:text="• Picasso by Square - Apache 2.0"
+            android:text="• Coil by Instacart - Apache 2.0"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"/>
 

gradle/libs.versions.toml

@@ -1,5 +1,5 @@
 [versions]
-agp = "8.13.0"
+agp = "8.13.1"
 kotlin = "2.2.10"
 
 [plugins]
@@ -31,8 +31,8 @@ okhttp-core = "com.squareup.okhttp3:okhttp:5.2.0"
 okhttp-urlconnection = "com.squareup.okhttp3:okhttp-urlconnection:5.2.0"
 
 shimmer = "com.facebook.shimmer:shimmer:0.5.0"
-picasso = "com.squareup.picasso:picasso:2.8"
 jsoup = "org.jsoup:jsoup:1.21.2"
+coil = "io.coil-kt:coil:2.7.0"
 
 [bundles]
 androidx-lifecycle = ["androidx-lifecycle-viewmodel", "androidx-lifecycle-viewmodel-ktx"]