From ddb856edc7294ca68ab223ea50358e9323ed9a3f Mon Sep 17 00:00:00 2001
From: Ivan Iskandar <12537387+ivaniskandar@users.noreply.github.com>
Date: Wed, 16 Mar 2022 09:21:30 +0700
Subject: [PATCH] Add cover error drawable (#6782)

---
 .../util/view/ImageViewExtensions.kt          |  7 +++---
 app/src/main/res/drawable/cover_error.xml     | 25 +++++++++++++++++++
 app/src/main/res/values/colors.xml            |  2 ++
 3 files changed, 30 insertions(+), 4 deletions(-)
 create mode 100644 app/src/main/res/drawable/cover_error.xml

diff --git a/app/src/main/java/eu/kanade/tachiyomi/util/view/ImageViewExtensions.kt b/app/src/main/java/eu/kanade/tachiyomi/util/view/ImageViewExtensions.kt
index c64f04575..5df8fcb33 100644
--- a/app/src/main/java/eu/kanade/tachiyomi/util/view/ImageViewExtensions.kt
+++ b/app/src/main/java/eu/kanade/tachiyomi/util/view/ImageViewExtensions.kt
@@ -1,19 +1,18 @@
 package eu.kanade.tachiyomi.util.view
 
 import android.content.Context
-import android.graphics.Color
 import android.graphics.drawable.Animatable
 import android.graphics.drawable.ColorDrawable
 import android.widget.ImageView
 import androidx.annotation.AttrRes
 import androidx.annotation.DrawableRes
 import androidx.appcompat.content.res.AppCompatResources
-import androidx.core.graphics.ColorUtils
 import coil.ImageLoader
 import coil.imageLoader
 import coil.load
 import coil.request.ImageRequest
 import coil.target.ImageViewTarget
+import eu.kanade.tachiyomi.R
 import eu.kanade.tachiyomi.util.system.animatorDurationScale
 import eu.kanade.tachiyomi.util.system.getResourceColor
 
@@ -42,8 +41,8 @@ fun ImageView.loadAutoPause(
     builder: ImageRequest.Builder.() -> Unit = {}
 ) {
     load(data, loader) {
-        val placeholderColor = ColorUtils.setAlphaComponent(Color.GRAY, 0x1F) // 12% gray
-        placeholder(ColorDrawable(placeholderColor))
+        placeholder(ColorDrawable(context.getColor(R.color.cover_placeholder)))
+        error(R.drawable.cover_error)
 
         // Build the original request so we can add on our success listener
         val originalListener = apply(builder).build().listener
diff --git a/app/src/main/res/drawable/cover_error.xml b/app/src/main/res/drawable/cover_error.xml
new file mode 100644
index 000000000..98ca1458d
--- /dev/null
+++ b/app/src/main/res/drawable/cover_error.xml
@@ -0,0 +1,25 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--suppress AndroidElementNotAllowed -->
+<layer-list xmlns:android="http://schemas.android.com/apk/res/android"
+    android:shape="rectangle">
+    <item>
+        <color android:color="@color/cover_placeholder" />
+    </item>
+
+    <item
+        android:top="48dp"
+        android:bottom="48dp"
+        android:left="48dp"
+        android:right="48dp">
+        <vector
+            android:width="24dp"
+            android:height="24dp"
+            android:viewportWidth="24"
+            android:viewportHeight="24">
+            <path
+                android:fillColor="@color/cover_placeholder"
+                android:pathData="M21,5v6.59l-2.29,-2.3c-0.39,-0.39 -1.03,-0.39 -1.42,0L14,12.59 10.71,9.3c-0.39,-0.39 -1.02,-0.39 -1.41,0L6,12.59 3,9.58L3,5c0,-1.1 0.9,-2 2,-2h14c1.1,0 2,0.9 2,2zM18,11.42l3,3.01L21,19c0,1.1 -0.9,2 -2,2L5,21c-1.1,0 -2,-0.9 -2,-2v-6.58l2.29,2.29c0.39,0.39 1.02,0.39 1.41,0l3.3,-3.3 3.29,3.29c0.39,0.39 1.02,0.39 1.41,0l3.3,-3.28z"/>
+        </vector>
+    </item>
+
+</layer-list>
diff --git a/app/src/main/res/values/colors.xml b/app/src/main/res/values/colors.xml
index a106bf35a..9f9f62f19 100644
--- a/app/src/main/res/values/colors.xml
+++ b/app/src/main/res/values/colors.xml
@@ -2,6 +2,8 @@
 <resources xmlns:tools="http://schemas.android.com/tools">
     <color name="splash">@color/accent_blue</color>
 
+    <color name="cover_placeholder">#1F888888</color>
+
     <color name="error">#BA1B1B</color>
     <color name="errorContainer">#FFDAD4</color>
     <color name="onError">#FFFFFF</color>