search

Matts / MacroKey

A mod that allows you to bind keys
17 stars 10 forks source link

Trying to bind the Macrokey command to a macro prints that the command does not exist #66

Closed floral-qua-floral closed 2 years ago

floral-qua-floral commented 4 years ago

Issue description Binding "/macrokey layer toggle" to a macro does not work. Attempting to use such a macro will output to the chat that the command does not exist.

To Reproduce

  1. Load the mod.
  2. Create at least two macrokey layers
  3. Run the command "/macrokey layer toggle" manually to make sure it is working
  4. Copy the command and paste it into a Macrokey macro
  5. Press the button and observe the output in chat.

Expected behaviour The keybind should swap between Macrokey layers just the same as running the command does.

Screenshots 2019-12-10_14 11 06 2019-12-10_14 11 12

Desktop (please complete the following information):

Downloaded From CurseForge

Additional context This prevents toggle keybinds from being used.

mchorse commented 4 years ago

As far as I know, /macrokey command is client side, and MacroKey doesn't support client side commands. You can check that by Tabbing in the chat and seeing whether the name of the command is gray as opposed to white.

Matts commented 4 years ago

@mchorse is correct here. Currently, we do not support client-side commands. I do have this on the backlog IIRC (if not I will add an item). Been very busy with work and school the past few months, hope to be able to have time to come back to modding during my Christmas vacation (starting to miss it, and still have an almost ready 2.1 version I promised to bring out almost a year ago). Will keep the issue open until I have addressed it.

Thanks for choosing MacroKey

~Matt

Matts commented 2 years ago

Confirmed working in 2.1.0