From 5be20a50c2332336463ec175ec7aea1fc6540ea8 Mon Sep 17 00:00:00 2001
From: Benjamin Ihrig <benjamin.ihrig@gmail.com>
Date: Tue, 8 Nov 2022 14:34:42 +0100
Subject: [PATCH] Add an option to hide link nodes from the list

This adds an hide option to link in and link out in order to be able to hide both from the respective linking list.
---
 .../@node-red/nodes/core/common/60-link.html  | 31 ++++++++++++++++++-
 .../@node-red/nodes/locales/de/messages.json  |  4 ++-
 .../nodes/locales/en-US/common/60-link.html   |  4 +++
 .../nodes/locales/en-US/messages.json         |  4 +++
 4 files changed, 41 insertions(+), 2 deletions(-)

diff --git a/packages/node_modules/@node-red/nodes/core/common/60-link.html b/packages/node_modules/@node-red/nodes/core/common/60-link.html
index 4b8c9a3d6..ec658ad8b 100644
--- a/packages/node_modules/@node-red/nodes/core/common/60-link.html
+++ b/packages/node_modules/@node-red/nodes/core/common/60-link.html
@@ -4,6 +4,14 @@
         <label for="node-input-name"><i class="fa fa-tag"></i> <span data-i18n="common.label.name"></span></label>
         <input type="text" id="node-input-name" data-i18n="[placeholder]common.label.name">
     </div>
+    <div class="form-row">
+        <label for="node-input-hide"><i class="fa fa-eye"></i> <span data-i18n="link.visibility"></span></label>
+        <select id="node-input-visibility">
+            <option value="global" data-i18n="link.visibilityGlobal"></option>
+            <option value="flow" data-i18n="link.visibilityFlow"></option>
+            <option value="none" data-i18n="link.visibilityNone"></option>
+        </select>
+    </div>
     <div style="position:relative; height: 30px; text-align: right;"><div style="display:inline-block"><input type="text" id="node-input-link-target-filter"></div></div>
     <div class="form-row node-input-link-row"></div>
 </script>
@@ -12,6 +20,14 @@
         <label for="node-input-name"><i class="fa fa-tag"></i> <span data-i18n="common.label.name"></span></label>
         <input type="text" id="node-input-name" data-i18n="[placeholder]common.label.name">
     </div>
+    <div class="form-row">
+        <label for="node-input-visibility"><i class="fa fa-eye"></i> <span data-i18n="link.visibility"></span></label>
+        <select id="node-input-visibility">
+            <option value="global" data-i18n="link.visibilityGlobal"></option>
+            <option value="flow" data-i18n="link.visibilityFlow"></option>
+            <option value="none" data-i18n="link.visibilityNone"></option>
+        </select>
+    </div>
     <div class="form-row">
         <label for="node-input-mode"><span data-i18n="link.outMode"></span></label>
         <select id="node-input-mode" style="width: 70%">
@@ -76,7 +92,18 @@
                     RED.view.redraw();
                 }
             });
-        const candidateNodes = RED.nodes.filterNodes({type:targetType});
+        const candidateNodes = RED.nodes.filterNodes({type:targetType}).filter(n => {
+            console.log(node);
+            console.log(n);
+            if (n.visibility === "none") {
+                return false;
+            }
+            if (n.visibility === "global") {
+                return true;
+            }
+            // visibility === "flow"
+            return n.z === node.z;
+        });
         let candidateNodesCount = 0;
 
         const search = $("#node-input-link-target-filter").searchBox({
@@ -236,6 +263,7 @@
         color:"#ddd",//"#87D8CF",
         defaults: {
             name: { value: "_DEFAULT_" },
+            visibility: { value: "global" },
             links: { value: [], type:"link out[]" }
         },
         inputs:0,
@@ -330,6 +358,7 @@
         color:"#ddd",//"#87D8CF",
         defaults: {
             name: { value:"_DEFAULT_" },
+            visibility: { value: "global" },
             mode: { value: "link" },// link || return
             links: { value: [], type:"link in[]" }
         },
diff --git a/packages/node_modules/@node-red/nodes/locales/de/messages.json b/packages/node_modules/@node-red/nodes/locales/de/messages.json
index 65f251e98..577e5ea18 100755
--- a/packages/node_modules/@node-red/nodes/locales/de/messages.json
+++ b/packages/node_modules/@node-red/nodes/locales/de/messages.json
@@ -158,7 +158,9 @@
     },
     "link": {
         "linkIn": "Eingehende Verbindung",
-        "linkOut": "Ausgehende Verbindung"
+        "linkOut": "Ausgehende Verbindung",
+        "visibility": "Sichtbarkeit",
+        "visibilityNone": "Nicht"
     },
     "tls": {
         "tls": "TLS-Konfiguration",
diff --git a/packages/node_modules/@node-red/nodes/locales/en-US/common/60-link.html b/packages/node_modules/@node-red/nodes/locales/en-US/common/60-link.html
index 152717fc8..688cec420 100644
--- a/packages/node_modules/@node-red/nodes/locales/en-US/common/60-link.html
+++ b/packages/node_modules/@node-red/nodes/locales/en-US/common/60-link.html
@@ -22,6 +22,8 @@
     <p>The wires between link nodes are only displayed when a link node is selected.
        If there are any wires to other tabs, a virtual node is shown that can be clicked
        on to jump to the appropriate tab.</p>
+    <p>The link visibility makes it possible to hide links from other flows, or from all
+       other links to provide a better overview.</p>
     <p><b>Note: </b>Links cannot be created going into, or out of, a subflow.</p>
 </script>
 
@@ -34,6 +36,8 @@
     <p>When in 'send to all' mode, the wires between link nodes are only displayed when
        the node is selected. If there are any wires to other tabs, a virtual node
        is shown that can be clicked on to jump to the appropriate tab.</p>
+    <p>The link visibility makes it possible to hide links from other flows, or from all
+       other links to provide a better overview.</p>
     <p><b>Note: </b>Links cannot be created going into, or out of, a subflow.</p>
 </script>
 
diff --git a/packages/node_modules/@node-red/nodes/locales/en-US/messages.json b/packages/node_modules/@node-red/nodes/locales/en-US/messages.json
index 7fcd5eadc..b5ea57efc 100644
--- a/packages/node_modules/@node-red/nodes/locales/en-US/messages.json
+++ b/packages/node_modules/@node-red/nodes/locales/en-US/messages.json
@@ -179,6 +179,10 @@
         "timeout": "timeout",
         "linkCallType": "Link Type",
         "staticLinkCall": "Fixed target",
+        "visibility": "Visibility",
+        "visibilityGlobal": "Global",
+        "visibilityFlow": "Flow",
+        "visibilityNone": "None",
         "dynamicLinkCall": "Dynamic target (msg.target)",
         "dynamicLinkLabel": "Dynamic",
         "error": {