Skip to content

fix: add click enabled toggle command - #326

Closed
AeiouJx wants to merge 1 commit into
rfresh2:1.21.4from
AeiouJx:fix/click-enabled-command
Closed

fix: add click enabled toggle command#326
AeiouJx wants to merge 1 commit into
rfresh2:1.21.4from
AeiouJx:fix/click-enabled-command

Conversation

@AeiouJx

@AeiouJx AeiouJx commented Sep 3, 2026

Copy link
Copy Markdown

Summary

The Click module's enabled setting was not exposed via command, requiring manual config.json editing to enable it.

Problem

Running click on or click enabled on returns "Invalid command usage". The only way to enable the Click module is to manually edit config.json and restart.

Fix

Adds a toggle command consistent with other module commands:

click on
click off

The Click module's enabled setting was not exposed via command,
requiring manual config.json editing to enable it. This adds a
toggle command consistent with other module commands.
@rfresh2

rfresh2 commented Sep 3, 2026

Copy link
Copy Markdown
Owner

the module is not intended to be toggled. it's designed around the click <left/right> hold actions.

use click stop to disable any active click <left/right> hold

@rfresh2 rfresh2 closed this Sep 3, 2026
@AeiouJx

AeiouJx commented Sep 3, 2026

Copy link
Copy Markdown
Author

I understand the design is built around click <left/right> hold / click stop actions.

However, currently the hold actions do nothing unless the module is enabled in config. There is no way to use these hold behaviors without turning on the module toggle first.

This PR makes the toggle control whether the whole module is active, so users can completely opt‑out of this feature if they don’t want it, without the hold actions being functional at all when disabled.

@rfresh2

rfresh2 commented Sep 3, 2026

Copy link
Copy Markdown
Owner

your issue is caused by yourself.

the module is always toggled on by default - so its you who has manually gone in and edited the config.json

@AeiouJx
AeiouJx deleted the fix/click-enabled-command branch September 3, 2026 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants