cavalion-vcl 1.1.65 → 1.1.68

package/.md

@@ -1,5 +1,17 @@
 * [CHANGELOG.md]() - [README.md]() - [package.json]()
-* [Workspace<vcl>](/Library/vcl-comps/devtools/Workspace$/vcl.js)
+* [Workspace<vcl>](/Library/vcl-comps/devtools/Workspace$/vcl.js) - [.js]()
+
+# `0728` Control.prototype.makeVisible
+
+Been working on coding in Markdown documents, basically in order to be able print results in the workspace console. Therefore the method `p()` has been introduced. It cancels the behaviour of opening a document while linking a link. Instead it evaluates the code and print its result in the workspace console. 
+
+So, now we can do something like:
+
+* ```[`123`](`${p(:)}`)```  ==>  [`123`](`${p(:)}`) - check the console after clicking 123
+
+However, the console doesn't show yet automatically. Therefore the method `Control.prototype.makeVisible` is introduced. It will call the method `Control.prototype.show` and travel up in the parent-hierarchy and call more in case necessary.
+
+* makeVisible() already exists in vcl/ui/Tab.prototype.
 
 # `0519` vcl/Factory `#CVLN-20220519-1`
 
@@ -57,8 +69,12 @@
 				* [Popup](src/ui/:.js) - [Ace](src/ui/:.js) - [Console](src/ui/:.js)
 				* [Form](src/ui/:.js) - [FormContainer](src/ui/:.js)
 				* [List](src/ui/:.js) | [ListColumn](src/ui/:.js) - [ListBody](src/ui/:.js) - [ListFooter](src/ui/:.js) - [ListHeader](src/ui/:.js) - [ListRow](src/ui/:.js)
-		* [Node](src/ui/:.js) [.closeable](src/ui/Node:.js)
-		* [Sizer](src/ui/:.js) - [Tab](src/ui/:.js) - [Tabs](src/ui/:.js) - [Tree](src/ui/:.js)
+		* [Sizer](src/ui/:.js)
+		* [Tabs](src/ui/:.js) 
+			* [Tab](src/ui/:.js)
+		* [Tree](src/ui/:.js)
+			* [Node](src/ui/:.js)
+				* [.closeable](src/ui/Node:.js)
 
 
 # resources

package/CHANGELOG.md

@@ -1,3 +1,22 @@
+### 2022/07/28 - 1.1.68
+
+* 768156d - fixes an issue where the vertical scrollbar was not accesible on macOS
+* cd60d4b - introduces new methods for vcl/Control: 
+	* reflectStates, 
+	* makeVisible and 
+	* selectVisible 
+* cd60d4b - changing some css/rendering details for Tree and Node 
+
+### 2022/07/04 - 1.1.67
+
+* 7dc04fb - adds Component.prototype.getTimeout
+* 52bebc9 - introducing Query.prototype.error
+* 18cd481 - destroying element upon remove (seems decent)
+
+### 2022/06/27 - 1.1.66
+
+* 07c6e83 - introducing onError event-property
+
 ### 2022/06/27 - 1.1.65
 
 * 52c79ea - fixes bug where applyClasses was called without a _node being allocated/available
@@ -10,7 +29,7 @@
 * ca79dd6 - preventDefault when clicked A
 * 0e3c6e0 - rendering pending values as -
 * d1aeeae - fixes executesAction enum item No -> no
-* 2239c5b - cosmetic (5 days ago) <Ralph Kazemier>
+* 2239c5b - cosmetic 
 
 ### 2022/05/31 - 1.1.63
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cavalion-vcl",
-  "version": "1.1.65",
+  "version": "1.1.68",
   "description": "Visual Component Library for vcl-comps",
   "main": "index.js",
   "scripts": {

package/src/Component.js

@@ -235,7 +235,7 @@ define(function (require) {
             	}
             	return this.setTimeout(name, f, 0, args);
             },
-            setTimeout: function (name, f, ms, args) {
+            setTimeout: function(name, f, ms, args) {
 	            /**
 	             * @param name Used to identify the timeout. Successive calls will cancel a previous timeout with the same name.
 	             * @param f {String/Function} Identifies the function which should be called when at least ms has passed. Optional, when omitted it defaults to the same value as name. A string value to identify a member function or simply a reference to a function.
@@ -301,6 +301,13 @@ define(function (require) {
 
                 return (this._timeouts[name] = js.setTimeout(g, ms, this._timeouts[name]));
             },
+            getTimeout: function(name) {
+	            /**
+	             * @param name Used to identify the timeout. 
+	             * @returns The timeout function set, if any, otherwise undefined
+	             */
+	             return this._timeouts[name];
+            },
             hasTimeout: function(name) {
             	return this._timeouts.hasOwnProperty(name);
             },

package/src/Control.js

@@ -958,6 +958,25 @@ define(function(require) {
 					}
 				}
 			},
+			reflectStates: function(states) {
+				if(typeof states === "string") {
+					states = states.split(",").map(state => ControlState[state]);
+				}
+				
+				states.forEach(state => {
+					switch(state) {
+						case ControlState.showing:
+							this.makeVisible();
+							break;
+							
+						case ControlState.selected:
+							this.setSelected(true);
+							break;
+					}
+					
+				})
+				
+			},
 
 			// scrollIntoView: function() {
 			// 	this.nodeNeeded();
@@ -1084,6 +1103,21 @@ define(function(require) {
 			hide: function() {
 				this.setVisible(false);
 			},
+			makeVisible: function(control) {
+				var parent = this._parent;
+				if(this._parent && !this._parent.isVisible()) {
+					this._parent.makeVisible();
+					this._parent.update(() => this.makeVisible());
+					return;
+				}
+				if(!this.isVisible()) {
+					this.show();
+				}
+			},
+			selectVisible: function() {
+				this.setSelected(true);
+				this.makeVisible();
+			},
 
 			allowsUpdateChildren: function() {
 				return this.isShowing();

package/src/entities/Query.js

@@ -47,6 +47,18 @@ define(function(require) {
 				this._monitors = [];
 				this._pageReqs = [];
 			},
+			error: function(e) {
+				if(this._onError) {
+					this._onError(e);
+				} else {
+					if(e && e.responseJSON && e.responseJSON.message) {
+						console.error(e.responseJSON.message, e);
+						throw e;
+					}
+					console.error(e);
+					throw e;
+				}
+			},
 			loaded: function() {
 			/** @overrides ../vcl/Compopnent.prototype.loaded */
 				return this.inherited(arguments);
@@ -284,12 +296,7 @@ define(function(require) {
 						}
 						return res;
 					}).catch(function(e) {
-						if(e && e.responseJSON && e.responseJSON.message) {
-							console.error(e.responseJSON.message, e)
-							throw e;
-						}
-						console.error(e);
-						throw e;
+						me.error(e);
 					})
 				);
 				
@@ -602,6 +609,12 @@ define(function(require) {
     				/*	This method provides an interface to dynamically determine the request criteria, based upon for example user input. This method must return a criteria object. The function receives the criteria as they are currently indicated by its properties. Return null to prevent the request from going out.
     				*/
     			}
+    		},
+    		"onError": {
+    			type: Type.EVENT,
+    			f: function(e) {
+    				/*	This method provides an interface to handle errors, if not set an error will be thrown and logged to the console */
+    			}
     		}
     	}
     }));

package/src/prototypes/App.toast.js

@@ -34,7 +34,7 @@
 				elem.setTimeout("disappear", () => {
 					var h = elem.on("transitionend", () => {
 						elem.un(h);
-						elem.setParent(null);
+						elem.destroy();
 					});
 					elem.replaceClass("appear", "disappear");
 				}, timeout_ !== undefined ? timeout_ : timeout);

package/src/ui/List.js

@@ -58,7 +58,8 @@ define(function(require) {
 						position: "absolute",
 						width: "1px",
 						height: "1px",
-						overflow: "hidden"
+						overflow: "hidden",
+						'z-index': 2
 					}
 				},
 				"&.header-invisible .{./ListHeader}": "height:0;"

package/src/ui/Tab.js

@@ -168,7 +168,14 @@ define(function(require) {
 					this.setState("invalidated", true);
 				}
 			},
-		
+			
+			makeVisible: function(control) {
+				this.inherited(arguments);
+				if(!control && this._parent) {
+					this._parent.makeVisible(this);
+				}
+			},
+
 			getControl: function() {
 				return this._control;
 			},

package/src/ui/Tabs.js

@@ -210,17 +210,21 @@ define(function(require) {
 				// }
 	   // 	},
 	    	makeVisible: function(control) {
-	    		/*- this assumes horizontal scrolling only */
-	    	    control.scrollIntoView(); 
-	    	    // HACK
-	    		this._node.scrollTop = 0;
-	    	    this.nextTick("position-scrollbar", function() {
-		    		if(this._node.scrollLeft < 100) {
-		    			this._node.scrollLeft = 0;
-		    		} else {
-		    			this._node.scrollLeft += 100;
-		    		}
-	    	    }.bind(this));
+	    		if(control) {
+		    		/*- this assumes horizontal scrolling only */
+		    	    control.scrollIntoView(); 
+		    	    // HACK
+		    		this.nodeNeeded().scrollTop = 0;
+		    	    this.nextTick("position-scrollbar", function() {
+			    		if(this._node.scrollLeft < 100) {
+			    			this._node.scrollLeft = 0;
+			    		} else {
+			    			this._node.scrollLeft += 100;
+			    		}
+		    	    }.bind(this));
+	    		} else {
+					return this.inherited(arguments);
+	    		}
 	    	},
     		initializeNodes: function(control) {
 	    		/** @overrides ../Control.prototype.initializeNodes */

package/src/ui/Tree.js

@@ -74,13 +74,14 @@ define(function (require) {
                         }
                     },
                     ">.text": {
-                        "input[type=checkbox]": `position:relative;top:${checkbox_top}px;`,
+                        'input[type=checkbox]': `position:relative;top:${checkbox_top}px;`,
                     	cursor: "pointer",
                         position: "relative",
 //        				width: "100%",
                         display: 'inline-block',
-                        "margin-left": "2px",
+                        'margin-left': "2px",
                         padding: "3px 4px 3px 4px",
+                        // 'vertical-align': "top"
                     },
                     "&.selected": {
                         ">.selection": {
@@ -231,10 +232,14 @@ define(function (require) {
                 this.dispatch("nodesneeded", null);
             },
 			makeVisible: function(childNode) {
-			    var node = this.nodeNeeded();
-			    var pos = jquery(childNode.nodeNeeded()).position();
-			    var top = this.getAbsoluteRect().height / 3;
-			    node.scrollTop -= (top - pos.top);
+				if(childNode) {
+				    var node = this.nodeNeeded();
+				    var pos = jquery(childNode.nodeNeeded()).position();
+				    var top = this.getAbsoluteRect().height / 3;
+				    node.scrollTop -= (top - pos.top);
+				} else {
+					return this.inherited(arguments);
+				}
             },
             
             do_keyup: function(evt) {