From a43f9f3412e4f4f51cbc23fcd82213cca1fc9c74 Mon Sep 17 00:00:00 2001
From: austinried <4966622+austinried@users.noreply.github.com>
Date: Sat, 26 Sep 2026 03:57:48 +0000
Subject: [PATCH] keep the menu icons on their content colour so they are not
 black

Color.Unspecified disables Icon's colour filter, which fell back to the vector's black; only
the filled star should be the primary colour.
---
 .../kotlin/com/subtracks/ui/components/ItemContextMenu.kt  | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/app/src/main/kotlin/com/subtracks/ui/components/ItemContextMenu.kt b/app/src/main/kotlin/com/subtracks/ui/components/ItemContextMenu.kt
index 6335c729..3bb34644 100644
--- a/app/src/main/kotlin/com/subtracks/ui/components/ItemContextMenu.kt
+++ b/app/src/main/kotlin/com/subtracks/ui/components/ItemContextMenu.kt
@@ -22,6 +22,7 @@ import androidx.compose.material3.ExperimentalMaterial3Api
 import androidx.compose.material3.Icon
 import androidx.compose.material3.ListItem
 import androidx.compose.material3.ListItemDefaults
+import androidx.compose.material3.LocalContentColor
 import androidx.compose.material3.MaterialTheme
 import androidx.compose.material3.ModalBottomSheet
 import androidx.compose.material3.Text
@@ -235,7 +236,7 @@ private fun StarItem(
     MenuItem(
         starIcon(starred),
         starLabel(starred),
-        tint = if (starred == null) Color.Unspecified else MaterialTheme.colorScheme.primary,
+        tint = if (starred == null) null else MaterialTheme.colorScheme.primary,
     ) {
         val previous = starred
         val starring = previous == null
@@ -250,12 +251,12 @@ private fun StarItem(
 private fun MenuItem(
     icon: ImageVector,
     label: String,
-    tint: Color = Color.Unspecified,
+    tint: Color? = null,
     onClick: () -> Unit,
 ) {
     ListItem(
         headlineContent = { Text(label) },
-        leadingContent = { Icon(icon, contentDescription = null, tint = tint) },
+        leadingContent = { Icon(icon, contentDescription = null, tint = tint ?: LocalContentColor.current) },
         colors = ListItemDefaults.colors(containerColor = Color.Transparent),
         modifier = Modifier.clickable(onClick = onClick),
     )