mapDoubleTap breaks the normal behavior (single tap, holding)

[mkjawadi]

Hi, mapDoubleTap messes up the usual key behavior (single tap/holding). For example, below config maps that the double tapping the right shift key should toggle the “show desktop” option which works. However single-tap and hold actions for right shift don’t work. So, if you press the right shift key and a regular key (right shift + a), you won’t get the output you want, like typing the capital “A”. BTW, the double tap issue is with all keys not just the right shift key.

Karabiner Element: v15.3.0
karabiner.ts: v1.30.3

writeToProfile("Default profile", [
  rule("Caps Lock → Hyper").manipulators([
    map("⇪").toHyper().toIfAlone("⎋"), // Or map('caps_lock').toHyper().toIfAlone('escape')
    map("⇪", "⇧").to("⇪"), // Or map('caps_lock', 'shift').to('caps_lock')

    // Toggle Show Desktop
    mapDoubleTap("right_shift").to("f11", "fn")
  ])
]);

I have also tried the followings but didn't work either.

mapDoubleTap("right_shift").to("f11", "fn").toIfAlone("right_shift")

mapDoubleTap("right_shift").to("f11", "fn").singleTap(toKey("right_shift"))

Am I missing something?
Thanks in advance. 🙏

[evan-liu]

Try

mapDoubleTap("right_shift").to("f11", "fn").toIfHeldDown("right_shift")

[mkjawadi]

Thanks for the response. No, it doesn't work unless I hold the shift for at least one second which is super slow specially while typing.

[evan-liu]

Hi @situmorang-com , can you please open a new discussion and explain more about what you need? It is not very clear on which part of the JSON you were talking about.

[thienanle]

@evan-liu I think the key part of @situmorang-com 's JSON is this:

            "to": [
                {
                    "set_variable": {
                        "name": "double-tap-left_shift-any",
                        "value": 1
                    }
                },
                { "key_code": "left_shift" }
            ],

This allows left_shift to work when held down. This is not desirable for keys that produce a character, but for modifiers it is better than toIfHeldDown as there is zero delay.

I suggest either it be the default behavior for modifiers, OR singleTap keyCode is inserted at line 155 of double-tap.ts if it is a modifier and the same as the double-tap keyCode.

If you'd like, I could try to create a PR, but wanted to discuss the best approach first.

[evan-liu]

Thanks, @thienanle. A PR would be great. Could you please open an issue to describe the problem and proposed solution, and we can discuss there?

[evan-liu]

I finally get what everyone is talking about. Sorry, I kept missing the issue is that modifiers need different behavior. @thienanle I am on it now. Thanks for offering the help.

[evan-liu]

Please upgrade to 1.34.1 and let me know if it works for you now.

[bboydflo]

Hey everyone!

First of all, thank you @evan-liu for this awesome project! I am just getting started, but I have tried multiple things so far and my conclusion is that §§mapDoubleTap§§ is not working as intended.

I have tried a few things, among others, something like below:

writeToProfile('default', [

  // tried to get this to work
  rule('Aliases', ifApp('Ghostty')).manipulators([
    mapDoubleTap('a')
      .to$(`aerospace list-windows --all | fzf --bind 'enter:execute(fish -c "aerospace focus --window-id {1}")+abort'`),
  ]),

  // also this one as a test
  rule('Double Tap of a').manipulators([
    mapDoubleTap('s').to$('ls -la'),
  ]),

])

And here are my karabiner.json parameters:

"parameters": {
      "basic.to_if_alone_timeout_milliseconds": 1000,
      "basic.to_if_held_down_threshold_milliseconds": 500,
      "basic.to_delayed_action_delay_milliseconds": 500,
      "basic.simultaneous_threshold_milliseconds": 50,
      "mouse_motion_to_scroll.speed": 100
}

Neither work for me. Am I missing something?

[evan-liu]

Hi @bboydflo,

Can you try to confirm whether it is mapDoubleTap() or the command that is not working?

1. Choose/create a simple mapping that is easy to verify. e.g. map('f').toApp('Finder'), and make sure it works.
   1. Change map() to mapDoubleTap() and check if it works. e.g. mapDoubleTap('f').toApp('Finder')
   2. Change the action to a command you are using and check if it works. e.g. map('f').to$('{yourCmdForDoubleTap}')

[bboydflo]

Hi. I just tried map('s').toApp('Finder') and it does work. I have played with different variants and here are some results

rule('Double Tap1').manipulators([
  // map('f').toApp('Finder'), // does work
  mapDoubleTap('a').toApp('Finder'), // does work
]),

rule('Double Tap2', ifApp('com.mitchellh.ghostty')).manipulators([
    // map('x').toApp('Finder'), // works
    map('x').to$('ls -la'), // does not work
    // mapDoubleTap('x').to$('ls -la'), // does not work
]),

rule('Double Tap3', ifApp('Ghostty')).manipulators([
    map('x').toApp('Finder'), // does not work
]),

I learned that ifApp() works if I use the bundle identifier instead of the app name. But to$() will still not work. Tried with different terminals and shells.

I am using karabiner-elements 15.3.0

[bboydflo]

I have a suspicion about what's going on. Issue might be related to my default shell which is fish :( I've seen some logs in karabiner-elements and it tries to run a command with /bin/sh but I am pretty sure that the tool can't be run from /bin/sh.

Although, i've tried other commands as well, such as: ls -la

[evan-liu]

I think what you need is toPaste() which paste the command into your terminal, instead of running the command.

[bboydflo]

Thanks! It works. Sorry for the trouble! I probably misunderstand the documentation

[evan-liu]

mapDoubleTap('left_shift') now produce:

{
  "description": "mapDoubleTap()",
  "manipulators": [
    {
      "type": "basic",
      "from": { "key_code": "left_shift" },
      "conditions": [
        { "type": "variable_if", "name": "double-tap-left_shift", "value": 1 }
      ],
      "description": "__support__manipulator"
    },
    {
      "type": "basic",
      "from": { "key_code": "left_shift" },
      "to": [
        { "set_variable": { "name": "double-tap-left_shift", "value": 1 } },
        { "key_code": "left_shift", "lazy": true } // 1 Quick roll over without holding down
      ],
      "conditions": [
        {
          "type": "variable_unless",
          "name": "double-tap-left_shift",
          "value": 1
        }
      ],
      "to_delayed_action": {
        "to_if_invoked": [
          { "key_code": "left_shift" }, // 2 Tap
          { "set_variable": { "name": "double-tap-left_shift", "value": 0 } }
        ],
        "to_if_canceled": [
          { "set_variable": { "name": "double-tap-left_shift", "value": 0 } }
        ]
      },
      "to_if_held_down": [{ "key_code": "left_shift" }], // 3 Held
      "parameters": {
        "basic.to_if_held_down_threshold_milliseconds": 200,
        "basic.to_delayed_action_delay_milliseconds": 200
      }
    }
  ]
}