AG-11547 - fixed scrollbar not showing after expanding column group

ag-grid · May 15, 2024 · f0c2df4 · f0c2df4
1 parent af0a966
commit f0c2df4
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/community-modules/core/src/columns/columnModel.ts b/community-modules/core/src/columns/columnModel.ts
@@ -3469,7 +3469,6 @@ export class ColumnModel extends BeanStub {
     }
 
     private updateGroupsAndDisplayedColumns(source: ColumnEventType) {
-
         this.updateOpenClosedVisibilityInColumnGroups();
         this.deriveDisplayedColumns(source);
         this.refreshFlexedColumns();

diff --git a/community-modules/core/src/gridBodyComp/scrollVisibleService.ts b/community-modules/core/src/gridBodyComp/scrollVisibleService.ts
@@ -23,7 +23,6 @@ export class ScrollVisibleService extends BeanStub {
     private postConstruct(): void {
         this.addManagedListener(this.eventService, Events.EVENT_DISPLAYED_COLUMNS_CHANGED, this.onDisplayedColumnsChanged.bind(this));
         this.addManagedListener(this.eventService, Events.EVENT_DISPLAYED_COLUMNS_WIDTH_CHANGED, this.onDisplayedColumnsWidthChanged.bind(this));
-
     }
 
     public onDisplayedColumnsChanged(): void {

diff --git a/community-modules/core/src/rendering/columnAnimationService.ts b/community-modules/core/src/rendering/columnAnimationService.ts
@@ -98,8 +98,11 @@ export class ColumnAnimationService extends BeanStub {
         this.getFrameworkOverrides().wrapIncoming(() => {
             window.setTimeout(() => runFuncs(this.executeNextFuncs), 0);
             window.setTimeout(() => {
-                runFuncs(this.executeLaterFuncs);
+                // run the callback before executeLaterFuncs
+                // because some functions being executed later
+                // check if this service is `active`.
                 callback();
+                runFuncs(this.executeLaterFuncs);
             }, 200);
         });
     }