Fix release workflow

Author: theapache64

.github/workflows/release.yml

@@ -27,7 +27,7 @@ jobs:
         run: ./gradlew :app:assembleRelease
 
       - name: Sign APK (if secrets are configured)
-        if: ${{ secrets.SIGNING_KEY != '' }}
+        continue-on-error: true
         uses: r0adkll/sign-android-release@v1
         id: sign_app
         with:
@@ -40,12 +40,12 @@ jobs:
           BUILD_TOOLS_VERSION: "34.0.0"
 
       - name: Rename signed APK
-        if: ${{ secrets.SIGNING_KEY != '' }}
+        if: steps.sign_app.outcome == 'success'
         run: |
           mv ${{steps.sign_app.outputs.signedReleaseFile}} klokk-android.apk
 
       - name: Rename unsigned APK
-        if: ${{ secrets.SIGNING_KEY == '' }}
+        if: steps.sign_app.outcome != 'success'
         run: |
           mv app/build/outputs/apk/release/app-release-unsigned.apk klokk-android.apk
 

app/src/main/kotlin/com/theapache64/klokk/MainActivity.kt

@@ -1,5 +1,7 @@
 package com.theapache64.klokk
 
+import android.app.UiModeManager
+import android.content.res.Configuration
 import android.os.Build
 import android.os.Bundle
 import android.view.View
@@ -20,6 +22,10 @@ class MainActivity : ComponentActivity() {
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
 
+        // Detect if running on Android TV
+        val uiModeManager = getSystemService(UI_MODE_SERVICE) as UiModeManager
+        val isTvMode = uiModeManager.currentModeType == Configuration.UI_MODE_TYPE_TELEVISION
+
         // Enable edge-to-edge display
         WindowCompat.setDecorFitsSystemWindows(window, false)
 
@@ -31,6 +37,7 @@ class MainActivity : ComponentActivity() {
 
         setContent {
             MainScreen(
+                isTvMode = isTvMode,
                 onControlsVisibilityChanged = { visible ->
                     areControlsVisible = visible
                     updateSystemUiVisibility(visible)

shared/src/commonMain/kotlin/com/theapache64/klokk/MainScreen.kt

@@ -39,6 +39,7 @@ const val TOOLBAR_HEIGHT = 60
 @OptIn(ExperimentalFoundationApi::class)
 @Composable
 fun MainScreen(
+    isTvMode: Boolean = false,
     onControlsVisibilityChanged: (Boolean) -> Unit = {},
     onShowControlsHandler: ((showControls: () -> Unit) -> Unit) = {}
 ) {
@@ -188,6 +189,7 @@ fun MainScreen(
                 // Show toolbar only when controls are visible
                 if (areControlsVisible) {
                     BottomToolBar(
+                        isTvMode = isTvMode,
                         activeMovement = activeMovement,
                         isAnimPlaying = shouldPlayAutoAnim,
 

shared/src/commonMain/kotlin/com/theapache64/klokk/composable/BottomToolBar.kt

@@ -38,6 +38,7 @@ import com.theapache64.klokk.movement.core.Movement
 
 @Composable
 fun BottomToolBar(
+    isTvMode: Boolean = false,
     activeMovement: Movement, // to show debug info
     isAnimPlaying: Boolean,
     textInput: String,
@@ -74,7 +75,8 @@ fun BottomToolBar(
             IconTextButton(
                 text = "START AUTOPLAY",
                 imageVector = Icons.Outlined.PlayArrow,
-                onClicked = onPlayClicked
+                onClicked = onPlayClicked,
+                isTvMode = isTvMode
             )
         }
 
@@ -83,7 +85,8 @@ fun BottomToolBar(
             IconTextButton(
                 text = "STOP AUTOPLAY",
                 imageVector = Icons.Outlined.Stop,
-                onClicked = onStopClicked
+                onClicked = onStopClicked,
+                isTvMode = isTvMode
             )
         }
 
@@ -93,15 +96,17 @@ fun BottomToolBar(
                 text = "SHOW TIME",
                 imageVector = Icons.Outlined.Update,
                 onClicked = onShowTimeClicked,
-                focusRequester = showTimeFocusRequester
+                focusRequester = showTimeFocusRequester,
+                isTvMode = isTvMode
             )
         }
 
         // Immersive Mode Button
         IconTextButton(
             text = "HIDE CONTROLS",
             imageVector = Icons.Outlined.Fullscreen,
-            onClicked = onHideControlsClicked
+            onClicked = onHideControlsClicked,
+            isTvMode = isTvMode
         )
 
         Row(
@@ -137,10 +142,14 @@ private fun IconTextButton(
     text: String,
     imageVector: ImageVector,
     onClicked: () -> Unit,
+    isTvMode: Boolean = false,
     focusRequester: FocusRequester? = null,
 ) {
     var isFocused by remember { mutableStateOf(false) }
 
+    // Show focus highlighting only on TV
+    val showFocusHighlight = isTvMode && isFocused
+
     Box(
         modifier = Modifier
             .then(
@@ -154,34 +163,34 @@ private fun IconTextButton(
                 isFocused = focusState.isFocused
             }
             .border(
-                width = if (isFocused) 2.dp else 0.dp,
-                color = if (isFocused) Color(0xFFFFD700) else Color.Transparent,
+                width = if (showFocusHighlight) 2.dp else 0.dp,
+                color = if (showFocusHighlight) Color(0xFFFFD700) else Color.Transparent,
                 shape = RoundedCornerShape(4.dp)
             )
-            .padding(if (isFocused) 2.dp else 0.dp)
+            .padding(if (showFocusHighlight) 2.dp else 0.dp)
     ) {
         OutlinedButton(
             onClick = onClicked,
             modifier = Modifier.focusable(),
             border = BorderStroke(
                 width = 1.dp,
-                color = if (isFocused) Color(0xFFFFD700) else Color.Gray
+                color = if (showFocusHighlight) Color(0xFFFFD700) else Color.Gray
             ),
             colors = ButtonDefaults.outlinedButtonColors(
-                backgroundColor = if (isFocused) Color(0xFFFFD700).copy(alpha = 0.2f) else Color.Transparent
+                backgroundColor = if (showFocusHighlight) Color(0xFFFFD700).copy(alpha = 0.2f) else Color.Transparent
             )
         ) {
 
             Icon(
                 imageVector = imageVector,
-                tint = if (isFocused) Color(0xFFFFD700) else Color.White,
+                tint = if (showFocusHighlight) Color(0xFFFFD700) else Color.White,
                 contentDescription = "ToolBar Icon",
                 modifier = Modifier.padding(end = 10.dp)
             )
 
             Text(
                 text = text,
-                color = if (isFocused) Color(0xFFFFD700) else Color.White
+                color = if (showFocusHighlight) Color(0xFFFFD700) else Color.White
             )
         }
     }