@oracle/oraclejet 10.0.3 → 10.0.7

package/dist/js/libs/oj/debug_es5/ojtable.js

@@ -2471,9 +2471,8 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
     if (rows.length === 0 && tableBodyRows.length === 0) {
       this._showNoDataMessage();
     } else {
-      var tableBodyDocFrag; // for the pre-fetching case, we'll render items during idle cycles
+      // for the pre-fetching case, we'll render items during idle cycles
       // don't do it when the scroll is caused by handling scroll position
-
       if (startIndex > 0 && this._isOverflow() && !this._fetchBySyncScroll) {
         return new Promise(function (resolve, reject) {
           this._setIdleRenderBusyState(); // clone the rows array since we'll manipulate it
@@ -2483,6 +2482,9 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
         }.bind(this));
       }
 
+      var layoutManager = this._getLayoutManager();
+
+      var tableBodyDocFrag = document.createDocumentFragment();
       var rowsCount = rows.length;
 
       if (this._animateOnFetch && this._IsCustomElement()) {
@@ -2492,11 +2494,16 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
 
         for (i = 0; i < rowsCount; i++) {
           rowIdxArray.push(rows[i].rowIdx);
-          addedTableBodyRows.push(this._addSingleTableBodyRow(rows[i].rowIdx, rows[i].row));
+          addedTableBodyRows.push(this._addSingleTableBodyRow(rows[i].rowIdx, rows[i].row, tableBodyDocFrag, startIndex));
         }
 
+        layoutManager.handleAfterRowsProcessed(tableBodyDocFrag);
+        tableBody.appendChild(tableBodyDocFrag); // @HTMLUpdateOK
+
+        this._clearCachedDomRowData();
+
         return this._animateVisibleRows(addedTableBodyRows, rowIdxArray, 'add').then(function () {
-          return this._finalizeBodyRowRendering(addedTableBodyRows);
+          return this._afterRowsRendered(tableBody);
         }.bind(this));
       }
 
@@ -2506,12 +2513,11 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
         tableBody.style[Table.CSS_PROP._VISIBILITY] = Table.CSS_VAL._HIDDEN;
       }
 
-      tableBodyDocFrag = document.createDocumentFragment();
-
       for (i = 0; i < rowsCount; i++) {
         this._renderRow(rows[i], tableBodyDocFrag, startIndex);
       }
 
+      layoutManager.handleAfterRowsProcessed(tableBodyDocFrag);
       tableBody.appendChild(tableBodyDocFrag); // @HTMLUpdateOK
 
       return this._afterRowsRendered(tableBody);
@@ -6327,6 +6333,14 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
    * @private
    */
 
+  TableLayoutManager.prototype.unregisterListeners = function () {
+    this.unregisterScrollListeners();
+  };
+  /**
+   * @private
+   */
+
+
   TableLayoutManager.prototype._getSizingState = function () {
     if (this._sizingState == null) {
       this._sizingState = {
@@ -6432,19 +6446,13 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
       if (this._table._isLoadMoreOnScroll() && maxScrollTop === newScrollTop) {
         // Do not set to maxScrollTop or we will cause another fetch
         newScrollTop -= 1;
-      } // set flag so that scroll() will not cause an scrollPosition update
-
+      }
 
-      this._table._skipScrollUpdate = true;
       this.getScroller().scrollTop = newScrollTop;
-      $(this.getScroller()).scroll();
     }
 
     if (scrollLeft != null) {
-      // set flag so that scroll() will not cause an scrollPosition update
-      this._table._skipScrollUpdate = true;
-      DomUtils.setScrollLeft(this.getScroller(), scrollLeft);
-      $(this.getScroller()).scroll();
+      this._restoreScrollLeft(scrollLeft);
     } // cache the final dimensions
 
 
@@ -6456,6 +6464,8 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
     this._clearTableUpdates();
 
     this._enableTableVisibility();
+
+    this.registerScrollListeners();
   };
   /**
    * @private
@@ -6468,13 +6478,110 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
    */
 
 
-  TableLayoutManager.prototype.handleScrollerScrollLeft = function () {};
+  TableLayoutManager.prototype._handleScrollerScrollLeft = function (scrollLeft) {
+    this._table._scrollLeft = scrollLeft;
+  };
+  /**
+   * Handle scrollTop on scroller
+   * @private
+   */
+
+
+  TableLayoutManager.prototype._handleScrollerScrollTop = function (scrollTop) {
+    // if no domScroller is set, make sure scroll pos busy state is cleared here
+    if (!this._table._domScroller) {
+      this._table._clearScrollPosBusyState();
+    }
+
+    if (scrollTop < 0) {
+      // eslint-disable-next-line no-param-reassign
+      scrollTop = 0;
+    }
+
+    this._table._scrollTop = scrollTop; // keep track when scrollTop is triggered internally (syncScrollPosition)
+
+    this._table._scrollY = this._table._skipScrollUpdate ? scrollTop : null; // update mobile touch selection affordance if present
+
+    this._table._moveTableBodyRowTouchSelectionAffordanceTop();
+
+    this._table._moveTableBodyRowTouchSelectionAffordanceBottom();
+  };
+  /**
+   * @private
+   */
+
+
+  TableLayoutManager.prototype.registerScrollListeners = function () {
+    var scroller = this.getScroller();
+
+    if (scroller != null) {
+      // if width or height is defined then we can have scrollbars so register scroll event listeners
+      if (this._scrollEventListener == null) {
+        this._scrollEventListener = function (event) {
+          var newScrollLeft = this._table._getElementScrollLeft(event.target);
+
+          var newScrollTop = event.target.scrollTop;
+
+          if (newScrollLeft === this._table._scrollLeft && newScrollTop === this._table._scrollTop) {
+            // discard bogus scroll event
+            return;
+          }
+
+          this._handleScrollerScrollLeft(newScrollLeft);
+
+          this._handleScrollerScrollTop(newScrollTop); // do not update scrollPosition if we are already adjusting
+
+
+          if (this._table._isScrollPositionAdjusted()) {
+            return;
+          }
+
+          if (!this._table._skipScrollUpdate && !this._ticking) {
+            window.requestAnimationFrame(function () {
+              // scrollPosition could have been adjusted or a new value is set while waiting for animation frame
+              if (!this._table._isScrollPositionAdjusted() && !this._table._skipScrollUpdate) {
+                this._table.option('scrollPosition', this._table._getCurrentScrollPosition(), {
+                  _context: {
+                    originalEvent: event,
+                    internalSet: true
+                  }
+                });
+              }
+
+              this._ticking = false;
+            }.bind(this));
+            this._ticking = true;
+          }
+
+          this._table._skipScrollUpdate = false;
+        }.bind(this);
+      } // remove the event listener before adding to make sure only one is active at a time
+
+
+      scroller.removeEventListener('scroll', this._scrollEventListener);
+      scroller.addEventListener('scroll', this._scrollEventListener, false);
+    }
+  };
+  /**
+   * @private
+   */
+
+
+  TableLayoutManager.prototype.unregisterScrollListeners = function () {
+    var scroller = this.getScroller();
+
+    if (scroller != null && this._scrollEventListener != null) {
+      scroller.removeEventListener('scroll', this._scrollEventListener);
+    }
+  };
   /**
    * @private
    */
 
 
-  TableLayoutManager.prototype.registerScrollListeners = function () {};
+  TableLayoutManager.prototype._restoreScrollLeft = function (scrollLeft) {
+    DomUtils.setScrollLeft(this.getScroller(), scrollLeft);
+  };
   /**
    * @param {Element} cell
    * @private
@@ -6766,6 +6873,8 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
 
 
   TableLayoutManager.prototype._removeTableDimensionsStyling = function () {
+    this.unregisterScrollListeners();
+
     var tableElem = this._table._getTable();
 
     var tableHeader = this._table._getTableHeader();
@@ -7130,7 +7239,9 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
    */
 
 
-  TableLegacyLayoutManager.prototype.handleScrollerScrollLeft = function (scrollLeft) {
+  TableLegacyLayoutManager.prototype._handleScrollerScrollLeft = function (scrollLeft) {
+    TableLegacyLayoutManager.superclass._handleScrollerScrollLeft.call(this, scrollLeft);
+
     var tableHeaderRow = this._table._getTableHeaderRow();
 
     if (tableHeaderRow) {
@@ -7149,14 +7260,15 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
 
 
   TableLegacyLayoutManager.prototype.registerScrollListeners = function () {
-    // although header region does not contain scrollbars, browser may auto-scroll when contents gain focus
+    TableLegacyLayoutManager.superclass.registerScrollListeners.call(this); // although header region does not contain scrollbars, browser may auto-scroll when contents gain focus
+
     var tableHeaderRow = this._table._getTableHeaderRow();
 
     if (tableHeaderRow != null) {
       var tableHeader = tableHeaderRow.parentNode;
 
-      if (this._table._headerScrollEventListener == null) {
-        this._table._headerScrollEventListener = function (event) {
+      if (this._headerScrollEventListener == null) {
+        this._headerScrollEventListener = function (event) {
           var scrollLeft = this._table._getElementScrollLeft(event.target);
 
           if (this._table._scrollLeft !== scrollLeft) {
@@ -7166,8 +7278,71 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
       } // remove the event listener before adding to make sure only one is active at a time
 
 
-      tableHeader.removeEventListener('scroll', this._table._headerScrollEventListener);
-      tableHeader.addEventListener('scroll', this._table._headerScrollEventListener, false);
+      tableHeader.removeEventListener('scroll', this._headerScrollEventListener);
+      tableHeader.addEventListener('scroll', this._headerScrollEventListener, false);
+    } // although footer region does not contain scrollbars, browser may auto-scroll when contents gain focus
+
+
+    var tableFooterRow = this._table._getTableFooterRow();
+
+    if (tableFooterRow != null) {
+      var tableFooter = tableFooterRow.parentNode;
+
+      if (this._footerScrollEventListener == null) {
+        this._footerScrollEventListener = function (event) {
+          var scrollLeft = this._table._getElementScrollLeft(event.target);
+
+          if (this._table._scrollLeft !== scrollLeft) {
+            this._table._setScrollX(scrollLeft);
+          }
+        }.bind(this);
+      } // remove the event listener before adding to make sure only one is active at a time
+
+
+      tableFooter.removeEventListener('scroll', this._footerScrollEventListener);
+      tableFooter.addEventListener('scroll', this._footerScrollEventListener, false);
+    }
+  };
+  /**
+   * @private
+   */
+
+
+  TableLegacyLayoutManager.prototype.unregisterScrollListeners = function () {
+    TableLegacyLayoutManager.superclass.unregisterScrollListeners.call(this);
+
+    var tableHeaderRow = this._table._getTableHeaderRow();
+
+    if (tableHeaderRow != null && this._headerScrollEventListener != null) {
+      var tableHeader = tableHeaderRow.parentNode;
+      tableHeader.removeEventListener('scroll', this._headerScrollEventListener);
+    }
+
+    var tableFooterRow = this._table._getTableFooterRow();
+
+    if (tableFooterRow != null && this._footerScrollEventListener != null) {
+      var tableFooter = tableFooterRow.parentNode;
+      tableFooter.removeEventListener('scroll', this._footerScrollEventListener);
+    }
+  };
+  /**
+   * @private
+   */
+
+
+  TableLegacyLayoutManager.prototype._restoreScrollLeft = function (scrollLeft) {
+    DomUtils.setScrollLeft(this.getScroller(), scrollLeft);
+
+    var tableHeaderRow = this._table._getTableHeaderRow();
+
+    if (tableHeaderRow) {
+      DomUtils.setScrollLeft(tableHeaderRow.parentNode, scrollLeft);
+    }
+
+    var tableFooterRow = this._table._getTableFooterRow();
+
+    if (tableFooterRow) {
+      DomUtils.setScrollLeft(tableFooterRow.parentNode, scrollLeft);
     }
   };
   /**
@@ -7218,7 +7393,6 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
     var columns = this._table._getColumnDefs();
 
     var columnsCount = columns.length;
-    var tableBodyCell;
 
     if (!this._frozenWidthRows) {
       this._frozenWidthRows = [];
@@ -7227,11 +7401,16 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
     var minWidths = [];
 
     for (var i = 0; i < columnsCount; i++) {
-      tableBodyCell = this._table._getTableBodyCell(0, i, null);
-      var minWidth = parseFloat(tableBodyCell.style[Table.CSS_PROP._MIN_WIDTH]);
+      var tableBodyCell = this._table._getTableBodyCell(0, i, null);
 
-      if (!isNaN(minWidth)) {
-        minWidths.push(minWidth);
+      if (tableBodyCell != null) {
+        var minWidth = parseFloat(tableBodyCell.style[Table.CSS_PROP._MIN_WIDTH]);
+
+        if (!isNaN(minWidth)) {
+          minWidths.push(minWidth);
+        } else {
+          minWidths.push(null);
+        }
       } else {
         minWidths.push(null);
       }
@@ -8953,15 +9132,20 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
           return undefined;
         }
 
-        this._getLayoutManager().notifyTableUpdate(Table._UPDATE._ROWS_ADDED); // see if we should batch add
-        // only batch if we are adding a block of contiguous rows
+        var layoutManager = this._getLayoutManager();
 
+        layoutManager.notifyTableUpdate(Table._UPDATE._ROWS_ADDED); // see if we should batch add
+        // only batch if we are adding a block of contiguous rows
 
         var batchAdd = false;
         var addedTableBodyRows = [];
         var rowIdxArray = [];
         var i;
         var rowsCount;
+        var tableBodyDocFrag;
+        var tableBodyRowBefore;
+
+        var tableBody = this._getTableBody();
 
         if (rows.length > 1) {
           rowsCount = rows.length;
@@ -8978,30 +9162,22 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
           }
 
           if (isContiguous) {
-            var tableBody = this._getTableBody();
-
-            var tableBodyDocFrag = document.createDocumentFragment();
+            tableBodyDocFrag = document.createDocumentFragment();
             rowsCount = rows.length;
 
             for (i = 0; i < rowsCount; i++) {
               addedTableBodyRows.push(this._addSingleTableBodyRow(rows[i].rowIdx, rows[i].row, tableBodyDocFrag, rows[0].rowIdx));
             }
 
-            if (rows[0].rowIdx === 0) {
-              tableBody.insertBefore(tableBodyDocFrag, tableBody.firstChild); // @HTMLUpdateOK
-            } else {
-              var tableBodyRowBefore = this._getTableBodyRow(rows[0].rowIdx);
+            layoutManager.handleAfterRowsProcessed(tableBodyDocFrag);
+            tableBodyRowBefore = this._getTableBodyRow(rows[0].rowIdx);
 
-              if (tableBodyRowBefore != null) {
-                tableBody.insertBefore(tableBodyDocFrag, tableBodyRowBefore); // @HTMLUpdateOK
-              } else {
-                tableBody.appendChild(tableBodyDocFrag, null); // @HTMLUpdateOK
-              }
+            if (tableBodyRowBefore != null) {
+              tableBody.insertBefore(tableBodyDocFrag, tableBodyRowBefore); // @HTMLUpdateOK
+            } else {
+              tableBody.appendChild(tableBodyDocFrag, null); // @HTMLUpdateOK
             }
 
-            this._clearCachedDomRowData();
-
-            ojcomponentcore.subtreeAttached(tableBody);
             batchAdd = true;
           }
         }
@@ -9010,7 +9186,17 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
           rowsCount = rows.length;
 
           for (i = 0; i < rowsCount; i++) {
-            addedTableBodyRows.push(this._addSingleTableBodyRow(rows[i].rowIdx, rows[i].row));
+            var rowIndex = rows[i].rowIdx;
+            tableBodyDocFrag = document.createDocumentFragment();
+            addedTableBodyRows.push(this._addSingleTableBodyRow(rowIndex, rows[i].row, tableBodyDocFrag, rowIndex));
+            layoutManager.handleAfterRowsProcessed(tableBodyDocFrag);
+            tableBodyRowBefore = this._getTableBodyRow(rowIndex);
+
+            if (tableBodyRowBefore != null) {
+              tableBody.insertBefore(tableBodyDocFrag, tableBodyRowBefore); // @HTMLUpdateOK
+            } else {
+              tableBody.appendChild(tableBodyDocFrag, null); // @HTMLUpdateOK
+            }
           }
         }
 
@@ -9026,11 +9212,11 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
 
         if (this._IsCustomElement()) {
           return this._animateVisibleRows(addedTableBodyRows, rowIdxArray, 'add').then(function () {
-            return this._finalizeBodyRowRendering(addedTableBodyRows);
+            return this._afterRowsRendered(tableBody);
           }.bind(this));
         }
 
-        return this._finalizeBodyRowRendering(addedTableBodyRows);
+        return this._afterRowsRendered(tableBody);
       }.bind(this));
     }
   };
@@ -10682,10 +10868,6 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
   Table.prototype._getTableBodyCell = function (rowIdx, columnIdx, tableBodyRow) {
     var tableBodyCells = this._getTableBodyCells(rowIdx, tableBodyRow);
 
-    if (!tableBodyCells) {
-      return null;
-    }
-
     if (tableBodyCells.length > columnIdx) {
       return tableBodyCells[columnIdx];
     }
@@ -10704,10 +10886,6 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
   Table.prototype._getTableBodyLogicalCells = function (rowIdx, tableBodyRow) {
     var tableBodyCells = this._getTableBodyCells(rowIdx, tableBodyRow);
 
-    if (!tableBodyCells) {
-      return null;
-    }
-
     return this._getColspanLogicalElements(tableBodyCells);
   };
   /**
@@ -10723,19 +10901,13 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
     if (!tableBodyRow) {
       // eslint-disable-next-line no-param-reassign
       tableBodyRow = this._getTableBodyRow(rowIdx);
-    }
-
-    if (!tableBodyRow) {
-      return null;
-    }
 
-    var tableBodyCellElements = this._getTableElementsByClassName(tableBodyRow, Table.CSS_CLASSES._TABLE_DATA_CELL_CLASS);
-
-    if (tableBodyCellElements.length > 0) {
-      return tableBodyCellElements;
+      if (!tableBodyRow) {
+        return [];
+      }
     }
 
-    return null;
+    return this._getTableElementsByClassName(tableBodyRow, Table.CSS_CLASSES._TABLE_DATA_CELL_CLASS);
   };
   /**
    * Return the table body message cell element
@@ -14471,10 +14643,6 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
 
     var tableBodyCells = this._getTableBodyCells(rowIdx);
 
-    if (!tableBodyCells) {
-      return;
-    }
-
     var tableBodyCellsCount = tableBodyCells.length;
 
     for (var i = 0; i < tableBodyCellsCount; i++) {
@@ -14850,7 +15018,7 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
   Table.prototype._updateRowStateCellsClass = function (rowIdx, state) {
     var tableBodyCells = this._getTableBodyCells(rowIdx);
 
-    if (!tableBodyCells) {
+    if (tableBodyCells.length === 0) {
       return;
     }
 
@@ -15670,42 +15838,6 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
 
     return result;
   };
-  /**
-   * Handle scrollLeft on scroller
-   * @private
-   */
-
-
-  Table.prototype._handleScrollerScrollLeft = function (scrollLeft) {
-    this._scrollLeft = scrollLeft;
-
-    this._getLayoutManager().handleScrollerScrollLeft(scrollLeft);
-  };
-  /**
-   * Handle scrollTop on scroller
-   * @private
-   */
-
-
-  Table.prototype._handleScrollerScrollTop = function (scrollTop) {
-    // if no domScroller is set, make sure scroll pos busy state is cleared here
-    if (!this._domScroller) {
-      this._clearScrollPosBusyState();
-    }
-
-    if (scrollTop < 0) {
-      // eslint-disable-next-line no-param-reassign
-      scrollTop = 0;
-    }
-
-    this._scrollTop = scrollTop; // keep track when scrollTop is triggered internally (syncScrollPosition)
-
-    this._scrollY = this._skipScrollUpdate ? scrollTop : null; // update mobile touch selection affordance if present
-
-    this._moveTableBodyRowTouchSelectionAffordanceTop();
-
-    this._moveTableBodyRowTouchSelectionAffordanceBottom();
-  };
   /**
    * Register event listeners which need to be registered directly on the DOM element.
    * @private
@@ -15713,59 +15845,7 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
 
 
   Table.prototype._registerDomEventListeners = function () {
-    var layoutManager = this._getLayoutManager();
-
-    var scroller = layoutManager.getScroller();
-
-    if (scroller != null) {
-      // if width or height is defined then we can have scrollbars so register scroll event listeners
-      if (this._scrollEventListener == null) {
-        this._scrollEventListener = function (event) {
-          var newScrollLeft = this._getElementScrollLeft(event.target);
-
-          var newScrollTop = event.target.scrollTop;
-
-          if (newScrollLeft === this._scrollLeft && newScrollTop === this._scrollTop) {
-            // discard bogus scroll event
-            return;
-          }
-
-          this._handleScrollerScrollLeft(newScrollLeft);
-
-          this._handleScrollerScrollTop(newScrollTop); // do not update scrollPosition if we are already adjusting
-
-
-          if (this._isScrollPositionAdjusted()) {
-            return;
-          }
-
-          if (!this._skipScrollUpdate && !this._ticking) {
-            window.requestAnimationFrame(function () {
-              // scrollPosition could have been adjusted or a new value is set while waiting for animation frame
-              if (!this._isScrollPositionAdjusted() && !this._skipScrollUpdate) {
-                this.option('scrollPosition', this._getCurrentScrollPosition(), {
-                  _context: {
-                    originalEvent: event,
-                    internalSet: true
-                  }
-                });
-              }
-
-              this._ticking = false;
-            }.bind(this));
-            this._ticking = true;
-          }
-
-          this._skipScrollUpdate = false;
-        }.bind(this);
-      } // remove the event listener before adding to make sure only one is active at a time
-
-
-      scroller.removeEventListener('scroll', this._scrollEventListener);
-      scroller.addEventListener('scroll', this._scrollEventListener, false);
-    }
-
-    layoutManager.registerScrollListeners();
+    this._getLayoutManager().registerScrollListeners();
   };
   /**
    * Return whether the component is in table actionable mode
@@ -19427,7 +19507,8 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
    */
 
 
-  TableFixedLayoutManager.prototype._removeTableDimensionsStyling = function () {// no-op for fixed layout
+  TableFixedLayoutManager.prototype._removeTableDimensionsStyling = function () {
+    this.unregisterScrollListeners();
   };
   /**
    * @private
@@ -22845,6 +22926,10 @@ define(['touchr', 'ojdnd', 'ojs/ojeditablevalue', 'ojs/ojinputnumber', 'ojs/ojme
 
 
   Table.prototype._clearLayoutManager = function () {
+    if (this._layoutManager != null) {
+      this._layoutManager.unregisterListeners();
+    }
+
     this._layoutManager = null;
   };
   /* Later when needed

package/dist/js/libs/oj/debug_es5/ojtabs.js

@@ -2214,15 +2214,15 @@ define(['ojs/ojcore', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojdomutils', 'ojs/oj
           $(this).css('max-width', '').removeClass('oj-tabs-title-overflow');
         }); //      this._logMessage("remove max width");
       },
-      _logMessage: function _logMessage()
-      /* msg */
-      {//      console.log(msg);
+      _logMessage: function
+        /* msg */
+      _logMessage() {//      console.log(msg);
       },
 
       /* resize handler */
-      _handleResize: function _handleResize()
-      /* width */
-      {
+      _handleResize: function
+        /* width */
+      _handleResize() {
         //      this._logMessage("width " + width + " ulWidth " + this._originalWidth +
         //                       " clientWidth " + this._getTabsWidth());
         //  - truncation=none tabs + buttons are not laid out correctly.