Fix PC Start button Meta command

Route the PC Window Start tile through keyboard.shortcut with Meta so the PC opens Start reliably.

Author: enaboapps

app/src/main/java/com/enaboapps/switchify/screens/pc/PcWindowControlScreen.kt

@@ -21,6 +21,7 @@ import androidx.compose.ui.unit.dp
 import com.enaboapps.switchify.R
 import com.enaboapps.switchify.pc.PcControlCommand
 import com.enaboapps.switchify.pc.PcKeyboardKey
+import com.enaboapps.switchify.pc.PcKeyboardShortcutKey
 import com.enaboapps.switchify.pc.PcWindowControlAction
 
 data class PcWindowControlSpec(
@@ -77,7 +78,7 @@ fun pcWindowControlSpecs(): List<PcWindowControlSpec> {
     return listOf(
         PcWindowControlSpec(
             R.string.pc_key_start,
-            PcControlCommand.PressKey(PcKeyboardKey.Meta),
+            PcControlCommand.KeyboardShortcut(listOf(PcKeyboardShortcutKey.Meta)),
             Icons.Rounded.Computer
         ),
         PcWindowControlSpec(

app/src/test/java/com/enaboapps/switchify/screens/pc/PcMouseControlViewModelTest.kt

@@ -16,6 +16,7 @@ import com.enaboapps.switchify.pc.PcDiscovery
 import com.enaboapps.switchify.pc.PcDiscoveryStatus
 import com.enaboapps.switchify.pc.PcErrorReason
 import com.enaboapps.switchify.pc.PcKeyboardKey
+import com.enaboapps.switchify.pc.PcKeyboardShortcutKey
 import com.enaboapps.switchify.pc.PcLiveControlResult
 import com.enaboapps.switchify.pc.PcMouseRepeatManager
 import com.enaboapps.switchify.pc.PcMouseRepeatSettings
@@ -1006,11 +1007,11 @@ class PcMouseControlViewModelTest {
     }
 
     @Test
-    fun windowSurfaceCanSendStartKeyCommand() = runTest(dispatcher) {
+    fun windowSurfaceCanSendStartShortcutCommand() = runTest(dispatcher) {
         val connector = FakeConnector()
         val controller = connectedController(connector = connector)
         val viewModel = viewModel(controller)
-        val command = PcControlCommand.PressKey(PcKeyboardKey.Meta)
+        val command = PcControlCommand.KeyboardShortcut(listOf(PcKeyboardShortcutKey.Meta))
 
         viewModel.selectControlSurface(PcControlSurface.Window)
         viewModel.send(command)

app/src/test/java/com/enaboapps/switchify/screens/pc/PcWindowControlScreenTest.kt

@@ -3,6 +3,7 @@ package com.enaboapps.switchify.screens.pc
 import com.enaboapps.switchify.R
 import com.enaboapps.switchify.pc.PcControlCommand
 import com.enaboapps.switchify.pc.PcKeyboardKey
+import com.enaboapps.switchify.pc.PcKeyboardShortcutKey
 import com.enaboapps.switchify.pc.PcWindowControlAction
 import org.junit.Assert.assertEquals
 import org.junit.Test
@@ -14,7 +15,7 @@ class PcWindowControlScreenTest {
 
         assertEquals(
             listOf(
-                PcControlCommand.PressKey(PcKeyboardKey.Meta),
+                PcControlCommand.KeyboardShortcut(listOf(PcKeyboardShortcutKey.Meta)),
                 PcControlCommand.WindowControl(PcWindowControlAction.SwitchNext),
                 PcControlCommand.WindowControl(PcWindowControlAction.SwitchPrevious),
                 PcControlCommand.WindowControl(PcWindowControlAction.TaskView),
@@ -32,7 +33,7 @@ class PcWindowControlScreenTest {
         val specs = pcWindowControlSpecs()
 
         assertEquals(R.string.pc_key_start, specs[0].labelResId)
-        assertEquals(PcControlCommand.PressKey(PcKeyboardKey.Meta), specs[0].command)
+        assertEquals(PcControlCommand.KeyboardShortcut(listOf(PcKeyboardShortcutKey.Meta)), specs[0].command)
         assertEquals(R.string.pc_window_switch_next, specs[1].labelResId)
         assertEquals(PcControlCommand.WindowControl(PcWindowControlAction.SwitchNext), specs[1].command)
         assertEquals(R.string.pc_window_switch_previous, specs[2].labelResId)