Skip to content

Commit

Permalink
fix: only show error for dpad triggers requiring a keyboard if the ke…
Browse files Browse the repository at this point in the history
…y is detected from an input method
  • Loading branch information
sds100 committed Jan 20, 2025
1 parent 115cadb commit 017e6bf
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ import io.github.sds100.keymapper.data.repositories.PreferenceRepository
import io.github.sds100.keymapper.mappings.DisplaySimpleMappingUseCase
import io.github.sds100.keymapper.mappings.keymaps.trigger.AssistantTriggerKey
import io.github.sds100.keymapper.mappings.keymaps.trigger.KeyCodeTriggerKey
import io.github.sds100.keymapper.mappings.keymaps.trigger.KeyEventDetectionSource
import io.github.sds100.keymapper.mappings.keymaps.trigger.TriggerError
import io.github.sds100.keymapper.purchasing.ProductId
import io.github.sds100.keymapper.purchasing.PurchasingManager
Expand Down Expand Up @@ -128,7 +129,7 @@ class DisplayKeyMapUseCaseImpl(

val containsDpadKey = trigger.keys
.mapNotNull { it as? KeyCodeTriggerKey }
.any { InputEventUtils.isDpadKeyCode(it.keyCode) }
.any { InputEventUtils.isDpadKeyCode(it.keyCode) && it.detectionSource == KeyEventDetectionSource.INPUT_METHOD }

if (showDpadImeSetupError.first() && !isKeyMapperImeChosen && containsDpadKey) {
errors.add(TriggerError.DPAD_IME_NOT_SELECTED)
Expand Down

0 comments on commit 017e6bf

Please sign in to comment.