From 4146c4c31d069ce9fb7ab36d72f1f8fa2be1e050 Mon Sep 17 00:00:00 2001 From: arkon Date: Sat, 4 Nov 2023 17:50:33 -0400 Subject: [PATCH] Ensure page indicator texts are centered Maybe fixes #9976 --- .../presentation/reader/PageIndicatorText.kt | 44 ++++++++++++------- 1 file changed, 29 insertions(+), 15 deletions(-) diff --git a/app/src/main/java/eu/kanade/presentation/reader/PageIndicatorText.kt b/app/src/main/java/eu/kanade/presentation/reader/PageIndicatorText.kt index 69df2a727..470a731c8 100644 --- a/app/src/main/java/eu/kanade/presentation/reader/PageIndicatorText.kt +++ b/app/src/main/java/eu/kanade/presentation/reader/PageIndicatorText.kt @@ -4,11 +4,14 @@ import androidx.compose.foundation.layout.Box import androidx.compose.material3.MaterialTheme import androidx.compose.material3.Text import androidx.compose.runtime.Composable +import androidx.compose.ui.Alignment import androidx.compose.ui.graphics.Color import androidx.compose.ui.graphics.drawscope.Stroke import androidx.compose.ui.text.TextStyle import androidx.compose.ui.text.font.FontWeight import androidx.compose.ui.unit.sp +import eu.kanade.presentation.theme.TachiyomiTheme +import tachiyomi.presentation.core.util.ThemePreviews @Composable fun PageIndicatorText( @@ -19,24 +22,35 @@ fun PageIndicatorText( val text = "$currentPage / $totalPages" - Box { - Text( - text = text, - color = Color(45, 45, 45), - fontSize = MaterialTheme.typography.bodySmall.fontSize, - fontWeight = FontWeight.Bold, - letterSpacing = 1.sp, - style = TextStyle.Default.copy( - drawStyle = Stroke(width = 4f), - ), - ) + val style = TextStyle( + color = Color(235, 235, 235), + fontSize = MaterialTheme.typography.bodySmall.fontSize, + fontWeight = FontWeight.Bold, + letterSpacing = 1.sp, + ) + val strokeStyle = style.copy( + color = Color(45, 45, 45), + drawStyle = Stroke(width = 4f), + ) + Box( + contentAlignment = Alignment.Center, + ) { Text( text = text, - color = Color(235, 235, 235), - fontSize = MaterialTheme.typography.bodySmall.fontSize, - fontWeight = FontWeight.Bold, - letterSpacing = 1.sp, + style = strokeStyle, + ) + Text( + text = text, + style = style, ) } } + +@ThemePreviews +@Composable +private fun PageIndicatorTextPreview() { + TachiyomiTheme { + PageIndicatorText(currentPage = 10, totalPages = 69) + } +}