@eclipse-scout/core 22.0.11 → 22.0.19

package/dist/file-list

@@ -1,9 +1,9 @@
-eclipse-scout-core-e5e8740e3649f5b9f279.min.js
-eclipse-scout-core-e5e8740e3649f5b9f279.min.js.map
-eclipse-scout-core-theme-74b63e0d57bed407a729.min.css
-eclipse-scout-core-theme-dark-b82aea152f416e38ce7f.min.css
+eclipse-scout-core-d110a434bcc809661ce5.min.js
+eclipse-scout-core-d110a434bcc809661ce5.min.js.map
+eclipse-scout-core-theme-dark-fd0e080c10f65e67b68b.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.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.11",
+  "version": "22.0.19",
   "description": "Eclipse Scout runtime",
   "author": "BSI Business Systems Integration AG",
   "homepage": "https://www.eclipse.org/scout",
@@ -26,7 +26,7 @@
     "src"
   ],
   "devDependencies": {
-    "@eclipse-scout/cli": "22.0.11",
+    "@eclipse-scout/cli": "22.0.19",
     "@eclipse-scout/releng": "^22.0.0",
     "jasmine-core": "3.10.1",
     "jasmine-ajax": "4.0.0",

package/src/App.js

@@ -62,6 +62,7 @@ export default class App {
    * @param {string|string[]} [options.bootstrap.textsUrl] URL or multiple URLs pointing to a json resource containing texts that will be available through texts.js.
    * @param {string} [options.bootstrap.localesUrl] URL pointing to a json resource containing locale information processed by locales.js
    * @param {string} [options.bootstrap.codesUrl] URL pointing to a json resources containing codes that will be available through codes.js
+   * @param {object} [options.session] Object to configure the session, see {@link Session.init} for the available options.
    */
   init(options) {
     options = options || {};

package/src/desktop/DesktopDense.less

@@ -1,9 +1,9 @@
 /*
- * Copyright (c) 2010-2021 BSI Business Systems Integration AG.
+ * Copyright (c) 2010-2022 BSI Business Systems Integration AG.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
- * http://www.eclipse.org/legal/epl-v10.html
+ * https://www.eclipse.org/legal/epl-v10.html
  *
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
@@ -50,7 +50,8 @@
 
   .menubar:not(.main-menubar) {
     & > .menubar-box {
-      & > .menu-item {
+      & > .menu-item,
+      & > .combo-menu > .menu-item {
         padding-top: 3px; /* @menubar-item-padding-y; */
         padding-bottom: 3px; /* @menubar-item-padding-y; */
 
@@ -58,10 +59,22 @@
         min-height: @logical-grid-row-height-dense; /* @logical-grid-row-height; */
       }
 
-      & > .menu-button {
+      & > .menu-button,
+      & > .combo-menu > .menu-button {
         padding-top: 2px;
         padding-bottom: 2px;
       }
+
+      & > .combo-menu {
+        padding: 0;
+      }
+
+      & > .form-field-menu {
+        padding-top: 0;
+        padding-bottom: 0;
+        margin-top: 0;
+        margin-bottom: 0;
+      }
     }
   }
 
@@ -155,14 +168,18 @@
     }
   }
 
-
   /* ContextMenuPopup.less */
 
   .context-menu {
-    & > .menu-item {
+    & > .menu-item,
+    & > .combo-menu > .menu-item {
       padding-top: 6px; /* padding: 11px 15px; */
       padding-bottom: 6px;
     }
+
+    & > .combo-menu {
+      padding: 0;
+    }
   }
 
   /* CheckBoxField.less */
@@ -279,6 +296,7 @@
   }
 
   /* BreadcrumbItem.less */
+
   .breadcrumb-item {
     font-size: @font-size-normal;
   }

package/src/desktop/OpenUriHandler.js

@@ -1,9 +1,9 @@
 /*
- * Copyright (c) 2014-2018 BSI Business Systems Integration AG.
+ * Copyright (c) 2010-2022 BSI Business Systems Integration AG.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
- * http://www.eclipse.org/legal/epl-v10.html
+ * https://www.eclipse.org/legal/epl-v10.html
  *
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
@@ -50,7 +50,7 @@ export default class OpenUriHandler {
       } else {
         this.openUriAsNewWindow(uri);
       }
-    } else if (Device.get().browser === Device.Browser.CHROME) {
+    } else if (Device.get().browser === Device.Browser.CHROME && this.isUriWithExternallyHandledProtocol(uri)) {
       // "Hidden iframe"-solution is not working in Chromium (https://bugs.chromium.org/p/chromium/issues/detail?id=663325)
       this.openUriInSameWindow(uri);
     } else {
@@ -59,7 +59,7 @@ export default class OpenUriHandler {
   }
 
   isUriWithExternallyHandledProtocol(uri) {
-    return /^(callto|facetime|fax|geo|mailto|maps|notes|sip|skype|tel|google.navigation):/.test(uri);
+    return /^(callto|facetime|fax|geo|mailto|maps|notes|sip|skype|tel|google.navigation|sms|msteams):/.test(uri);
   }
 
   handleUriActionOpen(uri) {
@@ -69,7 +69,7 @@ export default class OpenUriHandler {
       // Additionally, some url types require to be opened in the same window like tel or mailto, at least on mobile devices
       this.openUriInSameWindow(uri);
     } else if (this.isUriWithExternallyHandledProtocol(uri)) {
-      if (Device.get().browser === Device.Browser.CHROME) {
+      if (Device.get().browser === Device.Browser.CHROME || Device.get().isAndroid()) {
         // "Hidden iframe"-solution is not working in Chromium (https://bugs.chromium.org/p/chromium/issues/detail?id=663325)
         this.openUriInSameWindow(uri);
       } else {

package/src/form/fields/FormField.js

@@ -1134,7 +1134,7 @@ export default class FormField extends Widget {
     if (!this.fieldStatus) {
       return;
     }
-    this.fieldStatus.remove();
+    this.fieldStatus.destroy();
     this.$status = null;
     this.fieldStatus = null;
   }

package/src/form/fields/datefield/DateField.js

@@ -205,6 +205,7 @@ export default class DateField extends ValueField {
       this.htmlDateTimeComposite.invalidateLayoutTree();
     }
     this._renderDateClearable();
+    this.$container.toggleClass('has-date', this.hasDate);
   }
 
   setHasTime(hasTime) {
@@ -257,6 +258,7 @@ export default class DateField extends ValueField {
       this.htmlDateTimeComposite.invalidateLayoutTree();
     }
     this._renderTimeClearable();
+    this.$container.toggleClass('has-time', this.hasTime);
   }
 
   setTimePickerResolution(timePickerResolution) {

package/src/form/fields/sequencebox/SequenceBox.js

@@ -1,9 +1,9 @@
 /*
- * Copyright (c) 2010-2021 BSI Business Systems Integration AG.
+ * Copyright (c) 2010-2022 BSI Business Systems Integration AG.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
- * http://www.eclipse.org/legal/epl-v10.html
+ * https://www.eclipse.org/legal/epl-v10.html
  *
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
@@ -19,6 +19,8 @@ export default class SequenceBox extends CompositeField {
     this.logicalGrid = scout.create('scout.HorizontalGrid');
     this.layoutConfig = null;
     this.fields = [];
+    this._fieldPropertyChangeHandler = this._onFieldPropertyChange.bind(this);
+    this._lastVisibleFieldSuppressStatusHandler = this._onLastVisibleFieldSuppressStatusChange.bind(this);
   }
 
   _init(model) {
@@ -65,7 +67,7 @@ export default class SequenceBox extends CompositeField {
     for (i = 0; i < this.fields.length; i++) {
       field = this.fields[i];
       field.labelUseUiWidth = true;
-      field.on('propertyChange', this._onFieldPropertyChange.bind(this));
+      field.on('propertyChange', this._fieldPropertyChangeHandler);
       field.render(this.$field);
       this._modifyLabel(field);
 
@@ -83,6 +85,14 @@ export default class SequenceBox extends CompositeField {
     return new SequenceBoxLayout(this, this.layoutConfig);
   }
 
+  _remove() {
+    this.fields.forEach(f => f.off('propertyChange', this._fieldPropertyChangeHandler));
+    if (this._lastVisibleField) {
+      this._lastVisibleField.off('propertyChange:suppressStatus', this._lastVisibleFieldSuppressStatusHandler);
+    }
+    super._remove();
+  }
+
   /**
    * @override
    */
@@ -137,6 +147,7 @@ export default class SequenceBox extends CompositeField {
   _handleStatus(visibilityChanged) {
     if (visibilityChanged && this._lastVisibleField) {
       // if there is a new last visible field, make sure the status is shown on the previously last one
+      this._lastVisibleField.off('propertyChange:suppressStatus', this._lastVisibleFieldSuppressStatusHandler);
       this._lastVisibleField.setSuppressStatus(null);
       if (this._lastVisibleField.rendered) {
         this._lastVisibleField._renderErrorStatus();
@@ -184,6 +195,7 @@ export default class SequenceBox extends CompositeField {
 
     // Make sure the last field won't display a status (but shows status CSS class)
     this._lastVisibleField.setSuppressStatus(FormField.SuppressStatus.ICON);
+    this._lastVisibleField.on('propertyChange:suppressStatus', this._lastVisibleFieldSuppressStatusHandler);
     if (visibilityChanged) {
       // If the last field got invisible, make sure the new last field does not display a status anymore (now done by the seq box)
       if (this._lastVisibleField.rendered) {
@@ -194,6 +206,11 @@ export default class SequenceBox extends CompositeField {
     }
   }
 
+  _onLastVisibleFieldSuppressStatusChange(e) {
+    // do not change suppressStatus
+    e.preventDefault();
+  }
+
   setErrorStatus(errorStatus) {
     if (this._isOverwritingStatusFromField && !this._isErrorStatusOverwritten) {
       // was not overwritten, will be overwritten now -> backup old value

package/src/form/fields/stringfield/StringField.js

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2014-2018 BSI Business Systems Integration AG.
+ * Copyright (c) 2010-2022 BSI Business Systems Integration AG.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
@@ -255,6 +255,14 @@ export default class StringField extends BasicField {
     this.revalidateLayout();
   }
 
+  /**
+   * @override
+   */
+  _renderEnabled() {
+    super._renderEnabled();
+    this.revalidateLayout();
+  }
+
   setFormatUpper(formatUpper) {
     if (formatUpper) {
       this.setFormat(StringField.Format.UPPER);

package/src/session/Session.js

@@ -100,36 +100,39 @@ export default class Session {
   static EMPTY_UPLOAD_FILENAME = '*empty*';
 
   /**
-   * $entryPoint is required to create a new session.
-   *
-   * @param model The 'model' argument holds all optional values that may be used during
-   * initialization (it is the same object passed to the scout.init() function).
-   * @param [model.portletPartId]
-   *     Optional, default is 0. Necessary when multiple UI sessions are managed
-   *     by the same window (portlet support). Each session's partId must be unique.
-   * @param model.clientSessionId
+   * @param {$} model.$entryPoint
+   *     The HTML element that is used by the {@link Desktop} to render its content.
+   * @param {string} [model.portletPartId]
+   *     Necessary when multiple UI sessions are managed by the same window (portlet support).
+   *     Each session's partId must be unique. Default is 0.
+   * @param {string} [model.clientSessionId]
    *     Identifies the 'client instance' on the UI server. If the property is not set
    *     (which is the default case), the clientSessionId is taken from the browser's
    *     session storage (per browser window, survives F5 refresh of page). If no
    *     clientSessionId can be found, a new one is generated on the server.
-   * @param [model.userAgent]
-   *     Default: DESKTOP
-   * @param [model.backgroundJobPollingEnabled]
+   * @param {boolean} [model.forceNewClientSession]
+   *     If set to true, the stored or passed clientSessionId will be ignored
+   *     and a new one generated by the server.
+   * @param {UserAgent} [model.userAgent]
+   *     By default the user agent for the running platform is used. Use this option if you want to set a custom one.
+   * @param {Locale|object} [model.locale]
+   *     If not specified, {@link Locale.DEFAULT} is used.
+   * @param {boolean} [model.backgroundJobPollingEnabled]
    *     Unless websockets is used, this property turns on (default) or off background
    *     polling using an async ajax call together with setTimeout()
-   * @param [model.suppressErrors]
+   * @param {boolean} [model.suppressErrors]
    *     Basically added because of Jasmine-tests. When working with async tests that
    *     use setTimeout(), sometimes the Jasmine-Maven plug-in fails and aborts the
    *     build because there were console errors. These errors always happen in this
    *     class. That's why we can skip suppress error handling with this flag.
-   * @param [model.focusManagerActive]
+   * @param {boolean} [model.focusManagerActive]
    *     Forces the focus manager to be active or not. If undefined, the value is
    *     auto detected by Device.js.
-   * @param [model.reconnectorOptions]
-   *     Optional, properties of this object are copied to the Session's reconnector
-   *     instance (see Reconnector.js).
-   * @param [model.ajaxCallOptions]
-   *     Optional, properties of this object are copied to all instances of AjaxCall.js.
+   * @param {object} [model.reconnectorOptions]
+   *     Properties of this object are copied to the Session's reconnector
+   *     instance (see {@link Reconnector}).
+   * @param {object} [model.ajaxCallOptions]
+   *     Properties of this object are copied to all instances of {@link AjaxCall}.
    */
   init(model) {
     let options = model || {};

package/src/table/Table.js

@@ -5271,15 +5271,14 @@ export default class Table extends Widget {
 
     for (i = 0; i < columns.length; i++) {
       column = columns[i];
-      currentPosition = this.columns.indexOf(column);
+      currentPosition = arrays.findIndex(this.columns, element => element.id === column.id);
       if (currentPosition < 0) {
         throw new Error('Column with id ' + column.id + 'not found.');
       }
 
       if (currentPosition !== i) {
         // Update model
-        arrays.remove(this.columns, column);
-        arrays.insert(this.columns, column, i);
+        arrays.move(this.columns, currentPosition, i);
       }
     }
 

package/src/table/columns/ColumnOptimalWidthMeasurer.js

@@ -147,7 +147,7 @@ export default class ColumnOptimalWidthMeasurer {
   _build$CellForAggregateRow(row) {
     let columns = this.table.visibleColumns();
     let colIndex = columns.indexOf(this.column);
-    let $row = this.table._buildAggregateRowDiv(row);
+    let $row = this.table._build$AggregateRow(row);
 
     $row.appendTo(this.table.$data);
     columns

package/src/testing/JasmineScoutUtil.js

@@ -0,0 +1,127 @@
+/*
+ * Copyright (c) 2010-2022 BSI Business Systems Integration AG.
+ * All rights reserved. This program and the accompanying materials
+ * are made available under the terms of the Eclipse Public License v1.0
+ * which accompanies this distribution, and is available at
+ * https://www.eclipse.org/legal/epl-v10.html
+ *
+ * Contributors:
+ *     BSI Business Systems Integration AG - initial API and implementation
+ */
+import {arrays, scout, strings} from '../index';
+import $ from 'jquery';
+
+let _jsonResourceCache = {};
+
+/**
+ * Utility functions for jasmine tests.
+ */
+export const JasmineScoutUtil = {
+
+  /**
+   * @param {string} jsonResourceUrl
+   * @param {object} [options]
+   * @param {boolean} [options.useCache=true]
+   * @return {Promise<object>} - the loaded JSON data structure
+   */
+  loadJsonResource(jsonResourceUrl, options = {}) {
+    scout.assertParameter('jsonResourceUrl', jsonResourceUrl);
+
+    if (scout.nvl(options.useCache, true)) {
+      let json = _jsonResourceCache[jsonResourceUrl];
+      if (json) {
+        return $.resolvedPromise(json);
+      }
+    }
+
+    return $.ajax({
+      async: false,
+      type: 'GET',
+      dataType: 'json',
+      contentType: 'application/json; charset=UTF-8',
+      cache: false,
+      url: jsonResourceUrl
+    })
+      .done(json => {
+        if (scout.nvl(options.useCache, true)) {
+          _jsonResourceCache[jsonResourceUrl] = json;
+        }
+        return $.resolvedPromise(json);
+      })
+      .fail((jqXHR, textStatus, errorThrown) => {
+        throw new Error('Could not load resource from url: ' + jsonResourceUrl);
+      });
+  },
+
+  /**
+   * @param {string} resourceUrlToMock
+   * @param {string} jsonResourceUrl
+   * @param {object} [options]
+   * @param {boolean} [options.useCache=true]
+   * @param {*} [options.restriction]
+   * @param {string} [options.method]
+   */
+  loadJsonResourceAndMockRestCall(resourceUrlToMock, jsonResourceUrl, options = {}) {
+    scout.assertParameter('resourceUrlToMock', resourceUrlToMock);
+
+    this.loadJsonResource(jsonResourceUrl, options)
+      .then(json => this.mockRestCall(resourceUrlToMock, json, options));
+  },
+
+  /**
+   * @param {string} resourceUrlToMock
+   * @param {array} lookupRows
+   * @param {*} [parentRestriction]
+   */
+  mockRestLookupCall(resourceUrlToMock, lookupRows, parentRestriction) {
+    scout.assertParameter('resourceUrlToMock', resourceUrlToMock);
+
+    // Normalize lookup rows
+    lookupRows = arrays.ensure(lookupRows).map(lookupRow => $.extend({
+      active: true,
+      enabled: true,
+      parentId: null
+    }, lookupRow));
+
+    // getAll()
+    this.mockRestCall(resourceUrlToMock, {
+      rows: lookupRows
+    }, {
+      restriction: parentRestriction
+    });
+
+    // getKey()
+    lookupRows.forEach(lookupRow => {
+      this.mockRestCall(resourceUrlToMock, {
+        rows: [lookupRow]
+      }, {
+        restriction: lookupRow.id
+      });
+    });
+  },
+
+  /**
+   * @param {string} resourceUrlToMock
+   * @param {*} responseData
+   * @param {object} [options]
+   * @param {*} [options.restriction]
+   * @param {string} [options.method]
+   */
+  mockRestCall(resourceUrlToMock, responseData, options = {}) {
+    let url = new RegExp('.*' + strings.quote(resourceUrlToMock) + '.*');
+    let data = options.restriction ? new RegExp('.*' + strings.quote(options.restriction) + '.*') : undefined;
+    jasmine.Ajax.stubRequest(url, data, options.method).andReturn({
+      status: 200,
+      responseText: JSON.stringify(responseData)
+    });
+  },
+
+  /**
+   * If a ajax call is not mocked, this fallback will be triggered to show information about which url is not mocked.
+   */
+  captureNotMockedCalls() {
+    jasmine.Ajax.stubRequest(/.*/).andCallFunction((stub, mockAjax) => {
+      fail('Ajax call not mocked for url: ' + mockAjax.url + ', method: ' + mockAjax.method);
+    });
+  }
+};

package/src/testing/index.js

@@ -8,8 +8,14 @@
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
  */
+
+// /////////////////////////////////////////////////////////////////
+// TEST SUPPORT - DO NOT USE IN PRODUCTION CODE
+// /////////////////////////////////////////////////////////////////
+
 export {default as TestingApp} from './TestingApp';
 export {default as JasmineScout} from './JasmineScout';
+export {JasmineScoutUtil} from './JasmineScoutUtil';
 export {default as LocaleSpecHelper} from './text/LocaleSpecHelper';
 export {default as MenuSpecHelper} from './menu/MenuSpecHelper';
 export {default as TreeSpecHelper} from './tree/TreeSpecHelper';
@@ -31,6 +37,4 @@ export {default as GroupBoxSpecHelper} from './form/fields/groupbox/GroupBoxSpec
 import * as self from './index.js';
 
 export default self;
-if (window) {
-  window.scout = Object.assign(window.scout || {}, self);
-}
+window.scout = Object.assign(window.scout || {}, self);

package/src/tile/TileGrid.js

@@ -1,9 +1,9 @@
 /*
- * Copyright (c) 2010-2021 BSI Business Systems Integration AG.
+ * Copyright (c) 2010-2022 BSI Business Systems Integration AG.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
- * http://www.eclipse.org/legal/epl-v10.html
+ * https://www.eclipse.org/legal/epl-v10.html
  *
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
@@ -42,8 +42,6 @@ export default class TileGrid extends Widget {
     this.selectedTiles = [];
     this.selectionHandler = new TileGridSelectionHandler(this);
     this.scrollable = true;
-    this.scrolling = false;
-    this.scrollTopDirty = false;
     this.startupAnimationDone = false;
     this.startupAnimationEnabled = false;
     this.tiles = [];
@@ -594,18 +592,14 @@ export default class TileGrid extends Widget {
     let scrollTop = this.$container[0].scrollTop;
     let scrollLeft = this.$container[0].scrollLeft;
     if (this.scrollTop !== scrollTop && this.virtual) {
-      this.scrolling = true;
-      this.revalidateLayout();
-      this.scrolling = false;
+      this.htmlComp.layout.updateViewPort();
     }
     this.scrollTop = scrollTop;
     this.scrollLeft = scrollLeft;
   }
 
   _onScrollParentScroll(event) {
-    this.scrolling = true;
-    this.revalidateLayoutTree(false);
-    this.scrolling = false;
+    this.htmlComp.layout.updateViewPort();
   }
 
   setWithPlaceholders(withPlaceholders) {
@@ -961,12 +955,8 @@ export default class TileGrid extends Widget {
     this.ensureTileRendered(tile);
     // If tile was not rendered it is not yet positioned correctly -> make sure layout is valid before trying to scroll
     // Layout must not render the viewport because scroll position is not correct yet -> just make sure tiles are at the correct position
-    this.scrolling = true;
-    this.scrollTopDirty = true;
-    this.validateLayoutTree();
-    this.scrolling = false;
+    this.htmlComp.layout.updateViewPort(true);
     tile.reveal(options);
-    this.scrollTopDirty = false;
   }
 
   revealSelection() {

package/src/tile/TileGridLayout.js

@@ -3,7 +3,7 @@
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
- * http://www.eclipse.org/legal/epl-v10.html
+ * https://www.eclipse.org/legal/epl-v10.html
  *
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
@@ -44,19 +44,29 @@ export default class TileGridLayout extends LogicalGridLayout {
     this.maxWidth = -1;
   }
 
-  layout($container) {
-    let htmlComp = this.widget.htmlComp;
-    if (this.widget.scrolling) {
-      // Try to layout only as much as needed while scrolling in virtual mode
-      // Scroll top may be dirty when layout is validated before scrolling to a specific tile (see tileGrid.scrollTo)
-      if (!this.widget.scrollTopDirty) {
-        this.widget._renderViewPort();
-      }
-      this._layout($container);
-      this.widget.trigger('layoutAnimationDone');
+  /**
+   *
+   * @param {boolean} [scrollTopDirty] If the scroll top position should be considered dirty while updating the view port.
+   * If true, the view port is not rendered, as the scroll positions are not reliable anyway. Then only the layout of the TileGrid is updated.
+   */
+  updateViewPort(scrollTopDirty) {
+    let tileGrid = this.widget;
+    if (!tileGrid.rendered) {
       return;
     }
 
+    // Try to layout only as much as needed while scrolling in virtual mode
+    // Scroll top may be dirty when layout is validated before scrolling to a specific tile (see tileGrid.scrollTo)
+    if (!scout.nvl(scrollTopDirty, false)) {
+      tileGrid._renderViewPort();
+    }
+    this._layout(tileGrid.$container);
+    tileGrid.trigger('layoutAnimationDone');
+  }
+
+  layout($container) {
+    let htmlComp = this.widget.htmlComp;
+
     // Animate only once on startup (if enabled) but animate every time on resize
     let animated = htmlComp.layouted || (this.widget.startupAnimationEnabled && !this.widget.startupAnimationDone) || this.widget.renderAnimationEnabled;
     this.tiles = this.widget.renderedTiles();

package/src/tile/accordion/TileAccordion.js

@@ -3,7 +3,7 @@
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
- * http://www.eclipse.org/legal/epl-v10.html
+ * https://www.eclipse.org/legal/epl-v10.html
  *
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
@@ -350,24 +350,7 @@ export default class TileAccordion extends Accordion {
   }
 
   _filter() {
-    this.groups.forEach(group => {
-      group.body.filter();
-
-      // If the layout has not been invalidated as part of the filtering above, it even though must be validated here.
-      // This is because groups above might have fewer visible Tiles now which makes room for this group.
-      // The revalidateLayout() with scrolling=true here ensures TileGrid._renderViewPort() is called to ensure these Tiles become visible as there is space available now.
-      // It is executed as postValidateFunction because the groups above must have completed their layouting so that
-      // TileGrid._renderViewPort() knows that there is more space available now.
-      if (group.body.htmlComp && group.body.htmlComp.valid && !group.body._accordionLayoutHandler /* skip if already registered */) {
-        group.body._accordionLayoutHandler = () => {
-          group.body.scrolling = true;
-          group.body.revalidateLayout();
-          group.body.scrolling = false;
-          group.body._accordionLayoutHandler = null;
-        };
-        this.session.layoutValidator.schedulePostValidateFunction(group.body._accordionLayoutHandler);
-      }
-    });
+    this.groups.forEach(group => group.body.filter());
   }
 
   /**
@@ -700,10 +683,8 @@ export default class TileAccordion extends Accordion {
         // Btw: another group may be doing it as well at the same time (e.g. because of exclusiveExpand)
         return;
       }
-      if (group.body.virtual) {
-        group.body.scrolling = true;
-        group.body.revalidateLayout();
-        group.body.scrolling = false;
+      if (group.body.virtual && group.body.htmlComp) {
+        group.body.htmlComp.layout.updateViewPort();
       }
     });
   }