@adaptabletools/adaptable 23.0.4 → 23.0.5-canary.1

package/src/agGrid/AgGridMenuAdapter.js

@@ -3,15 +3,6 @@ import { iconToString } from '../components/icons';
 import ArrayExtensions from '../Utilities/Extensions/ArrayExtensions';
 import { MENU_SEPARATOR } from '../Utilities/Constants/GeneralConstants';
 import { getAggFuncName, getDisplayAggFuncNames, isWeightedAverageAggregation, WEIGHTED_AVERAGE_AGG_FN_NAME, } from '../AdaptableState/Common/AggregationColumns';
-/**
- * Creates an AG Grid icon element for the given icon name.
- *
- * Mimics AG Grid's internal icon creation, producing a
- * `<span class="ag-icon ag-icon-{name}" role="presentation" unselectable="on">`
- * so the icon picks up AG Grid's own icon font/theme styling.
- *
- * @see https://github.com/ag-grid/ag-grid/blob/5ca14e73188f6f52fd7d545d595c067bf1b72168/packages/ag-grid-community/src/utils/icon.ts#L139
- */
 function createAgGridIcon(name) {
     const element = document.createElement('span');
     element.className = `ag-icon ag-icon-${name}`;
@@ -28,21 +19,6 @@ const toFuncLabel = (funcName) => {
     }
     return funcName ? funcName.charAt(0).toUpperCase() + funcName.slice(1) : funcName;
 };
-/**
- * Builds the replacement for AG Grid's built-in `valueAggSubMenu` column menu item.
- *
- * AG Grid does not allow customizing the built-in submenu, so we rebuild an
- * equivalent one: the column's existing aggregation functions plus a
- * `Weighted Average` item whose submenu lists the numeric columns that can be
- * used as the weight parameter.
- *
- * Both standard and weighted selections persist through the current Layout
- * (`updateCurrentLayout`), so the existing `weightedAvg` aggFunc can read the
- * chosen `weightedColumnId` back from the Layout.
- *
- * Weighted Average is never used in Pivot Layouts, so this only deals with the
- * Table Layout aggregations and the caller skips it entirely in Pivot mode.
- */
 function buildValueAggregationMenuItem(adaptableApi, menuContext) {
     const column = menuContext.adaptableColumn;
     const columnId = column.columnId;
@@ -86,7 +62,6 @@ function buildValueAggregationMenuItem(adaptableApi, menuContext) {
             };
         });
     };
-    // 'None' clears any aggregation for the column and is checked when none is set
     const subMenu = [
         {
             name: 'None',
@@ -94,11 +69,7 @@ function buildValueAggregationMenuItem(adaptableApi, menuContext) {
             action: () => removeAggregation(),
         },
     ];
-    // getDisplayAggFuncNames already positions weightedAvg right after 'avg'
-    // (and only includes it when the column is eligible)
     const aggFuncItems = getDisplayAggFuncNames(column).map((funcName) => {
-        // Weighted Average needs a weight column parameter, so instead of a plain
-        // action it gets a submenu listing the numeric columns usable as the weight.
         if (funcName === WEIGHTED_AVERAGE_AGG_FN_NAME) {
             const weightColumnItems = adaptableApi.columnApi
                 .getNumericColumns()
@@ -146,23 +117,17 @@ export class AgGridMenuAdapter {
     buildColumnMenu(params, originalGetMainMenuItems) {
         const agGridMenuItems = (params.defaultItems ? [...params.defaultItems] : []);
         if (!params.column) {
-            // return only AG Grid context if the Adaptable column is not found (should not happen)
             return agGridMenuItems;
         }
-        // If Adaptable UI is hidden, return only AG Grid default items
         if (!this.adaptableApi.userInterfaceApi.isAdaptableUIVisible()) {
             return agGridMenuItems;
         }
         const columnMenuOptions = this.adaptableOptions.columnMenuOptions;
         const adaptableColumn = this.adaptableApi.columnApi.getColumnWithColumnId(params.column?.getColId());
         const menuContext = this.createColumnMenuContextObject(adaptableColumn, params.column);
-        /**
-         * AG Grid items
-         */
         const agGridDefaultStructure = agGridMenuItems.map((itemName) => this.mapAgGridItemTypeToAgGridMenuItem(itemName));
         const adaptableMenuItems = this.createAdaptableColumnMenuItems(menuContext);
         const defaultColumnMenuStructure = this.buildColumnMenuDefaultStructure(adaptableMenuItems, menuContext);
-        // 1. first check if there is a custom column menu defined
         if (typeof columnMenuOptions.customColumnMenu === 'function') {
             const defaultAgGridMenuItems = agGridMenuItems.map((itemName) => this.mapAgGridItemTypeToAgGridMenuItem(itemName));
             const defaultAdaptableMenuItems = adaptableMenuItems.map((adaptableItem) => ({ menuType: 'Adaptable', ...adaptableItem }));
@@ -179,12 +144,6 @@ export class AgGridMenuAdapter {
                 .map((customMenuItem) => this.mapCustomMenuItemToAgGridMenuDefinition(customMenuItem, menuContext))
                 .filter(Boolean);
         }
-        // 2. if not, return the default context menu
-        // AG Grid does not allow customizing the built-in `valueAggSubMenu`, so we
-        // replace it with an Adaptable-built equivalent that also offers a Weighted
-        // Average aggregation (with a submenu to pick the weight column).
-        // Weighted Average is never used in Pivot Layouts, so we leave the AG Grid
-        // default item untouched there.
         const isPivotLayout = this.adaptableApi.layoutApi.isCurrentLayoutPivot();
         const mappedAgGridMenuItems = agGridMenuItems.map((itemName) => itemName === 'valueAggSubMenu' && !isPivotLayout
             ? {
@@ -203,23 +162,16 @@ export class AgGridMenuAdapter {
         if (!params.column) {
             return [];
         }
-        // If Adaptable UI is hidden, return only AG Grid default items
         if (!this.adaptableApi.userInterfaceApi.isAdaptableUIVisible()) {
             return (params.defaultItems ? [...params.defaultItems] : []);
         }
-        // we do this in order to refresh the internal state of selected cells (technically query the AG Grid cellRanges)
-        // (right-click selected the current cell, but this was not reflected in the internal state of the selected cells)
         this._adaptableInstance.refreshSelectedCellsState();
         const contextMenuOptions = this.adaptableOptions.contextMenuOptions;
         const adaptableColumn = this.adaptableApi.columnApi.getColumnWithColumnId(params.column?.getColId());
         const menuContext = this.createContextMenuContextObject(params, adaptableColumn);
-        /**
-         * AG Grid Items
-         */
         const agGridMenuItems = (params.defaultItems ? [...params.defaultItems] : []);
         const agGridCopyItems = agGridMenuItems.filter((item) => ['copy', 'copyWithHeaders', 'copyWithGroupHeaders', 'cut', 'paste'].includes(item));
         const otherAgGridItems = agGridMenuItems.filter((item) => !agGridCopyItems.includes(item) &&
-            // we provide Adaptable exports in the context menu
             !['export', 'csvExport', 'excelExport'].includes(item));
         const agGridDefaultStructure = [
             {
@@ -234,15 +186,8 @@ export class AgGridMenuAdapter {
                 .filter((itemName) => itemName !== 'separator')
                 .map((itemName) => this.mapAgGridItemTypeToAgGridMenuItem(itemName)),
         ];
-        /**
-         * Adaptable Items
-         */
         const adaptableMenuItems = this.createAdaptableContextMenuItems(menuContext);
         const adaptableDefaultStructure = this.buildContextMenuDefaultStructure(adaptableMenuItems, menuContext);
-        /**
-         * Build the context menu
-         */
-        // 1. first check if there is a custom context menu defined
         if (typeof contextMenuOptions.customContextMenu === 'function') {
             const defaultAgGridMenuItems = agGridMenuItems.map((itemName) => this.mapAgGridItemTypeToAgGridMenuItem(itemName));
             const defaultAdaptableMenuItems = adaptableMenuItems.map((adaptableItem) => ({ menuType: 'Adaptable', ...adaptableItem }));
@@ -261,7 +206,6 @@ export class AgGridMenuAdapter {
                 .map((customMenuItem) => this.mapCustomMenuItemToAgGridMenuDefinition(customMenuItem, menuContext))
                 .filter(Boolean);
         }
-        // 2. if not, return the default context menu
         const defaultContextMenu = [
             ...agGridDefaultStructure.map((agGridItem) => this.mapCustomMenuItemToAgGridMenuDefinition(agGridItem, menuContext)),
             'separator',
@@ -275,7 +219,6 @@ export class AgGridMenuAdapter {
             name: itemName,
         };
     }
-    // due to entitlements or other reasons, some menu items might be hidden, leading to consecutive separators
     removeConsecutiveSeparators(menuItems, separator = 'separator') {
         return menuItems.reduce((acc, item, index, array) => {
             if (item === separator && array[index + 1] === separator) {
@@ -308,11 +251,9 @@ export class AgGridMenuAdapter {
         return contextMenuItems;
     }
     createContextMenuContextObject(params, adaptableColumn) {
-        // lets build a picture of what has been right clicked.  Will take time to get right but lets start
         let isSingleSelectedColumn = false;
         let isSelectedCell = false;
         let isSelectedRow = false;
-        // row group columns dont provide an AdapTable Column so return bare minimum
         if (!adaptableColumn) {
             return {
                 ...this.adaptableApi.internalApi.buildBaseContext(),
@@ -427,22 +368,13 @@ export class AgGridMenuAdapter {
             })
                 .filter(Boolean),
         };
-        // If a framework component is supplied, attach it so AG Grid renders the
-        // menu row content via its own framework adapter. React, Vue and Angular
-        // variants are supported; for other variants we silently fall back to
-        // the default name/icon/action rendering.
         if (userMenuItem.frameworkComponent) {
             const variant = this.adaptableApi.gridApi.getVariant();
             let menuItemComponent;
             if (variant === 'react') {
-                // React function components are passed through as-is; AG Grid's
-                // React adapter will instantiate them with `CustomMenuItemProps`.
                 menuItemComponent = userMenuItem.frameworkComponent;
             }
             else if (variant === 'vue') {
-                // Vue framework components are factories of shape
-                // `({ adaptableApi }) => Component`; invoke once to obtain the
-                // component options object that AG Grid Vue's adapter expects.
                 try {
                     menuItemComponent = userMenuItem.frameworkComponent({
                         adaptableApi: this.adaptableApi,
@@ -454,12 +386,6 @@ export class AgGridMenuAdapter {
                 }
             }
             else if (variant === 'angular') {
-                // Angular framework components have shape `{ type: ComponentClass }`;
-                // hand the class to AG Grid Angular which uses its own
-                // `viewContainerRef.createComponent` to instantiate it. The optional
-                // `onSetup` field is intentionally NOT honoured here because AG Grid
-                // owns the lifecycle - menu-item Angular components should consume
-                // `menuItemParams` (or AG Grid's `agInit`) to receive `adaptableApi`.
                 const angularComponent = userMenuItem.frameworkComponent;
                 menuItemComponent = angularComponent.type;
             }
@@ -475,45 +401,24 @@ export class AgGridMenuAdapter {
         return menuItemDef;
     }
     buildContextMenuDefaultStructure(availableMenuItems, menuContext) {
-        // Alert
         const alertMenuItems = this.getModuleSpecificStructure('Alert', availableMenuItems);
-        // BulkUpdate
         const bulkUpdateMenuItems = this.getModuleSpecificStructure('BulkUpdate', availableMenuItems);
-        // CalculatedColumn
         const calculatedColumnMenuItems = this.getModuleSpecificStructure('CalculatedColumn', availableMenuItems);
-        // CellSummary
         const cellSummaryMenuItems = this.getModuleSpecificStructure('CellSummary', availableMenuItems);
-        // ColumnFilter
         const columnFilterMenuItems = this.getModuleSpecificStructure('ColumnFilter', availableMenuItems, 'column-filter-group');
-        // ColumnInfo
         const columnInfoMenuItems = this.getModuleSpecificStructure('ColumnInfo', availableMenuItems);
-        // Comment
         const commentMenuItems = this.getModuleSpecificStructure('Comment', availableMenuItems);
-        // Dashboard
         const dashboardMenuItems = this.getModuleSpecificStructure('Dashboard', availableMenuItems, 'dashboard-group');
-        // DataImport
         const dataImportMenuItems = this.getModuleSpecificStructure('DataImport', availableMenuItems);
-        // Export
         const exportMenuItems = this.getModuleSpecificStructure('Export', availableMenuItems, 'export-group');
-        // FDC3
         const fdc3MenuItems = this.getModuleSpecificStructure('Fdc3', availableMenuItems);
-        // FlashingCell
         const flashingCellMenuItems = this.getModuleSpecificStructure('FlashingCell', availableMenuItems);
-        // GridInfo
         const gridInfoMenuItems = this.getModuleSpecificStructure('GridInfo', availableMenuItems);
-        // Layout
         const [gridActionItems, otherLayoutItems] = this.getLayoutContextMenuStructure(availableMenuItems);
-        // Note
         const noteMenuItems = this.getModuleSpecificStructure('Note', availableMenuItems);
-        // SettingsPanel
         const settingsPanelMenuItems = this.getModuleSpecificStructure('SettingsPanel', availableMenuItems);
-        // SmartEdit
         const smartEditMenuItems = this.getModuleSpecificStructure('SmartEdit', availableMenuItems);
-        // SystemStatus
         const systemStatusMenuItems = this.getModuleSpecificStructure('SystemStatus', availableMenuItems);
-        /**
-         * Custom structures
-         */
         const gridMenuItem = {
             name: 'grid-group',
             label: 'Grid',
@@ -561,9 +466,6 @@ export class AgGridMenuAdapter {
             ...columnInfoMenuItems,
         ], '-');
     }
-    /**
-     * Hide menu group with no subitems or elevate single subitem to parent level
-     */
     normalizeMenuGroup(menuItem) {
         if (!menuItem.subItems?.length) {
             return [];
@@ -573,9 +475,6 @@ export class AgGridMenuAdapter {
         }
         return [menuItem];
     }
-    /**
-     * Default strategy for menu items: return as is if there is only one item, otherwise group them under a parent item
-     */
     getModuleSpecificStructure(module, menuItems, groupName) {
         const moduleItems = menuItems.filter((menuItem) => menuItem.category === module);
         if (moduleItems.length > 1) {
@@ -616,44 +515,24 @@ export class AgGridMenuAdapter {
         };
     }
     buildColumnMenuDefaultStructure(availableMenuItems, menuContext) {
-        // CalculatedColumn
         const calculatedColumnMenuItems = this.getModuleSpecificStructure('CalculatedColumn', availableMenuItems);
-        // CellSummary
         const cellSummaryMenuItems = this.getModuleSpecificStructure('CellSummary', availableMenuItems);
-        // Chart
         const chartMenuItems = this.getModuleSpecificStructure('Charting', availableMenuItems);
-        // ColumnFilter
         const [columnFilterGroup, filterVisibilityItems] = this.getColumnFilterColumnMenuStructure(availableMenuItems);
-        // ColumnInfo
         const columnInfoMenuItems = this.getModuleSpecificStructure('ColumnInfo', availableMenuItems);
-        // CustomSort
         const customSortMenuItems = this.getModuleSpecificStructure('CustomSort', availableMenuItems);
-        // Dashboard
         const dashboardMenuItems = this.getModuleSpecificStructure('Dashboard', availableMenuItems, 'dashboard-group');
-        // DataImport
         const dataImportMenuItems = this.getModuleSpecificStructure('DataImport', availableMenuItems);
-        // FlashingCell
         const flashingCellMenuItems = this.getModuleSpecificStructure('FlashingCell', availableMenuItems);
-        // FormatColumn
         const formatColumnMenuItems = this.getModuleSpecificStructure('FormatColumn', availableMenuItems);
-        // FreeTextColumn
         const freeTextColumnMenuItems = this.getModuleSpecificStructure('FreeTextColumn', availableMenuItems);
-        // GridInfo
         const gridInfoMenuItems = this.getModuleSpecificStructure('GridInfo', availableMenuItems);
-        // Layout
         const [gridSelectItems, columnSelectItems, columnActionGroup, otherLayoutItems] = this.getLayoutColumnMenuStructure(availableMenuItems);
         const selectionMenuItems = this.buildSelectColumnMenuGroup(gridSelectItems, columnSelectItems);
-        // PlusMinus
         const plusMinusMenuItems = this.getModuleSpecificStructure('PlusMinus', availableMenuItems);
-        // SettingsPanel
         const settingsPanelMenuItems = this.getModuleSpecificStructure('SettingsPanel', availableMenuItems);
-        // StyledColumn
         const styledColumnMenuItems = this.getStyledColumnColumnMenuStructure(availableMenuItems);
-        // SystemStatus
         const systemStatusMenuItems = this.getModuleSpecificStructure('SystemStatus', availableMenuItems);
-        /**
-         * Custom structures
-         */
         const gridMenuItem = {
             name: 'grid-group',
             label: 'Grid',
@@ -673,20 +552,6 @@ export class AgGridMenuAdapter {
                 ...gridInfoMenuItems,
             ],
         };
-        /*
-        const calculatedColumnMenuItem: AdaptableMenuItem = {
-          name: 'calculated-column-group',
-          label: 'Calculated Column',
-          module: 'CalculatedColumn',
-          isVisible: true,
-          icon: {
-            name: 'columns',
-          },
-          subItems: [
-            ...calculatedColumnMenuItems,
-          ],
-        };
-        */
         const columnMenuItem = {
             name: 'column-group',
             label: 'Column',
@@ -810,7 +675,6 @@ export class AgGridMenuAdapter {
             if (menuItem === '-') {
                 return menuItem;
             }
-            // @ts-ignore
             const subItems = this.mapAdaptableMenuItemToSystemMenuItems(menuItem.subItems);
             return {
                 ...menuItem,
@@ -829,7 +693,6 @@ export class AgGridMenuAdapter {
         });
         return columnMenuItems;
     }
-    // TODO AFL MIG: pretty sure this logic is duplicated in several other places
     mapAdaptableIconToAgGridIcon(adaptableIcon, style) {
         const icon = this.adaptableApi.userInterfaceApi.internalApi.prepareAdaptableIconDef(adaptableIcon);
         if (isAdaptableElementIcon(icon)) {
@@ -837,8 +700,6 @@ export class AgGridMenuAdapter {
             if (typeof element === 'string') {
                 return element;
             }
-            // THe element neets to be cloned.
-            // when it is used in more than one plce the element is removed from the DOM
             return element.cloneNode(true);
         }
         else {
@@ -847,10 +708,6 @@ export class AgGridMenuAdapter {
             });
         }
     }
-    /**
-     * The output of this function is used to build the column header menu if the AG Grid Menu Module is NOT present
-     * This is controlled by the AdaptableAgGrid.embedColumnMenu property
-     */
     buildStandaloneColumnHeader(adaptableColumn) {
         const agGridColumn = this._adaptableInstance.getAgGridColumnForColumnId(adaptableColumn.columnId);
         const menuContext = this.createColumnMenuContextObject(adaptableColumn, agGridColumn);

package/src/agGrid/AgGridModulesAdapter.js

@@ -19,8 +19,6 @@ export class AgGridModulesAdapter {
     isAgGridModuleRegistered(moduleName) {
         let isModuleRegistered = this.agGridApi.isModuleRegistered(moduleName);
         if (!isModuleRegistered) {
-            // SSRM module comes with its own implementations of Pivot, MasterDetail and TreeData
-            // #ssrm_pivot_module
             const serverSideDependencies = [
                 'PivotModule',
                 'MasterDetailModule',

package/src/agGrid/AgGridOptionsService.js

@@ -29,7 +29,6 @@ export class AgGridOptionsService {
         let adaptableValue = propertyGetter(userValue);
         if (typeof adaptableValue === 'function') {
             const previousValue = adaptableValue;
-            // @ts-ignore we know that adaptableValue is a function
             adaptableValue = (...args) => {
                 if (this.adaptableInstance.isDestroyed) {
                     return;
@@ -55,7 +54,6 @@ export class AgGridOptionsService {
     }
     revertGridOptionsPropertiesToUserValue(gridOptions, propertyNames) {
         for (const propertyName of propertyNames) {
-            // see this.setGridOptionsProperty(...)
             const userKey = `user.${propertyName}`;
             const userValue = this.gridOptionsPropertyCache.get(userKey);
             gridOptions[propertyName] = userValue;

package/src/agGrid/AgGridThemeAdapter.js

@@ -51,7 +51,6 @@ export class AgGridThemeAdapter {
             }
         }
         if (!adaptableTheme.AgGridClassName) {
-            // default AG Grid to its light theme
             adaptableTheme.AgGridClassName = getAgGridLightThemeName();
         }
         if (agGridContainer != null) {
@@ -62,7 +61,6 @@ export class AgGridThemeAdapter {
                     }
                 });
             }
-            // also remove all AG Grid theme class names
             const agGridClassNamesToRemove = [];
             agGridContainer.classList.forEach((x) => {
                 if (x && x.indexOf('ag-theme-') === 0) {
@@ -83,19 +81,16 @@ export class AgGridThemeAdapter {
         if (currentAgGridTheme === 'legacy') {
             return this.legacy_getAgGridCurrentThemeClassNames();
         }
-        // @ts-ignore no other way than to use internals
         const currentAgGridTheme__cssClassCache = currentAgGridTheme?._cssClassCache;
         return currentAgGridTheme__cssClassCache ?? '';
     }
     getAgGridLightThemeName() {
         const container = this.getAgGridContainerElement();
         if (container && container.classList) {
-            // we detect the ag theme class
             const classList = container.classList;
             for (let i = 0, len = classList.length; i < len; i++) {
                 const cls = classList[i];
                 if (cls.indexOf('ag-theme-') === 0) {
-                    // even if dark theme is included, we compute the light theme name out of it
                     return cls.replace('-dark', '');
                 }
             }
@@ -104,13 +99,11 @@ export class AgGridThemeAdapter {
             this.logger.warn('AG Grid container not found. Defaulting to ag-theme-balham for the light theme.');
         }
         this.logger.warn('No ag-theme-* class found on the grid container. Defaulting to ag-theme-balham.');
-        // fallback to the default light theme
         return 'ag-theme-balham';
     }
     legacy_getAgGridCurrentThemeClassNames() {
         const container = this.getAgGridContainerElement();
         if (container && container.classList) {
-            // we detect the ag theme class
             const classList = container.classList;
             for (let i = 0, len = classList.length; i < len; i++) {
                 const cls = classList[i];

package/src/agGrid/agGridDataTypeDefinitions.js

@@ -14,13 +14,10 @@ export const ALL_ADAPTABLE_DATA_TYPES = [
     OBJECT_ARRAY_DATA_TYPE,
     'unknown',
 ];
-// #agGridDataTypeDefinitions
 export const agGridDataTypeDefinitions = {
-    // AG Grid default types
     number: {
         baseDataType: 'number',
         extendsDataType: 'number',
-        // suppress AG Grid editor, we provide our own
         suppressDefaultProperties: true,
     },
     date: {
@@ -30,11 +27,6 @@ export const agGridDataTypeDefinitions = {
         valueFormatter: null,
         suppressDefaultProperties: true,
     },
-    // dateString is force-cast to date in AdapTable, see #data_type_dateString
-    // text: is kept as is
-    // boolean: is kept as is
-    // object: is kept as is
-    // Adaptable specific types
     unknown: {
         baseDataType: 'text',
         extendsDataType: 'text',

package/src/agGrid/cellRenderers/BadgeRenderer.js

@@ -7,11 +7,6 @@ const OVERFLOW_CLASS = {
     Wrap: 'ab-Badge__wrapper--wrap',
     Scroll: 'ab-Badge__wrapper--scroll',
 };
-/**
- * Maps a column-level `BadgeStyle.Font.Alignment` to the flex
- * `justify-content` value for the badge row. Returns `null` to mean
- * "leave the wrapper alone" so the default flex layout applies.
- */
 const resolveBadgeRowJustify = (alignment) => {
     switch (alignment) {
         case 'Center':
@@ -43,12 +38,9 @@ export const getBadgeRendererForColumn = (styledColumn, abColumn, api) => {
             this.eGui = document.createElement('div');
             this.eGui.className = wrapperClassName;
             if (spacing != null) {
-                // Set the inter-badge gap directly so it doesn't depend on theme CSS.
                 this.eGui.style.setProperty('--ab-cmp-badge__spacing', `${spacing}px`);
             }
             if (rowJustify) {
-                // Position the badge row within the cell. `width: 100%` is needed so
-                // `justify-content` has room to act on an `inline-flex` container.
                 this.eGui.style.setProperty('justify-content', rowJustify);
                 this.eGui.style.setProperty('width', '100%');
             }
@@ -110,7 +102,6 @@ export const getBadgeRendererForColumn = (styledColumn, abColumn, api) => {
             };
             const badge = api.styledColumnApi.internalApi.getApplicableBadge(badgeStyle, predicateDefHandlerContext);
             const isNullValue = formattedValue === '' || formattedValue === null || formattedValue === undefined;
-            // Return if empty values or no badge
             if (isNullValue || !badge) {
                 this.eGui.innerHTML = formattedValue;
                 return;
@@ -157,7 +148,6 @@ export const getBadgeRendererForColumn = (styledColumn, abColumn, api) => {
             this.unmountReactRoot?.();
         }
         refresh(params) {
-            // by returning FALSE we force a re-render every time the cell value changes
             return false;
         }
     };