From 4694422c0c992f2583f43634809cbdb86d96092a Mon Sep 17 00:00:00 2001
From: my <my@email.com>
Date: Thu, 24 Sep 2026 11:23:02 +0200
Subject: [PATCH] imporve help

---
 src/meshberry/chat.cpp | 55 ++++++++++++++++++++++++++++--------------
 1 file changed, 37 insertions(+), 18 deletions(-)

diff --git a/src/meshberry/chat.cpp b/src/meshberry/chat.cpp
index e7768fa..798bb8a 100644
--- a/src/meshberry/chat.cpp
+++ b/src/meshberry/chat.cpp
@@ -100,8 +100,8 @@ void handleCommand(const char* in) {
         strncpy(currentNick, in + 6, 15); currentNick[15] = '\0';
         settings_set_nick(currentNick); // persist to LittleFS
         ui_set_header(nodesCount, currentNick); // refresh header bar
-        char l[MAX_MSG_LEN]; snprintf(l, sizeof(l),
-                         "Nickname set to: %s", currentNick);
+        char l[MAX_MSG_LEN]; 
+        snprintf(l, sizeof(l), "Nickname set to: %s", currentNick);
         ui_push_message("", l, false, true);
 
     } else if (!strcmp(in, "/clear")) {
@@ -125,7 +125,7 @@ void handleCommand(const char* in) {
             String key = args.substring(0, spaceIdx);
             String value = args.substring(spaceIdx + 1);
             if ( apply_setting (key, value)) {
-                ui_push_message("", "Saved. Restart to apply.", false, true);
+                ui_push_message("", "Saved. For colors: restart to apply.", false, true);
             } else {
                 ui_push_message("", "Unknown key or invalid value.", false, true);
             }
@@ -145,22 +145,41 @@ void handleCommand(const char* in) {
             ui_push_message("", b, false, true);
         }
 
-    } else if (!strcmp(in, "/help")) {
-        //addMessage("/nick <name> : change pseudo");
-        //addMessage("/me <action> : action roleplay");
-        //addMessage("/clear       : clear the chat");
-        //addMessage("/people      : list connected");
-        ui_push_message("", "/nick <name> : change pseudo",   false, true);
-        ui_push_message("", "/me <action> : action roleplay", false, true);
-        ui_push_message("", "/clear       : clear the chat",  false, true);
-        ui_push_message("", "/clearnotes  : erase saved notes", false, true);
-        ui_push_message("", "/bt          : toggle BT send mode", false, true);
-        ui_push_message("", "/set color_bg ff0000 : set a color (restart to apply)", false, true);
-        ui_push_message("", "/people      : list connected",  false, true);
-        ui_push_message("", "/sleep or /s : sleep device",  false, true);
-        ui_push_message("", "TAB : notes mode",  false, true);
-        //drawChatWindow();
+    } else if (strncmp(in, "/help", 5) == 0) {
+        if (in[5] == '\0' || in[5] == ' ') {
+            const char* hargs = in + 5;
+            while (*hargs == ' ') {
+                hargs++;
+            }
 
+            Serial.println("help?");
+            if (*hargs == '\0') {
+                ui_push_message("", "/nick <name> : change nickname",   false, true);
+                ui_push_message("", "/clear : clear the chat",  false, true);
+                ui_push_message("", "/clearnotes  : erase saved notes", false, true);
+                ui_push_message("", "/bt : toggle BT send mode", false, true);
+                ui_push_message("", "/set <key> <value> : change a setting",  false, true);
+                ui_push_message("", "/people : list connected",  false, true);
+                ui_push_message("", "/s or /sleep: sleep device",  false, true);
+                ui_push_message("", "TAB : toggle notes/chat mode",  false, true);
+                ui_push_message("", "/help colors, /help tones or /help settings for more help",  false, true);
+            }
+            else
+            {
+                if(!strcmp(hargs, "colors")) {
+                    ui_push_message("", "/set <region> <color>: change colors",   false, true);
+                    ui_push_message("", "<region>: color_bg, color_header_bg, color_msg, color_nick_other, color_nick_own, color_system, color_input_prompt",   false, true);
+                    ui_push_message("", "<color> is a hexadecimal color, like ff0000 for red. Dont add the #",   false, true);
+                } else if(!strcmp(hargs, "tones")) {
+                    ui_push_message("", "/set <event> <tones>: change tones",   false, true);
+                    ui_push_message("", "<event>: tones_msg (incoming messages) and tones_sys (other messages)",   false, true);
+                    ui_push_message("", "<tones> is a MML string. Like '/set tones_msg t255 O5 l8 cag': Message tone is Tempo 255, Octave 5, 8th length C, A, G notes.",   false, true);
+                } else if(!strcmp(hargs, "settings")) {
+                    ui_push_message("", "/set brightness <0-255> : screen brightness",   false, true);
+                    ui_push_message("", "/set touch_threshold <n> : touch wake sensitivity",   false, true);
+                }
+            }
+        }
     } else if (!strcmp(in, "/sleep") || !strcmp(in, "/s")) {
         go_to_sleep(); // sleep device. maybe add a button?