From aa78456c8880b16dcc969b1d4962c9041d7ef8a5 Mon Sep 17 00:00:00 2001
From: Noah Bond <nbond@mozilla.com>
Date: Tue, 23 Aug 2022 14:43:34 -0700
Subject: [PATCH] [fenix] For
 https://github.com/mozilla-mobile/fenix/issues/26604 - Add missing preview to
 RecentTabs

---
 .../fenix/home/recenttabs/view/RecentTabs.kt  | 69 ++++++++++++++++---
 1 file changed, 59 insertions(+), 10 deletions(-)

diff --git a/app/src/main/java/org/mozilla/fenix/home/recenttabs/view/RecentTabs.kt b/app/src/main/java/org/mozilla/fenix/home/recenttabs/view/RecentTabs.kt
index 2c56d6b0f2..0167233d1f 100644
--- a/app/src/main/java/org/mozilla/fenix/home/recenttabs/view/RecentTabs.kt
+++ b/app/src/main/java/org/mozilla/fenix/home/recenttabs/view/RecentTabs.kt
@@ -6,6 +6,7 @@
 
 package org.mozilla.fenix.home.recenttabs.view
 
+import android.content.res.Configuration
 import android.graphics.Bitmap
 import androidx.compose.foundation.ExperimentalFoundationApi
 import androidx.compose.foundation.Image
@@ -43,15 +44,19 @@ import androidx.compose.ui.graphics.painter.BitmapPainter
 import androidx.compose.ui.layout.ContentScale
 import androidx.compose.ui.platform.LocalConfiguration
 import androidx.compose.ui.text.style.TextOverflow
+import androidx.compose.ui.tooling.preview.Preview
 import androidx.compose.ui.unit.dp
 import androidx.compose.ui.unit.sp
 import mozilla.components.browser.icons.compose.Loader
 import mozilla.components.browser.icons.compose.Placeholder
 import mozilla.components.browser.icons.compose.WithIcon
+import mozilla.components.browser.state.state.ContentState
+import mozilla.components.browser.state.state.TabSessionState
 import mozilla.components.ui.colors.PhotonColors
 import org.mozilla.fenix.components.components
 import org.mozilla.fenix.compose.Image
 import org.mozilla.fenix.compose.ThumbnailCard
+import org.mozilla.fenix.compose.inComposePreview
 import org.mozilla.fenix.home.recenttabs.RecentTab
 import org.mozilla.fenix.theme.FirefoxTheme
 
@@ -141,7 +146,9 @@ private fun RecentTabItem(
                 Row {
                     RecentTabIcon(
                         url = tab.state.content.url,
-                        modifier = Modifier.size(18.dp).clip(RoundedCornerShape(2.dp)),
+                        modifier = Modifier
+                            .size(18.dp)
+                            .clip(RoundedCornerShape(2.dp)),
                         contentScale = ContentScale.Crop,
                         icon = tab.state.content.icon,
                     )
@@ -287,17 +294,10 @@ private fun RecentTabIcon(
                 alignment = alignment
             )
         }
-        else -> {
+        !inComposePreview -> {
             components.core.icons.Loader(url) {
                 Placeholder {
-                    Box(
-                        modifier = Modifier.background(
-                            color = when (isSystemInDarkTheme()) {
-                                true -> PhotonColors.DarkGrey30
-                                false -> PhotonColors.LightGrey30
-                            }
-                        )
-                    )
+                    PlaceHolderTabIcon(modifier)
                 }
 
                 WithIcon { icon ->
@@ -310,5 +310,54 @@ private fun RecentTabIcon(
                 }
             }
         }
+        else -> {
+            PlaceHolderTabIcon(modifier)
+        }
+    }
+}
+
+/**
+ * A placeholder for the recent tab icon.
+ *
+ * @param modifier [Modifier] used to shape the content.
+ */
+@Composable
+private fun PlaceHolderTabIcon(modifier: Modifier) {
+    Box(
+        modifier = modifier.background(
+            color = when (isSystemInDarkTheme()) {
+                true -> PhotonColors.DarkGrey30
+                false -> PhotonColors.LightGrey30
+            }
+        )
+    )
+}
+
+@Preview(uiMode = Configuration.UI_MODE_NIGHT_YES)
+@Preview(uiMode = Configuration.UI_MODE_NIGHT_NO)
+@Composable
+private fun RecentTabsPreview() {
+    val tab = RecentTab.Tab(
+        TabSessionState(
+            id = "tabId",
+            content = ContentState(
+                url = "www.mozilla.com",
+            )
+        )
+    )
+
+    FirefoxTheme {
+        RecentTabs(
+            recentTabs = listOf(
+                tab
+            ),
+            menuItems = listOf(
+                RecentTabMenuItem(
+                    title = "Menu item",
+                    onClick = {},
+                )
+            ),
+            onRecentTabClick = {},
+        )
     }
 }