cavalion-vcl 1.1.64 → 1.1.67

package/.md

@@ -1,7 +1,7 @@
 * [CHANGELOG.md]() - [README.md]() - [package.json]()
 * [Workspace<vcl>](/Library/vcl-comps/devtools/Workspace$/vcl.js)
 
-# `0519` vcl/Factory
+# `0519` vcl/Factory `#CVLN-20220519-1`
 
 * how is the implicit base determined in `["", {}, []]`
 

package/CHANGELOG.md

@@ -1,6 +1,26 @@
+* ### 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
+* 90bba04 - foolproofing rendering (vcl/ui/List) while reseting array during usage - needs performance boost with splice
+* 89db212 - tweaking vertical-align
+
 ### 2022/06/23 - 1.1.64
 
-* 
+* ed814eb - updating
+* ca79dd6 - preventDefault when clicked A
+* 0e3c6e0 - rendering pending values as -
+* d1aeeae - fixes executesAction enum item No -> no
+* 2239c5b - cosmetic 
 
 ### 2022/05/31 - 1.1.63
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cavalion-vcl",
-  "version": "1.1.64",
+  "version": "1.1.67",
   "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

@@ -718,15 +718,12 @@ define(function(require) {
 				var classes = this.determineClasses();
 				delete this._computedStyle;
 				classes = String.trim(classes.join(" "));
-				//if(classes !== this._node.className) {
+				if(this._node) {
 				    this._node.className = classes;
     				if(this.inDocument()) {
     					this.layoutChanged();
     				}
-				// } else {
-				//     console.trace("Control.applyClasses-blocked", 
-				//          this.hashCode(), this);
-				// }
+				}
 			},
 			toggleClass: function(classes) {
 				if(this._classes_rt === null) {

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

@@ -496,12 +496,14 @@ define(function(require) {
 			setCount: function(value) {
 				if(this._count !== value) {
 					this._count = value;
+					this.setSelection([]);
 					if(value === 0) {
 						this.resetColumnAutoWidth();
 					}
-					this.setSelection([]);
 					//this.alignControls();
 					this.render_();
+					// this.nextTick(() => this._body.updateRows());
+					// value && this._body.updateRows();
 				}
 			},
 

package/src/ui/ListBody.js

@@ -124,7 +124,7 @@ if(firstRow !== 0 && delta === 0) return;
 			},
 			updateRows: function(range) {
 				if(this.hasOwnProperty("_controls")) {
-					for(var i = 0; i < this._controls.length; ++i) {
+					for(var i = 0; i < Math.min(this._controls.length, this._parent.getCount()); ++i) {
 						var c = this._controls[i];
 						if(c._node !== null) {
 							// Update row when the range is unknown or it's rowIndex is within the range

package/src/ui/Tree.js

@@ -64,7 +64,7 @@ define(function (require) {
                         // height: "20px",
                         width: "12px",
                         // "padding-top": "3px",
-                        "vertical-align": "text-top",
+                        // "vertical-align": "text-top",
                         "&::before": {
                             content: "' '",
                             display: "inline-block",