@qooxdoo/framework 7.9.2 → 7.9.3

package/source/class/qx/tool/compiler/resources/Asset.js

@@ -167,7 +167,7 @@ qx.Class.define("qx.tool.compiler.resources.Asset", {
 
     async load() {
       if (this.__loaders) {
-        this.__loaders.forEach(loader => loader.load(this));
+        await Promise.all(this.__loaders.map(loader => loader.load(this)));
       }
     },
 

package/source/class/qx/tool/compiler/targets/meta/PackageJavascript.js

@@ -85,12 +85,16 @@ qx.Class.define("qx.tool.compiler.targets.meta.PackageJavascript", {
 
       if (pkg.isEmbedAllJavascript()) {
         this.__sourceMapOffsets = [];
+        let packageWs = new qx.tool.utils.Utils.LineCountingTransform();
         let strip = new qx.tool.utils.Utils.StripSourceMapTransform();
-        strip.pipe(ws);
+        strip.pipe(packageWs);
+        packageWs.pipe(ws, {
+          end: false
+        });
         await new Promise(async resolve => {
           for (let i = 0; i < pkg.getJavascriptMetas().length; i++) {
             let js = pkg.getJavascriptMetas()[i];
-            this.__sourceMapOffsets.push(ws.getLineNumber());
+            this.__sourceMapOffsets.push(packageWs.getLineNumber());
             await js.unwrap().writeSourceCodeToStream(strip);
             strip.write("\n");
           }

package/source/class/qx/ui/core/Blocker.js

@@ -132,6 +132,7 @@ qx.Class.define("qx.ui.core.Blocker", {
   members: {
     __blocker: null,
     __blockerCount: 0,
+    __blockingContent: false,
 
     __activeElements: null,
     __focusElements: null,
@@ -186,11 +187,17 @@ qx.Class.define("qx.ui.core.Blocker", {
      * @param bounds {Map} Map with the new width, height, left and top values
      */
     _updateBlockerBounds(bounds) {
+      // When blocking content, the blocker is a child of the widget itself,
+      // so it should be positioned at 0,0 relative to the widget.
+      // Otherwise, it's positioned relative to the layout parent.
+      var left = this.__blockingContent ? 0 : bounds.left;
+      var top = this.__blockingContent ? 0 : bounds.top;
+
       this.getBlockerElement().setStyles({
         width: bounds.width + "px",
         height: bounds.height + "px",
-        left: bounds.left + "px",
-        top: bounds.top + "px"
+        left: left + "px",
+        top: top + "px"
       });
     },
 
@@ -338,7 +345,7 @@ qx.Class.define("qx.ui.core.Blocker", {
         if (!this.__appearListener) {
           this.__appearListener = this._widget.addListenerOnce(
             "appear",
-            this._block.bind(this, zIndex)
+            this._block.bind(this, zIndex, blockContent)
           );
         }
         return;
@@ -358,6 +365,9 @@ qx.Class.define("qx.ui.core.Blocker", {
 
       this.__blockerCount++;
       if (this.__blockerCount < 2) {
+        // Track if we're blocking content (blocker is child of widget)
+        this.__blockingContent = blockContent === true;
+
         this._backupActiveWidget();
 
         var bounds = this._widget.getBounds();
@@ -442,6 +452,9 @@ qx.Class.define("qx.ui.core.Blocker", {
       blocker.removeListener("keyup", this.__stopTabEvent, this);
       blocker.exclude();
 
+      // Reset blocking content state
+      this.__blockingContent = false;
+
       this.fireEvent("unblocked", qx.event.type.Event);
     },
 

package/source/class/qx/ui/form/validation/Manager.js

@@ -484,7 +484,7 @@ qx.Class.define("qx.ui.form.validation.Manager", {
 
             let msg = item.getInvalidMessage();
             if (
-              msg &&
+              !msg &&
               qx.core.Environment.get(
                 "qx.ui.form.validation.Manager.allowDefaultInvalidMessage"
               )

package/source/class/qx/ui/mobile/dialog/Popup.js

@@ -140,11 +140,16 @@ qx.Class.define("qx.ui.mobile.dialog.Popup", {
     __widget: null,
     __titleWidget: null,
     __lastPopupDimension: null,
+    /**
+     * Flag which ignores event domupdated caused by this widget and stopps infinite recursive `_updatePosition` calls
+     */
+    __updatePositionStarted: false,
 
     /**
      * Event handler. Called whenever the position of the popup should be updated.
      */
     _updatePosition() {
+      this.__updatePositionStarted = true;
       // Traverse single anchor classes for removal, for preventing 'domupdated' event if no CSS classes changed.
       var anchorClasses = ["top", "bottom", "left", "right", "anchor"];
       for (var i = 0; i < anchorClasses.length; i++) {
@@ -446,7 +451,14 @@ qx.Class.define("qx.ui.mobile.dialog.Popup", {
         flex: 1
       });
 
-      widget.addListener("domupdated", this._updatePosition, this);
+      widget.addListener("domupdated", () => {
+        if (!this.__updatePositionStarted){
+          this._updatePosition();
+        }
+        else {
+          this.__updatePositionStarted = false;
+        }
+      }, this);
 
       this.__widget = widget;
     },

package/source/class/qx/ui/table/Table.js

@@ -1570,10 +1570,11 @@ qx.Class.define("qx.ui.table.Table", {
      *          visible.
      */
     setFocusedCell(col, row, scrollVisible) {
-      if (
-        !this.isEditing() &&
-        (col != this.__focusedCol || row != this.__focusedRow)
-      ) {
+      let cellChanged = col != this.__focusedCol || row != this.__focusedRow;
+      if (this.isEditing() && cellChanged) {
+        this.stopEditing();
+      }
+      if (!this.isEditing() && cellChanged) {
         if (col === null) {
           col = 0;
         }

package/source/class/qx/ui/tree/VirtualTree.js

@@ -761,7 +761,10 @@ qx.Class.define("qx.ui.tree.VirtualTree", {
           }
         }
 
-        if (this.__lookupTable.indexOf(item) != -1) {
+        // Issue #9390: When hideRoot is true, the root is not in the lookup table,
+        // but we still need to update when its children change
+        var isRootAndHidden = this.isHideRoot() && item === this.getModel();
+        if (this.__lookupTable.indexOf(item) != -1 || isRootAndHidden) {
           this.__applyModelChanges();
         }
       }

package/source/class/qx/util/format/DateFormat.js

@@ -476,8 +476,9 @@ qx.Class.define("qx.util.format.DateFormat", {
 
       var timezoneOffset = date.getTimezoneOffset();
       var timezoneSign = timezoneOffset > 0 ? 1 : -1;
-      var timezoneHours = Math.floor(Math.abs(timezoneOffset) / 60);
-      var timezoneMinutes = Math.abs(timezoneOffset) % 60;
+      var absTimezoneOffset = Math.abs(timezoneOffset);
+      var timezoneHours = Math.floor(absTimezoneOffset / 60);
+      var timezoneMinutes = Math.trunc(absTimezoneOffset) % 60;
 
       // Create the output
       this.__initFormatTree();