New lua config, layout: ignoring "unsupported" layoutmsg

[vp1981]

Hello.

In the old config (hyprlang) it was possible to have such binds

bindd = $mod CTRL , Right    , Move the focus to right and center the layout, layoutmsg, focus r
bindd = $mod CTRL , Left     , Move the focus to left and center the layout , layoutmsg, focus l
bindd = $mod CTRL , Up       , Move the focus to right and center the layout, layoutmsg, focus u
bindd = $mod CTRL , Down     , Move the focus to left and center the layout , layoutmsg, focus d

these worked only in the scrolling layout, but did nothing in others. I could swear I saw information on the Wiki, that if the current layout doesn't support such messages it simply ignores them. And it used to work.

Now, as I'm gradually moving to the new lua config, I've found that this code

hl.bind(_kb(mod, "CTRL + right"),
  hl.dsp.layout("focus r"),
  { description = "Move the focus to right and center the layout" }
)
hl.bind(_kb(mod, "CTRL + left"),
  hl.dsp.layout("focus l"),
  { description = "Move the focus to left and center the layout" }
)
hl.bind(_kb(mod, "CTRL + up"),
  hl.dsp.layout("focus u"),
  { description = "Move the focus to right and center the layout" }
)
hl.bind(_kb(mod, "CTRL + down"),
  hl.dsp.layout("focus d"),
  { description = "Move the focus to right and center the layout" }
)

causes a warning, or actually an error, if run in the terminal like this

$ hyprctl dispatch 'hl.dsp.layout("focus r")'

Is there a way to ignore such "errors"?

[vaxerski]

what's the error? I don't get any.

[vaxerski]

that's a bug in dwindle my bad

[vaxerski]

also I forgot the above wont work, here's a fixed one

function dispatch_if_layout(dsp, layout_name)
    return function()
        local ws = hl.get_active_workspace()
        if ws == nil then return end
        if ws.tiled_layout == layout_name then return hl.dispatch(dsp) end
    end
end

hl.bind("SUPER + Left", dispatch_if_layout(hl.dsp.layout("focus left"), "scrolling"))

[vp1981]

@vaxerski:

that's a bug in dwindle

consistent behavior, hooray!

also I forgot the above wont work, here's a fixed one

Thanks again! I'll implement this in my hyprland.lua.

[vp1981]

For the record, now I use the following:

function _dsp_law(layout, dsp)
  --- Layout aware dispatcher. Thanks to vaxerski.
  return function()
    local ws = hl.get_active_workspace()
    if ws == nil then return end
    if ws.tiled_layout == layout then return hl.dispatch(hl.dsp.layout(dsp)) end
  end
end
-- For both monocle and scrolling layouts.
hl.bind(_kb(mod, "right"),
  function()
    local ws = hl.get_active_workspace()
    if ws.tiled_layout == "monocle" then
      hl.dispatch(hl.dsp.layout("cyclenext"))
    end
    if ws.tiled_layout == "scrolling" then
      hl.dispatch(hl.dsp.layout("move +col"))
    end
  end,
  { description = "Move focus to next window (monocle/scrolling layout)" }
)
hl.bind(_kb(mod, "left"),
  function()
    local ws = hl.get_active_workspace()
    if ws.tiled_layout == "monocle" then
      hl.dispatch(hl.dsp.layout("cycleprev"))
    end
    if ws.tiled_layout == "scrolling" then
      hl.dispatch(hl.dsp.layout("move -col"))
    end
  end,
  { description = "Move focus to previous window (monocle/scrolling layout)" }
)
-- scrolling specific
hl.bind(_kb(mod, "SHIFT + right"), _dsp_law("scrolling", "colresize +conf"), { description = "Enlarge current column by predefined value" })
hl.bind(_kb(mod, "SHIFT + left"),  _dsp_law("scrolling", "colresize -conf"), { description = "Shrink current column by predefined value" })
hl.bind(_kb(mod, "CTRL + right"),  _dsp_law("scrolling", "focus r"),         { description = "Move the focus to right and center the layout" })
hl.bind(_kb(mod, "CTRL + left"),   _dsp_law("scrolling", "focus l"),         { description = "Move the focus to left and center the layout" })
hl.bind(_kb(mod, "CTRL + up"),     _dsp_law("scrolling", "focus u"),         { description = "Move the focus to right and center the layout" })
hl.bind(_kb(mod, "CTRL + down"),   _dsp_law("scrolling", "focus d"),         { description = "Move the focus to right and center the layout" })