@eclipse-scout/core 22.0.26 → 22.0.33

package/dist/file-list

@@ -1,9 +1,9 @@
-eclipse-scout-core-a56958ac9954d00cc62a.min.js
-eclipse-scout-core-a56958ac9954d00cc62a.min.js.map
-eclipse-scout-core-theme-dark-fd0e080c10f65e67b68b.min.css
+eclipse-scout-core-b11dde8e4ac1367f5a06.min.js
+eclipse-scout-core-b11dde8e4ac1367f5a06.min.js.map
+eclipse-scout-core-theme-dark-0d9d468e5722f73f5075.min.css
 eclipse-scout-core-theme-dark.css
 eclipse-scout-core-theme-dark.css.map
-eclipse-scout-core-theme-f3a61fbc12acf8e27fcc.min.css
+eclipse-scout-core-theme-db0af3fa95956b820bfc.min.css
 eclipse-scout-core-theme.css
 eclipse-scout-core-theme.css.map
 eclipse-scout-core.js

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eclipse-scout/core",
-  "version": "22.0.26",
+  "version": "22.0.33",
   "description": "Eclipse Scout runtime",
   "author": "BSI Business Systems Integration AG",
   "homepage": "https://www.eclipse.org/scout",
@@ -26,12 +26,12 @@
     "src"
   ],
   "devDependencies": {
-    "@eclipse-scout/cli": "22.0.26",
+    "@eclipse-scout/cli": "22.0.33",
     "@eclipse-scout/releng": "^22.0.0",
     "jasmine-core": "3.10.1",
     "jasmine-ajax": "4.0.0",
     "jasmine-jquery": "2.1.1",
-    "karma": "6.3.9"
+    "karma": "6.3.20"
   },
   "dependencies": {
     "jquery": "3.6.0",

package/src/desktop/outline/Outline.less

@@ -322,6 +322,7 @@
       &.selected {
         display: flex;
         flex-wrap: wrap; // Move detail menubar to next line
+        background-clip: content-box; // selection background shines through on the bottom of the node sometimes when zoomed -> makes it slightly better
       }
 
       &.child-of-selected.animate-prev-selected {

package/src/desktop/outline/OutlineLayout.js

@@ -23,7 +23,7 @@ export default class OutlineLayout extends TreeLayout {
 
     super._layout($container);
 
-    containerSize = htmlContainer.availableSize()
+    containerSize = htmlContainer.availableSize({exact: true})
       .subtract(htmlContainer.insets());
 
     if (this.outline.embedDetailContent) {

package/src/desktop/outline/PageLayout.js

@@ -31,7 +31,7 @@ export default class PageLayout extends AbstractLayout {
       detailMenuBarHeight = 0,
       textWidth = 0;
 
-    containerSize = htmlContainer.availableSize()
+    containerSize = htmlContainer.availableSize({exact: true}) // exact is important to calculate text width correctly and to prevent node menubar from wrapping
       .subtract(htmlContainer.insets());
     textWidth = containerSize.width;
 
@@ -93,7 +93,9 @@ export default class PageLayout extends AbstractLayout {
     // needs a width to be able to calculate the pref height
     textHeight = graphics.prefSize($text, {
       includeMargin: true,
-      widthHint: textWidth
+      widthHint: textWidth,
+      enforceSizeHints: true,
+      exact: true
     }).height;
 
     if ($icon.length > 0) {
@@ -106,6 +108,7 @@ export default class PageLayout extends AbstractLayout {
     }
     if (this.outline.detailContent) {
       let htmlDetailContent = this.outline.detailContent.htmlComp;
+      options = $.extend({}, options, {enforceSizeHints: true, exact: true});
       detailContentPrefSize = htmlDetailContent.prefSize(options).add(htmlDetailContent.margins());
     }
 

package/src/layout/graphics.js

@@ -8,7 +8,7 @@
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
  */
-import {arrays, Device, Dimension, Insets, Point, Rectangle, scout, scrollbars} from '../index';
+import {arrays, Dimension, Insets, objects, Point, Rectangle, scout, scrollbars} from '../index';
 import $ from 'jquery';
 
 /**
@@ -35,6 +35,10 @@ import $ from 'jquery';
  *
  * heightHint               undefined       Same as 'widthHint' but for the height.
  *
+ * enforceSizeHints         false           Sets min/max-width/height in addition to with width/height if widthHint resp. heightHint is set.
+ *                                          The browser sometimes makes the element smaller or larger than specified by width/height, especially in a flex container.
+ *                                          To prevent that, set this option to true. Default is false, but may change in the future.
+ *
  * restoreScrollPositions   true            By default, the $elem's scrolling position is saved and restored
  *                                          during the execution of this method (because applying
  *                                          intermediate styles for measurement might change the current
@@ -96,6 +100,16 @@ export function prefSize($elem, options) {
     'width': newWidth,
     'height': newHeight
   };
+  if (scout.nvl(options.enforceSizeHints, false)) {
+    if (objects.isNumber(newWidth)) {
+      cssProperties['max-width'] = newWidth;
+      cssProperties['min-width'] = newWidth;
+    }
+    if (objects.isNumber(newHeight)) {
+      cssProperties['max-height'] = newHeight;
+      cssProperties['min-height'] = newHeight;
+    }
+  }
 
   // modify properties which prevent reading the preferred size
   $elem.css(cssProperties);

package/src/tree/Tree.js

@@ -398,7 +398,7 @@ export default class Tree extends Widget {
     this.scrollToSelection = false;
     let scrollTop = this.$data[0].scrollTop;
     let scrollLeft = this.$data[0].scrollLeft;
-    if (this.scrollTop !== scrollTop) {
+    if (this.scrollTop !== scrollTop && this.rendered) {
       this._renderViewport();
     }
     this.scrollTop = scrollTop;
@@ -1915,9 +1915,7 @@ export default class Tree extends Widget {
   _addToVisibleFlatListNoCheck(node, insertIndex, animatedRendering) {
     arrays.insert(this.visibleNodesFlat, node, insertIndex);
     this.visibleNodesMap[node.id] = true;
-    if (this.rendered) {
-      this.showNode(node, animatedRendering, insertIndex);
-    }
+    this.showNode(node, animatedRendering, insertIndex);
   }
 
   scrollTo(node, options) {
@@ -2154,10 +2152,16 @@ export default class Tree extends Widget {
     }
   }
 
+  /**
+   * @param {TreeNode} [parentNode]
+   */
   insertNode(node, parentNode) {
     this.insertNodes([node], parentNode);
   }
 
+  /**
+   * @param {TreeNode} [parentNode]
+   */
   insertNodes(nodes, parentNode) {
     nodes = arrays.ensure(nodes).slice();
     if (nodes.length === 0) {
@@ -3023,7 +3027,7 @@ export default class Tree extends Widget {
   }
 
   hideNode(node, useAnimation, suppressDetachHandling) {
-    if (!node.attached) {
+    if (!this.rendered || !node.attached) {
       return;
     }
     this.viewRangeDirty = true;