cavalion-vcl 1.1.63 → 1.1.66

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,3 +1,21 @@
+### 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 (5 days ago) <Ralph Kazemier>
+
 ### 2022/05/31 - 1.1.63
 
 * 837510b - fixes document.title bug 

package/package.json

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

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;
+						this.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/ui/Input.js

@@ -305,7 +305,7 @@ define(function(require) {
 		properties: {
 			"executesAction": {
 				/** @overrides ../Element.properties.executesAction */
-				type: ["No", "onClick", "onEnterPressed"]
+				type: ["no", "onClick", "onEnterPressed"]
 			},
 			"detectChangeTimeout": {
 				type: Type.INTEGER

package/src/ui/List.js

@@ -404,8 +404,10 @@ define(function(require) {
 				if(column._attribute !== "") {
 					orgValue = (value = this._source.getAttributeValue(column._attribute, row));
 				}
-
-				if(column._wantsNullValues || (value !== null && value !== undefined)) {
+				
+				if(value === Source.Pending) {
+					value = "-";
+				} else if(column._wantsNullValues || (value !== null && value !== undefined)) {
 					if(column._displayFormat !== "") {
 						value = String.format(column._displayFormat, value);
 					}
@@ -494,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/Popup.js

@@ -33,7 +33,7 @@ define(function(require) {
 						if(!HtmlElement.hasParent(evt.target, popup._node)) {
 							popup.close();
 							evt.bubbleUp = false;
-							//evt.preventDefault();
+							evt.preventDefault(); // for A's
 						}
 					}
 				});

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",