@node-red/editor-client 4.1.2 → 4.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@node-red/editor-client",
-    "version": "4.1.2",
+    "version": "4.1.3",
     "license": "Apache-2.0",
     "repository": {
         "type": "git",

package/public/red/about

@@ -1,5 +1,38 @@
-#### 4.1.2: Maintenance Release
+#### 4.1.3: Maintenance Release
+
+Editor
+
+ - 5343/Editor/Bug: Node help tab resets focus when arrow keys are used to switch between nodes (#5406) @piotrbogun
+ - Ensure quick-add filter is applied properly when retriggering add (#5427) @knolleary
+ - TreeList: Fix widget treeList keyboard navigation scroll behavior (#5421) @piotrbogun
+ - Editor: Flow & subflow names are changed to all lowercase in search dialog #5348 (#5401) @n-lark
+ - Allow actions show-next-tab and previous to loop (#5355) @GogoVega
+ - 5404/Editor/Bug: Junction error in Quick Add dialog (#5407) @piotrbogun
+ - Add tooltip to delete button in node property UI (#5410) @kazuhitoyokoi
+ - Fix invalid node size in quick add dialog (#5403) @kazuhitoyokoi
+ - Expand folder to avoid error in library (#5399) @kazuhitoyokoi
+ - Stricter validator for flow file name in project feature (#5398) @kazuhitoyokoi
+ - Fix size and scrolling in Git config UI (#5396) @kazuhitoyokoi
+ - Reveal node in search results via mouseover (#5368) @gorenje
+
+Runtime
 
+ - Add package-lock.json for reproducible dependency chains (#5426) @dimitrieh
+ - Readme markdown refactor for legibility in IDE's (#5423) @dimitrieh
+ - Update body-parser (#5418) @knolleary
+
+Nodes
+
+ - fix(http-request): prevent uncaught exceptions in async hooks (#5392) @Dennis-SEG
+ - Fix flushing when in variable delay mode (#5382) @dceejay
+ - File node TypedInput width fix (#5425) @knolleary
+ - Use TextDecoder() to decode UTF-8 characters (#5416) @kazuhitoyokoi
+ - Support source information in complete node (#5414) @kazuhitoyokoi
+ - Fix status node to retrieve status from all nodes (#5412) @kazuhitoyokoi
+ - Decrement count of http requests after error (#5409) @kazuhitoyokoi
+ - Fix debug tab to copy displayed value (#5400) @kazuhitoyokoi
+
+#### 4.1.2: Maintenance Release
 
 Editor
 

package/public/red/red.js

@@ -10047,7 +10047,13 @@ RED.utils = (function() {
         var copyPayload = $('<button class="red-ui-button red-ui-button-small"><i class="fa fa-clipboard"></i></button>').appendTo(copyTools).on("click", function(e) {
             e.preventDefault();
             e.stopPropagation();
-            RED.clipboard.copyText(msg,copyPayload,"clipboard.copyMessageValue");
+            var payloadToCopy;
+            if (typeof msg === "number") {
+                payloadToCopy = obj.find(".red-ui-debug-msg-type-number").first().text();
+            } else {
+                payloadToCopy = msg;
+            }
+            RED.clipboard.copyText(payloadToCopy, copyPayload, "clipboard.copyMessageValue");
         })
         RED.popover.tooltip(copyPayload,RED._("node-red:debug.sidebar.copyPayload"));
         if (enablePinning && strippedKey !== undefined && strippedKey !== '') {
@@ -10375,7 +10381,7 @@ RED.utils = (function() {
                         var sr = $('<div class="red-ui-debug-msg-object-entry collapsed"></div>').appendTo(stringRow);
                         var stringEncoding = "";
                         try {
-                            stringEncoding = String.fromCharCode.apply(null, new Uint16Array(data))
+                            stringEncoding = new TextDecoder().decode(new Uint8Array(data));
                         } catch(err) {
                             console.log(err);
                         }
@@ -11668,8 +11674,10 @@ RED.utils = (function() {
                 var deleteButton = $('<a/>',{href:"#",class:"red-ui-editableList-item-remove red-ui-button red-ui-button-small"}).appendTo(li);
                 $('<i/>',{class:"fa fa-remove"}).appendTo(deleteButton);
                 li.addClass("red-ui-editableList-item-removable");
+                var removeTip = RED.popover.tooltip(deleteButton, RED._("common.label.delete"));
                 deleteButton.on("click", function(evt) {
                     evt.preventDefault();
+                    removeTip.close();
                     var data = row.data('data');
                     li.addClass("red-ui-editableList-item-deleting")
                     li.fadeOut(300, function() {
@@ -11977,7 +11985,10 @@ RED.utils = (function() {
                     } else {
                         that._topList.find(".focus").removeClass("focus")
                     }
-                    target.treeList.label.addClass('focus')
+                    if (target.treeList.label) {
+                        target.treeList.label.addClass('focus')
+                    }
+                    that.reveal(target);
                 }
             });
             this._data = [];
@@ -12668,6 +12679,10 @@ RED.utils = (function() {
             }
 
             that._topList.find(".focus").removeClass("focus");
+            
+            if (item.treeList.label) {
+                item.treeList.label.addClass("focus");
+            }
 
             if (triggerEvent !== false) {
                 this._trigger("select",null,item)
@@ -14767,12 +14782,18 @@ RED.tabs = (function() {
         }
         function activatePreviousTab() {
             var previous = findPreviousVisibleTab();
+            if (previous.length === 0) {
+                previous = ul.find("li.red-ui-tab:not(.hide-tab)").last();
+            }
             if (previous.length > 0) {
                 activateTab(previous.find("a"));
             }
         }
         function activateNextTab() {
             var next = findNextVisibleTab();
+            if (next.length === 0) {
+                next = ul.find("li.red-ui-tab:not(.hide-tab)").first();
+            }
             if (next.length > 0) {
                 activateTab(next.find("a"));
             }
@@ -24023,6 +24044,13 @@ RED.view = (function() {
                 quickAddLink.virtualLink = true;
             }
             hideDragLines();
+        } else if (quickAddLink) {
+            // continuing an existing quick add - set the filter accordingly
+            if (quickAddLink.portType === PORT_TYPE_OUTPUT) {
+                filter = {input:true}
+            } else {
+                filter = {output:true}
+            }
         }
         if (linkToSplice || spliceMultipleLinks) {
             filter = {
@@ -29190,6 +29218,9 @@ RED.view = (function() {
                     suggestedNodes = [suggestedNodes]
                 }
                 suggestedNodes = suggestedNodes.filter(n => {
+                    if (n.type === 'junction') {                        
+                        return true
+                    }
                     const def = RED.nodes.getType(n.type)
                     if (def?.set && def.set.enabled === false) {
                         // Exclude disabled node set
@@ -47504,7 +47535,7 @@ RED.library = (function() {
                                 icon: 'fa fa-cube',
                                 label: options.type,
                                 path: "",
-                                expanded: false,
+                                expanded: true,
                                 children: function(done, item) {
                                     loadLibraryFolder(lib.id, options.url, "", function(children) {
                                         item.children = children;
@@ -48447,9 +48478,13 @@ RED.search = (function() {
     function indexNode(n) {
         var l = RED.utils.getNodeLabel(n);
         if (l) {
-            l = (""+l).toLowerCase();
-            index[l] = index[l] || {};
-            index[l][n.id] = {node:n,label:l}
+            const originalLabel = "" + l;
+            const indexLabel = originalLabel.toLowerCase();
+            index[indexLabel] = index[indexLabel] || {};
+            index[indexLabel][n.id] = {
+                node: n,
+                label: originalLabel
+            };
         }
         l = l||n.label||n.name||n.id||"";
 
@@ -48838,6 +48873,12 @@ RED.search = (function() {
                     $('<div>',{class:"red-ui-search-result-node-type"}).text(node.type).appendTo(contentDiv);
                     $('<div>',{class:"red-ui-search-result-node-id"}).text(node.id).appendTo(contentDiv);
 
+                    div.on("mouseover", function(evt) {
+                        if ( node.z == RED.workspaces.active() ) {
+                            RED.view.reveal(node.id)
+                        }
+                    });
+                    
                     div.on("click", function(evt) {
                         evt.preventDefault();
                         currentIndex = i;
@@ -49075,10 +49116,20 @@ RED.search = (function() {
         show: show,
         hide: hide,
         search: search,
-        getSearchOptions: getSearchOptions
+        getSearchOptions: getSearchOptions,
+        // Expose internals for testing
+        _indexNode: indexNode,
+        get _index() { return index; },
+        set _index(val) { index = val; }
     };
 
 })();
+
+
+// Allow CommonJS import for testing
+if (typeof module !== "undefined" && module.exports) {
+    module.exports = RED.search;
+}
 ;RED.contextMenu = (function () {
 
     let menu;
@@ -53387,7 +53438,7 @@ RED.projects = (function() {
                         var validateForm = function() {
                             var valid = true;
                             var flowFile = projectFlowFileInput.val();
-                            if (flowFile === "" || !/\.json$/.test(flowFile)) {
+                            if (flowFile === "" || !/^[a-zA-Z0-9\-_]+\.json$/.test(flowFile)) {
                                 valid = false;
                                 if (!projectFlowFileInput.hasClass("input-error")) {
                                     projectFlowFileInput.addClass("input-error");
@@ -53809,7 +53860,7 @@ RED.projects = (function() {
 
                             } else if (projectType === 'empty') {
                                 var flowFile = projectFlowFileInput.val();
-                                if (flowFile === "" || !/\.json$/.test(flowFile)) {
+                                if (flowFile === "" || !/^[a-zA-Z0-9\-_]+\.json$/.test(flowFile)) {
                                     valid = false;
                                     if (!projectFlowFileInput.hasClass("input-error")) {
                                         projectFlowFileInput.addClass("input-error");