Cannot get binds inside a function to work in lua

[LoricAndre]

Hi, I migrated my config to lua and hoped to get rid of my plugins.

I ended up writing this to get per-monitor workspaces (ignore the indentation), but the keybinds don't do anything:

local wsKeys = {
  [1] = 'ampersand',
  [2] = 'eacute',
  [3] = 'quotedbl',
  [4] = 'apostrophe',
  [5] = 'parenleft',
  [6] = 'minus',
  [7] = 'egrave',
  [8] = 'underscore',
  [9] = 'ccedilla',
  [10] = 'agrave',
}

local function mod(key)
  return 'SUPER + ' .. key
end
local function modShift(key)
  return mod('SHIFT + ' .. key)
end

local function wsId(ws)
  local mon = hl.get_active_monitor()
  return ws + 10 * mon.id
end


    for ws, key in pairs(wsKeys) do
      hl.bind(mod(key), function()
        return hl.dsp.focus({ workspace = wsId(ws) })
      end)
      hl.bind(modShift(key), function()
        return hl.dsp.window.move({ workspace = wsId(ws) })
      end)
    end

I debugged with notifications inside the keybind callback, and wsId returns the expected value, and without wrapping it in a function the keybinds get me to a workspace, except that the wsId call gets resolved statically when the config gets loaded.

The docs seems to say that it is possible to call functions inside binds (for multiple dispatchers), am I misunderstanding something ?

[wabarTM]

you dont need return in there
also why are you using function() for a 1-liner

[LoricAndre]

You could read my question properly, if I don't use a function wsId doesn't get called when the keybind is invoked but when the config is built, which makes them completely useless.

[vaxerski]

for ws, key in pairs(wsKeys) do
      hl.bind(mod(key), function()
        return hl.dsp.focus({ workspace = wsId(ws) })
      end)
      hl.bind(modShift(key), function()
        return hl.dsp.window.move({ workspace = wsId(ws) })
      end)
    end

hl.dsp. functions return an object, they don't invoke any operation. What you are essentially doing is attaching a lua fn which does nothing, to a keybind.

I think you simply want hl.dispatch() instead of return.

[LoricAndre]

Thanks for the answer, I'm curious though: I thought that the hl.dsp functions returned the table for later call why doesn't returning the table manually work ?

[vaxerski]

you're not calling the function. You're passing a lambda.

[LoricAndre]

Okay yeah I think I got it, thank you very much for the quick answers