[Bug] if layer has optionalAny, keys should allow optionalAny as well

[DrakeEvans]

karabiner.ts/src/config/layer.ts

Line 154 in 40f160e

if (

This code above enforces layers with modifiers do not allow keymaps to have modifiers but does not handle the case when both need to have optional any modifiers

This code produces valid configuration for example and is required to allow common shortcuts like command + left_arrow etc for the remapped keys

layer('semicolon', 'semicolon-down')
    .modifiers({ optional: 'any' })
    .manipulators([
      withModifier({ optional: 'any'})([
        map('l').toAfterKeyUp('right_arrow'),
        withMapper({
          j: 'left_arrow',
          k: 'down_arrow',
          l: 'right_arrow',
          i: 'up_arrow',
        })((k, v) => map(k).to(v as ToKeyCode)),
      ]),
    ]),

[evan-liu]

Hi @DrakeEvans , thanks for trying the project out. It is a cool idea to pass optionalAny on the layer to manipulators. I've made it the default behaviour, so you do not need to manually add withModifier('??'):

layer(';', 'arrow-mode')
  .modifiers('??')
  .manipulators({
    j: toKey('←'),
    i: toKey('↑'),
    k: toKey('↓'),
    l: toKey('→'),
  }),

Please update (1.20.0) and give it a try, and let me know if it works for you.

[DrakeEvans]

I rebased and it looks like its working!! Thank you