From 1faa301ae2a8b652498c6ffcae429e1ed1cb80a9 Mon Sep 17 00:00:00 2001
From: Caren Chang <carenc@google.com>
Date: Wed, 23 Mar 2022 16:30:24 -0700
Subject: [PATCH] Card UI: Display header image

Bug: 216019530
Change-Id: I98acc9951e5644ba5627baa2dc97ca25265b3c2b
---
 .../nowinandroid/core/ui/NewsResourceCard.kt   | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

diff --git a/core-ui/src/main/java/com/google/samples/apps/nowinandroid/core/ui/NewsResourceCard.kt b/core-ui/src/main/java/com/google/samples/apps/nowinandroid/core/ui/NewsResourceCard.kt
index becdda16d..db0236b59 100644
--- a/core-ui/src/main/java/com/google/samples/apps/nowinandroid/core/ui/NewsResourceCard.kt
+++ b/core-ui/src/main/java/com/google/samples/apps/nowinandroid/core/ui/NewsResourceCard.kt
@@ -68,6 +68,12 @@ fun NewsResourceCardExpanded(
     Column(
         modifier = Modifier.padding(16.dp)
     ) {
+        if (!newsResource.headerImageUrl.isNullOrEmpty()) {
+            Row {
+                NewsResourceHeaderImage(newsResource.headerImageUrl)
+            }
+            Spacer(modifier = Modifier.height(12.dp))
+        }
         Row {
             NewsResourceAuthors(newsResource.authors)
         }
@@ -84,9 +90,17 @@ fun NewsResourceCardExpanded(
 
 @Composable
 fun NewsResourceHeaderImage(
-    newsResource: NewsResource
+    headerImageUrl: String?
 ) {
-    TODO()
+    AsyncImage(
+        modifier = Modifier
+            .fillMaxWidth()
+            .height(180.dp),
+        contentScale = ContentScale.Crop,
+        model = headerImageUrl,
+        // TODO b/226661685: Investigate using alt text of  image to populate content description
+        contentDescription = null // decorative image
+    )
 }
 
 @Composable