Add tooltips

Author: alan412

src/blocks/mrc_mechanism_component_holder.ts

@@ -82,8 +82,16 @@ const MECHANISM_COMPONENT_HOLDER = {
     this.setInputsInline(false);
     this.appendStatementInput(INPUT_MECHANISMS).setCheck(MECHANISM_OUTPUT).appendField(Blockly.Msg.MECHANISMS);
     this.appendStatementInput(INPUT_COMPONENTS).setCheck(COMPONENT_OUTPUT).appendField(Blockly.Msg.COMPONENTS);
-    this.appendStatementInput(INPUT_PRIVATE_COMPONENTS).setCheck(COMPONENT_OUTPUT).appendField(Blockly.Msg.PRIVATE_COMPONENTS);
+    const privateComponentsInput = this.appendStatementInput(INPUT_PRIVATE_COMPONENTS).setCheck(COMPONENT_OUTPUT).appendField(Blockly.Msg.PRIVATE_COMPONENTS);
+    // Set tooltip on the private components field
+    const privateComponentsField = privateComponentsInput.fieldRow[0];
+    if (privateComponentsField) {
+      privateComponentsField.setTooltip(Blockly.Msg.PRIVATE_COMPONENTS_TOOLTIP);
+    }
     this.appendStatementInput(INPUT_EVENTS).setCheck(EVENT_OUTPUT).appendField(Blockly.Msg.EVENTS);
+    
+    // Update components tooltip based on private components visibility
+    this.updateComponentsTooltip_();
 
     this.setOutput(false);
     this.setStyle(MRC_STYLE_MECHANISMS);
@@ -112,6 +120,21 @@ const MECHANISM_COMPONENT_HOLDER = {
     this.mrcHidePrivateComponents = (extraState.hidePrivateComponents == undefined) ? false : extraState.hidePrivateComponents;
     this.updateBlock_();
   },
+  /**
+   * Update the components tooltip based on private components visibility.
+   */
+  updateComponentsTooltip_: function (this: MechanismComponentHolderBlock): void {
+    const componentsInput = this.getInput(INPUT_COMPONENTS);
+    if (componentsInput && componentsInput.fieldRow[0]) {
+      const componentsField = componentsInput.fieldRow[0];
+      // Only show tooltip if private components are also visible (not hidden)
+      if (!this.mrcHidePrivateComponents) {
+        componentsField.setTooltip(Blockly.Msg.COMPONENTS_TOOLTIP);
+      } else {
+        componentsField.setTooltip('');
+      }
+    }
+  },
   /**
      * Update the block to reflect the newly loaded extra state.
      */
@@ -137,10 +160,18 @@ const MECHANISM_COMPONENT_HOLDER = {
     }
     else {
       if (this.getInput(INPUT_PRIVATE_COMPONENTS) == null) {
-        this.appendStatementInput(INPUT_PRIVATE_COMPONENTS).setCheck(COMPONENT_OUTPUT).appendField(Blockly.Msg.PRIVATE_COMPONENTS);
+        const privateComponentsInput = this.appendStatementInput(INPUT_PRIVATE_COMPONENTS).setCheck(COMPONENT_OUTPUT).appendField(Blockly.Msg.PRIVATE_COMPONENTS);
+        // Set tooltip on the field
+        const privateComponentsField = privateComponentsInput.fieldRow[0];
+        if (privateComponentsField) {
+          privateComponentsField.setTooltip(Blockly.Msg.PRIVATE_COMPONENTS_TOOLTIP);
+        }
         this.moveInputBefore(INPUT_PRIVATE_COMPONENTS, INPUT_EVENTS)
       }
     }
+    
+    // Update components tooltip based on private components visibility
+    this.updateComponentsTooltip_();
   },
   onBlockChanged: function (block: Blockly.BlockSvg, blockEvent: Blockly.Events.BlockBase) {
     if (blockEvent.type == Blockly.Events.BLOCK_MOVE) {

src/blocks/tokens.ts

@@ -38,7 +38,9 @@ export function customTokens(t: (key: string) => string): typeof Blockly.Msg {
     MECHANISMS: t('MECHANISMS'),
     OPMODES: t('OPMODES'),
     COMPONENTS: t('BLOCKLY.COMPONENTS'),
+    COMPONENTS_TOOLTIP: t('BLOCKLY.TOOLTIP.COMPONENTS'),
     PRIVATE_COMPONENTS: t('BLOCKLY.PRIVATE_COMPONENTS'),
+    PRIVATE_COMPONENTS_TOOLTIP: t('BLOCKLY.TOOLTIP.PRIVATE_COMPONENTS'),
     EVENTS: t('BLOCKLY.EVENTS'),
     EVALUATE_BUT_IGNORE_RESULT: t('BLOCKLY.EVALUATE_BUT_IGNORE_RESULT'),
     EVALUATE_BUT_IGNORE_RESULT_TOOLTIP:

src/i18n/locales/en/translation.json

@@ -62,7 +62,9 @@
             "OPMODE_TYPE": "What sort of OpMode this is",
             "OPMODE_ENABLED": "Whether the OpMode is shown on Driver Station",
             "OPMODE_NAME": "The name shown on the Driver Station.  If blank will use the class name.",
-            "OPMODE_GROUP": "An optional group to group OpModes on Driver Station"
+            "OPMODE_GROUP": "An optional group to group OpModes on Driver Station",
+            "COMPONENTS": "These components are visible in this mechanism, the robot, and all opmodes.",
+            "PRIVATE_COMPONENTS": "These components will not be visible in the robot or opmodes. They are only accessible within this mechanism."
         },
         "CATEGORY":{
             "LISTS": "Lists",

src/i18n/locales/es/translation.json

@@ -63,7 +63,9 @@
             "OPMODE_TYPE": "Qué tipo de OpMode es este",
             "OPMODE_ENABLED": "Si el OpMode se muestra en la Estación del Conductor",
             "OPMODE_NAME": "El nombre mostrado en la Estación del Conductor. Si está en blanco usará el nombre de la clase.",
-            "OPMODE_GROUP": "Un grupo opcional para agrupar OpModes en la Estación del Conductor"
+            "OPMODE_GROUP": "Un grupo opcional para agrupar OpModes en la Estación del Conductor",
+            "COMPONENTS": "Estos componentes son visibles en este mecanismo, el robot y todos los opmodes.",
+            "PRIVATE_COMPONENTS": "Estos componentes no serán visibles en el robot o en los opmodes. Solo son accesibles dentro de este mecanismo."
         },
         "CATEGORY": {
             "LISTS": "Listas",

src/i18n/locales/he/translation.json

@@ -62,7 +62,9 @@
             "OPMODE_TYPE": "איזה סוג של מצב פעולה זה",
             "OPMODE_ENABLED": "האם מצב הפעולה מוצג בתחנת הנהג",
             "OPMODE_NAME": "השם המוצג בתחנת הנהג. אם ריק ישתמש בשם הכיתה.",
-            "OPMODE_GROUP": "קבוצה אופציונלית לקיבוץ מצבי פעולה בתחנת הנהג"
+            "OPMODE_GROUP": "קבוצה אופציונלית לקיבוץ מצבי פעולה בתחנת הנהג",
+            "COMPONENTS": "רכיבים אלה גלויים במנגנון זה, ברובוט ובכל מצבי הפעולה.",
+            "PRIVATE_COMPONENTS": "רכיבים אלה לא יהיו גלויים ברובוט או במצבי פעולה. הם נגישים רק בתוך המנגנון הזה."
         },
         "CATEGORY":{
             "LISTS": "רשימות",