From 1ea1596e4e357ef8db47a2d5aa48c41a1f16baa5 Mon Sep 17 00:00:00 2001 From: M57 <41645758+Hy4ri@users.noreply.github.com> Date: Wed, 12 Aug 2026 11:37:04 +0300 Subject: [PATCH] fix(chat): image viewer icons invisible on light backgrounds (#882) The save/share/close icons and the busy spinner were tinted with colorScheme.onPrimary, which stays white under dynamic light schemes, against the viewer's scrim-colored background. Switch to onSurface, the token designed for content over non-primary surfaces. Co-authored-by: hy4ri --- .../com/m57/hermescontrol/ui/chat/ImageViewerDialog.kt | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/app/src/main/java/com/m57/hermescontrol/ui/chat/ImageViewerDialog.kt b/app/src/main/java/com/m57/hermescontrol/ui/chat/ImageViewerDialog.kt index d5c84cd3..398d8562 100644 --- a/app/src/main/java/com/m57/hermescontrol/ui/chat/ImageViewerDialog.kt +++ b/app/src/main/java/com/m57/hermescontrol/ui/chat/ImageViewerDialog.kt @@ -204,7 +204,7 @@ fun ImageViewerDialog( Icon( imageVector = Icons.AutoMirrored.Filled.ArrowBack, contentDescription = stringResource(R.string.image_viewer_close), - tint = MaterialTheme.colorScheme.onPrimary, + tint = MaterialTheme.colorScheme.onSurface, ) } Row { @@ -212,21 +212,21 @@ fun ImageViewerDialog( CircularProgressIndicator( modifier = Modifier.size(24.dp).padding(horizontal = 8.dp), strokeWidth = 2.dp, - color = MaterialTheme.colorScheme.onPrimary, + color = MaterialTheme.colorScheme.onSurface, ) } else { IconButton(onClick = onSave, enabled = !isBusy) { Icon( imageVector = Icons.Filled.Download, contentDescription = stringResource(R.string.image_viewer_save), - tint = MaterialTheme.colorScheme.onPrimary, + tint = MaterialTheme.colorScheme.onSurface, ) } IconButton(onClick = onShare, enabled = !isBusy) { Icon( imageVector = Icons.Filled.Share, contentDescription = stringResource(R.string.image_viewer_share), - tint = MaterialTheme.colorScheme.onPrimary, + tint = MaterialTheme.colorScheme.onSurface, ) } }