#1657 feat: turn on repeat by default for volume actions

Author: sds100

CHANGELOG.md

@@ -4,18 +4,19 @@
 
 ## Added
 
-- #1652 Bring back the menu button to show input method picker
+- #1652 Bring back the menu button to show input method picker.
+- #1657 Turn on repeat by default for volume actions.
 
 ## Changed
 
 - #1654 The Key Mapper keyboard is now required again for Text actions because the accessibility service API does not work in all situations.
-- #1653 Hide the export/import menu buttons in groups
+- #1653 Hide the export/import menu buttons in groups.
 
 ## Bug fixes
 
 - Inputting key events with Shizuku does not crash the app if a Key Mapper keyboard is being used at the same time. And latency when inputting key events has been improved in some apps.
-- #1646 disabling Bluetooth clears the list of connected devices
-- #1655 do not crash when restoring key map groups
+- #1646 disabling Bluetooth clears the list of connected devices.
+- #1655 do not crash when restoring key map groups.
 
 ## [3.0.0](https://github.com/sds100/KeyMapper/releases/tag/v3.0.0)
 

app/src/main/java/io/github/sds100/keymapper/mappings/keymaps/ConfigKeyMapUseCase.kt

@@ -52,7 +52,6 @@ import kotlinx.coroutines.flow.stateIn
 import kotlinx.coroutines.flow.update
 import kotlinx.coroutines.launch
 import kotlinx.coroutines.withContext
-import kotlinx.serialization.encodeToString
 import kotlinx.serialization.json.Json
 import java.util.LinkedList
 
@@ -802,6 +801,10 @@ class ConfigKeyMapUseCaseController(
             }
         }
 
+        if (data is ActionData.Volume.Down || data is ActionData.Volume.Up || data is ActionData.Volume.Stream) {
+            repeat = true
+        }
+
         if (data is ActionData.AnswerCall) {
             addConstraint(Constraint.PhoneRinging())
         }