Merge pull request #376 from datavisyn/clehner/lineupjs4

Call itemRowHeight function also for groups
datavisyn · Jun 9, 2020 · 7ffa601 · 7ffa601
2 parents 642b587 + 406c4c3
commit 7ffa601
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/lineup/ARankingView.ts b/src/lineup/ARankingView.ts
@@ -147,7 +147,7 @@ export interface IARankingViewOptions {
    */
   enableStripedBackground: boolean;
 
-  itemRowHeight: number | ((item: IGroupItem) => number) | null;
+  itemRowHeight: number | ((item: IGroupItem | IGroupData) => number) | null;
 
   customOptions: Partial<ITaggleOptions>;
   customProviderOptions: Partial<ILocalDataProviderOptions & IDataProviderOptions  & { maxNestedSortingCriteria: number; maxGroupColumns: number; filterGlobally: true; }>;
@@ -281,7 +281,7 @@ export abstract class ARankingView extends AView {
         defaultHeight: taggleOptions.rowHeight,
         padding: () => 0,
         height: (item: IGroupItem | IGroupData) => {
-          return isGroup(item) ? taggleOptions.groupHeight : f(item);
+          return f(item) ?? (isGroup(item) ? taggleOptions.groupHeight : taggleOptions.rowHeight);
         }
       });
     }