Show image tags even if image loading is disabled

Author: Isira-Seneviratne

app/src/main/java/org/schabi/newpipe/ui/components/channel/AboutChannelSection.kt

@@ -16,10 +16,10 @@ import org.schabi.newpipe.R
 import org.schabi.newpipe.extractor.channel.ChannelInfo
 import org.schabi.newpipe.ui.components.metadata.ImageMetadataItem
 import org.schabi.newpipe.ui.components.metadata.MetadataItem
+import org.schabi.newpipe.ui.components.metadata.TagsSection
 import org.schabi.newpipe.ui.theme.AppTheme
 import org.schabi.newpipe.util.Localization
 import org.schabi.newpipe.util.NO_SERVICE_ID
-import org.schabi.newpipe.util.image.ImageStrategy
 
 @Composable
 fun AboutChannelSection(channelInfo: ChannelInfo) {
@@ -41,12 +41,12 @@ fun AboutChannelSection(channelInfo: ChannelInfo) {
             )
         }
 
-        ImageStrategy.choosePreferredImage(channelInfo.avatars)?.let {
-            ImageMetadataItem(R.string.metadata_avatars, channelInfo.avatars, it)
+        if (channelInfo.avatars.isNotEmpty()) {
+            ImageMetadataItem(R.string.metadata_avatars, channelInfo.avatars)
         }
 
-        ImageStrategy.choosePreferredImage(channelInfo.banners)?.let {
-            ImageMetadataItem(R.string.metadata_banners, channelInfo.banners, it)
+        if (channelInfo.banners.isNotEmpty()) {
+            ImageMetadataItem(R.string.metadata_banners, channelInfo.banners)
         }
 
         if (channelInfo.tags.isNotEmpty()) {

app/src/main/java/org/schabi/newpipe/ui/components/metadata/ImageMetadataItem.kt

@@ -26,22 +26,16 @@ import org.schabi.newpipe.util.image.ImageStrategy
 import org.schabi.newpipe.util.image.PreferredImageQuality
 
 @Composable
-fun ImageMetadataItem(
-    @StringRes title: Int,
-    images: List<Image>,
-    preferredUrl: String? = ImageStrategy.choosePreferredImage(images)
-) {
+fun ImageMetadataItem(@StringRes title: Int, images: List<Image>) {
     val context = LocalContext.current
-    val imageLinks = remember { convertImagesToLinks(context, images, preferredUrl) }
+    val imageLinks = remember(images) { convertImagesToLinks(context, images) }
 
     MetadataItem(title = title, value = imageLinks)
 }
 
-private fun convertImagesToLinks(
-    context: Context,
-    images: List<Image>,
-    preferredUrl: String?
-): AnnotatedString {
+private fun convertImagesToLinks(context: Context, images: List<Image>): AnnotatedString {
+    val preferredUrl = ImageStrategy.choosePreferredImage(images)
+
     fun imageSizeToText(size: Int): String {
         return if (size == Image.HEIGHT_UNKNOWN) context.getString(R.string.question_mark)
         else size.toString()

app/src/main/java/org/schabi/newpipe/ui/components/video/VideoDescriptionSection.kt

@@ -58,7 +58,6 @@ import org.schabi.newpipe.ui.components.metadata.TagsSection
 import org.schabi.newpipe.ui.theme.AppTheme
 import org.schabi.newpipe.util.Localization
 import org.schabi.newpipe.util.NO_SERVICE_ID
-import org.schabi.newpipe.util.image.ImageStrategy
 import java.time.OffsetDateTime
 
 @OptIn(ExperimentalMaterial3Api::class)
@@ -214,9 +213,9 @@ private fun LazyListScope.metadataItem(@StringRes title: Int, value: String) {
 }
 
 private fun LazyListScope.imageMetadataItem(@StringRes title: Int, images: List<Image>) {
-    ImageStrategy.choosePreferredImage(images)?.let {
+    if (images.isNotEmpty()) {
         item {
-            ImageMetadataItem(title, images, it)
+            ImageMetadataItem(title, images)
         }
     }
 }