@adaptabletools/adaptable 22.0.0-canary.5 → 22.0.0-canary.7

package/src/agGrid/AdaptableAgGrid.js

@@ -359,7 +359,7 @@ export class AdaptableAgGrid {
         this.adaptableOptions = this.normalizeAdaptableOptions(this.adaptableOptions);
         const { showLoadingScreen, loadingScreenDelay, loadingScreenText, loadingScreenTitle } = this.adaptableOptions.userInterfaceOptions.loadingScreenOptions;
         if (showLoadingScreen) {
-            this.logger.info(`Show Loading Screen`);
+            this.logger.info('Showing loading screen');
             // it's important to use ensureLoadingScreenPortalElement
             // and not ensurePortalElement, because multiple adaptable instances share the same portal element
             // so when displaying the second one, the react root associated to the portal element
@@ -375,7 +375,7 @@ export class AdaptableAgGrid {
                 }), portalElement);
             }
             else {
-                this.logger.consoleError(`Adaptable failed to show the loading screen!`);
+                this.logger.consoleError('Failed to create loading screen portal element.');
             }
         }
         this.forPlugins((plugin) => plugin.afterInitOptions(this, this.adaptableOptions));
@@ -434,7 +434,7 @@ export class AdaptableAgGrid {
         // see #map_dateString_to_date
         this.agGridAdapter.patchColDefs(columnDefs, (colDef) => {
             if (colDef.cellDataType === 'dateString') {
-                this.logger.consoleError(`AG Grid: Column '${colDef.colId}' specifies cellDataType='dateString' which is no longer supported. It has been automatically replaced with cellDataType='date'. Please update your column definition.`);
+                this.logger.consoleError(`Column "${colDef.colId}" uses cellDataType="dateString" which is no longer supported. It has been replaced with cellDataType="date". Please update your column definition.`);
                 colDef.cellDataType = 'date';
             }
         });
@@ -465,8 +465,8 @@ export class AdaptableAgGrid {
         const agGridApi = await this.initializeAgGrid(gridOptions, config.modules, config.renderAgGridFrameworkComponent);
         if (agGridApi === false) {
             this.midwayDestroy();
-            this.logger.consoleError(`Adaptable failed to initialize AG Grid!`);
-            return Promise.reject('Adaptable failed to initialize AG Grid!');
+            this.logger.consoleError('Failed to initialize AG Grid.');
+            return Promise.reject('Failed to initialize AG Grid.');
         }
         this.layoutManager = new LayoutManager({
             gridApi: agGridApi,
@@ -491,7 +491,7 @@ export class AdaptableAgGrid {
         this.layoutManager.onColumnDefsChanged(() => {
             this.updateColumnModelAndRefreshGrid();
         });
-        this.logger.info(`Hide Loading Screen`);
+        this.logger.info('Hiding loading screen');
         this.unmountLoadingScreen?.();
         perfInitAgGrid.end();
         initAgGridMarker.end();
@@ -502,7 +502,7 @@ export class AdaptableAgGrid {
         this.lifecycleState = 'agGridReady';
         this.api.entitlementApi.internalApi.setModulesAgGridDepsInfos();
         this.ModuleService.logMissingAgGridDepsInfos();
-        this.logger.info(`Registered AG Grid modules (incl. dependent modules): `, this.agGridModulesAdapter.getAgGridRegisteredModuleNames().sort());
+        this.logger.info('Registered AG Grid modules (including dependents):', this.agGridModulesAdapter.getAgGridRegisteredModuleNames().sort());
         /**
          * At this point AG Grid is initialized!
          */
@@ -592,10 +592,7 @@ export class AdaptableAgGrid {
         const layoutState = state.Layout;
         // ensure that at least one Layout has been provided
         if (!layoutState || !layoutState.Layouts?.length) {
-            this.logger
-                .consoleError(`You have not defined any Layout in your InitialState.Layout.Layouts[] state!
-
-You need to define at least one Layout!`);
+            this.logger.consoleError('No Layouts defined in InitialState.Layout.Layouts[]. At least one Layout is required.');
         }
         // ensure CurrentLayout is valid
         if (!layoutState.CurrentLayout ||
@@ -760,19 +757,12 @@ You need to define at least one Layout!`);
     }
     normalizeAdaptableOptions(adaptableOptions) {
         if (this.hasAutogeneratedPrimaryKey) {
-            this.logger
-                .warn(`Autogenerated primary key (adaptableOptions.autogeneratedPrimaryKey = TRUE) should be used only as a last resort,
-      when no unique column is available, as it limits some Adaptable functionalities!
-
-      For more details see: ${PrimaryKeyDocsLink}`);
+            this.logger.warn(`Autogenerated primary key is enabled (adaptableOptions.autogeneratedPrimaryKey = true). This should be a last resort as it limits some Adaptable features. See: ${PrimaryKeyDocsLink}`);
             this.adaptableOptions.primaryKey = AUTOGENERATED_PK_COLUMN;
             return this.adaptableOptions;
         }
         if (StringExtensions.IsNullOrEmpty(adaptableOptions.primaryKey)) {
-            this.logger.consoleError(`AdaptableOptions.primaryKey is required and cannot be empty or null!
-      As a fallback, you can set adaptableOptions.autogeneratedPrimaryKey = TRUE
-
-      For more details see: ${PrimaryKeyDocsLink}`);
+            this.logger.consoleError(`adaptableOptions.primaryKey is required and must not be empty. As a fallback, set adaptableOptions.autogeneratedPrimaryKey = true. See: ${PrimaryKeyDocsLink}`);
         }
         return adaptableOptions;
     }
@@ -804,7 +794,7 @@ You need to define at least one Layout!`);
          */
         this.agGridOptionsService.setGridOptionsProperty(gridOptions, 'defaultColDef', (original_defaultColDef) => {
             if (original_defaultColDef?.headerValueGetter) {
-                this.logger.warn(`defaultColDef.headerValueGetter and overrides the Adaptable custom header mechanism! We recommend using a ColumnOptions.columnHeader instead!`);
+                this.logger.warn('defaultColDef.headerValueGetter overrides the Adaptable header mechanism. Consider using ColumnOptions.columnHeader instead.');
                 return original_defaultColDef;
             }
             // #customize_header
@@ -821,7 +811,7 @@ You need to define at least one Layout!`);
          */
         this.agGridOptionsService.setGridOptionsProperty(gridOptions, 'defaultColGroupDef', (original_defaultColGroupDef) => {
             if (original_defaultColGroupDef?.headerValueGetter) {
-                this.logger.warn(`defaultColGroupDef.headerValueGetter and overrides the Adaptable custom header mechanism! We recommend using a ColumnOptions.tableColumnHeader instead!`);
+                this.logger.warn('defaultColGroupDef.headerValueGetter overrides the Adaptable header mechanism. Consider using ColumnOptions.tableColumnHeader instead.');
                 return original_defaultColGroupDef;
             }
             const defaultColGroupDef = { ...original_defaultColGroupDef };
@@ -841,12 +831,11 @@ You need to define at least one Layout!`);
             }
             if (original_autoGroupColumnDef?.filter === false) {
                 // if user disables filter on autoGroupColumnDef, we respect that
-                this.logger.info(`autoGroupColumnDef.filter is disabled as per user configuration.`);
+                this.logger.info('autoGroupColumnDef.filter is disabled via user configuration.');
                 return original_autoGroupColumnDef;
             }
             if (typeof original_autoGroupColumnDef?.filter === 'object') {
-                this.logger.consoleWarn(`autoGroupColumnDef.filter is set and overrides the Adaptable custom filter mechanism, so Adaptable will not be able to manage filtering for the Group Column!
-                      Please contact the Adaptable Support Team for further assistance and investigation.`);
+                this.logger.consoleWarn('autoGroupColumnDef.filter overrides the Adaptable filtering mechanism for the Group Column. Adaptable-managed filters will not apply to this column. Contact support for alternatives.');
                 return original_autoGroupColumnDef;
             }
             const autoGroupColumnDef = { ...original_autoGroupColumnDef };
@@ -854,8 +843,7 @@ You need to define at least one Layout!`);
             // !! DO NOT REMOVE THIS !!
             // see https://github.com/ag-grid/ag-grid/blob/6f43ff257c2e285068eb425b655e58d6eeb89816/packages/ag-grid-enterprise/src/rowHierarchy/autoColService.ts#L225
             autoGroupColumnDef.filterValueGetter = (params) => {
-                this.logger.consoleWarn(`An unexpected invocation of autoGroupColumnDef.filterValueGetter was detected. This execution path is not expected under normal operation.
-                      Please contact the Adaptable Support Team for further assistance and investigation.`, params);
+                this.logger.consoleWarn('Unexpected invocation of autoGroupColumnDef.filterValueGetter. This is not expected under normal operation. Please contact support.', params);
                 return '';
             };
             // !! it is important that we always return the same component / handler instances
@@ -1425,15 +1413,14 @@ You need to define at least one Layout!`);
             const rowData = agGridApi.getGridOption('rowData');
             this.initWithLazyData = rowData == undefined || rowData.length === 0;
             if (this.initWithLazyData) {
-                this.logger.info('initWithLazyData = TRUE');
+                this.logger.info('Initializing with lazy data (no initial rowData).');
             }
             if (!this.getAgGridContainerElement()) {
                 // initialize the agGridContainerElement from the AgGrid instance
                 const gridRoot = this.agGridAdapter.getAgGridRootElement();
                 const gridContainer = gridRoot?.closest('[class*="ag-theme"]');
                 if (!gridContainer) {
-                    this.logger.consoleError(`No AG Grid container could be derived from the Adaptable framework wrapper.
-            Please contact AdapTable Support and in the meantime provide a valid container element in 'ContainerOptions.agGridContainer'!`);
+                    this.logger.consoleError('Failed to derive AG Grid container from the framework wrapper. Please provide a valid container element via ContainerOptions.agGridContainer.');
                 }
                 this.DANGER_USE_GETTER_agGridContainerElement = gridContainer;
             }
@@ -1441,7 +1428,7 @@ You need to define at least one Layout!`);
         }
         const agGridContainer = this.getAgGridContainerElement();
         if (!agGridContainer) {
-            this.logger.consoleError('No AG Grid container element found in the DOM. Please provide a valid container element in `ContainerOptions.agGridContainer`');
+            this.logger.consoleError('AG Grid container element not found in the DOM. Please provide a valid element via ContainerOptions.agGridContainer.');
             return Promise.resolve(false);
         }
         let gridParams;
@@ -1856,7 +1843,7 @@ You need to define at least one Layout!`);
             // if no primary key column then lets check the first row to see if its a data item
             const primaryKeyDataItem = this.getFirstRowNode()?.data[primaryKey];
             if (!primaryKeyDataItem) {
-                errorMessage = `The Primary Key '${this.adaptableOptions.primaryKey}' does not exist.  This will affect many functions in AdapTable.`;
+                errorMessage = `Primary key column "${this.adaptableOptions.primaryKey}" not found. This will affect many Adaptable features.`;
                 if (this.adaptableOptions.alertOptions.showMissingPrimaryKeyAlert) {
                     // show an alert if that is the option
                     this.api.alertApi.showAlertError('No Primary Key', errorMessage);
@@ -1913,12 +1900,12 @@ You need to define at least one Layout!`);
     checkShouldClearExistingFiltersOrSearches() {
         // if they have selected to clear Filters on startup then do it
         if (this.adaptableOptions.filterOptions.clearFiltersOnStartUp) {
-            this.logger.warn('Clearing any saved Filters as "clearFiltersOnStartUp" is true');
+            this.logger.warn('Clearing saved filters (clearFiltersOnStartUp is enabled).');
             this.api.filterApi.clearAllFilters();
         }
         // if they have selected to clear searches on startup then do it
         if (this.adaptableOptions.quickSearchOptions.clearQuickSearchOnStartUp) {
-            this.logger.warn('Clearing any saved Searches as "clearQuickSearchOnStartUp" is true');
+            this.logger.warn('Clearing saved Quick Search (clearQuickSearchOnStartUp is enabled).');
             this.api.quickSearchApi.clearQuickSearch();
         }
     }
@@ -2147,7 +2134,7 @@ You need to define at least one Layout!`);
         }
     }
     _updateColumnModelAndRefreshGridNow() {
-        this.logger.info(`Updating Column Model and Refreshing Grid.`);
+        this.logger.info('Updating column model and refreshing grid.');
         this.deriveAdaptableColumnStateFromAgGrid();
         this.agGridColumnAdapter.setupColumns();
         this.redrawBody();
@@ -2173,7 +2160,7 @@ You need to define at least one Layout!`);
             this.agGridAdapter.getAgGridApi().redrawRows(redrawRowsParams);
         }
         catch (ex) {
-            this.logger.consoleError('AG Grid redrawRows was unable to find some row nodes. Tried to redraw row nodes: ', rowNodes, ex);
+            this.logger.consoleError('AG Grid redrawRows failed to locate some row nodes.', rowNodes, ex);
         }
     }
     redrawRow(rowNode) {
@@ -2583,14 +2570,14 @@ You need to define at least one Layout!`);
     }
     autoSizeColumns(columnIds) {
         if (!this.agGridModulesAdapter.isAgGridModuleRegistered('ColumnAutoSizeModule')) {
-            this.logger.consoleError(`Could not auto-size columns [${columnIds}] - missing required AG Grid module 'ColumnAutoSizeModule'`);
+            this.logger.consoleError(`Auto-sizing columns [${columnIds}] requires the AG Grid "ColumnAutoSizeModule". Please register this module.`);
             return;
         }
         this.layoutManager.autoSizeColumns(columnIds);
     }
     autoSizeAllColumns() {
         if (!this.agGridModulesAdapter.isAgGridModuleRegistered('ColumnAutoSizeModule')) {
-            this.logger.consoleError(`Could not auto-size all columns - missing required AG Grid module 'ColumnAutoSizeModule'`);
+            this.logger.consoleError('Auto-sizing all columns requires the AG Grid "ColumnAutoSizeModule". Please register this module.');
             return;
         }
         const agGridApi = this.agGridAdapter.getAgGridApi();
@@ -2748,7 +2735,7 @@ You need to define at least one Layout!`);
                 customInFilterValuesResult = await customInFilterValues(customInFilterValuesContext);
             }
             catch (error) {
-                this.logger.consoleError(`Failed to fetch custom filter values`, column, error);
+                this.logger.consoleError('Failed to retrieve custom filter values.', column, error);
             }
             return customInFilterValuesResult;
         }
@@ -2788,7 +2775,7 @@ You need to define at least one Layout!`);
                     customDistinctValues = await customEditColumnValues(customEditValuesContext);
                 }
                 catch (error) {
-                    this.logger.consoleError(`Failed to fetch custom edit values`, column, error);
+                    this.logger.consoleError('Failed to retrieve custom edit values.', column, error);
                 }
                 return customDistinctValues ?? [];
             }
@@ -3243,7 +3230,7 @@ You need to define at least one Layout!`);
         if (this.getAgGridRowModelType() !== 'clientSide') {
             // only in client-side row model can we loop through filtered&sorted rows
             // see https://www.ag-grid.com/javascript-data-grid/accessing-data/#iterating-rows
-            this.logger.warn('`forAllVisibleRowNodesDo()` is only supported in client-side row model. `forAllRowNodesDo` will be used instead.');
+            this.logger.warn('forAllVisibleRowNodesDo() is only supported with client-side row model. Falling back to forAllRowNodesDo().');
             return this.forAllRowNodesDo(func, config);
         }
         this.agGridAdapter.getAgGridApi().forEachNodeAfterFilterAndSort((rowNode, rowIndex) => {
@@ -3295,14 +3282,14 @@ You need to define at least one Layout!`);
     }
     selectNode(rowNode, clearSelection) {
         if (!rowNode) {
-            this.logger.error('No node to select');
+            this.logger.error('Cannot select: row node is null.');
             return;
         }
         rowNode.setSelected(true, clearSelection);
     }
     deSelectNode(rowNode, clearSelection) {
         if (!rowNode) {
-            this.logger.error('No node to deselect');
+            this.logger.error('Cannot deselect: row node is null.');
             return;
         }
         rowNode.setSelected(false, clearSelection);
@@ -3390,7 +3377,7 @@ You need to define at least one Layout!`);
     }
     showChart(chartDefinition, container) {
         if (!this.isReady) {
-            this.logger.consoleError('Adaptable must be instantiated before calling showChart');
+            this.logger.consoleError('Cannot call showChart before Adaptable is fully initialized.');
             return null;
         }
         /**
@@ -3411,7 +3398,7 @@ You need to define at least one Layout!`);
     }
     updateChart(chart) {
         if (!chart.Model) {
-            this.logger.consoleError(`Chart definition must have a Model property to be updated: ${chart.Name}`);
+            this.logger.consoleError(`Cannot update chart "${chart.Name}": the chart definition is missing the required Model property.`);
             return;
         }
         const upgradableProperties = {
@@ -3423,12 +3410,21 @@ You need to define at least one Layout!`);
             suppressChartRanges: Boolean(chart.Model.suppressChartRanges),
             aggFunc: chart.Model.aggFunc,
         };
+        // Skip the AG Grid API call if the upgradable properties haven't changed
+        // to avoid unnecessary chartOptionsChanged events (see #ag_grid_update_chart_loop)
+        const currentModel = this.getChartModels().find((model) => model.chartId === chart.Model.chartId);
+        if (currentModel &&
+            Boolean(currentModel.unlinkChart) === upgradableProperties.unlinkChart &&
+            Boolean(currentModel.suppressChartRanges) === upgradableProperties.suppressChartRanges &&
+            currentModel.aggFunc === upgradableProperties.aggFunc) {
+            return;
+        }
         // see also #ag_grid_update_chart_loop
         this.agGridAdapter.getAgGridApi().updateChart(upgradableProperties);
     }
     getChartModels() {
         if (!this.isReady) {
-            this.logger.consoleError('Adaptable must be instantiated before calling getChartModels');
+            this.logger.consoleError('Cannot call getChartModels before Adaptable is fully initialized.');
             return [];
         }
         return this.agGridAdapter.getAgGridApi().getChartModels();
@@ -3462,7 +3458,7 @@ You need to define at least one Layout!`);
     }
     setAgGridQuickSearch(searchText) {
         if (!this.agGridModulesAdapter.isAgGridModuleRegistered('QuickFilterModule')) {
-            this.logger.consoleError(`Can NOT set Quick Search text - missing required AG Grid module 'QuickFilterModule'`);
+            this.logger.consoleError('Quick Search requires the AG Grid "QuickFilterModule". Please register this module to enable Quick Search.');
             return;
         }
         this.agGridAdapter.setGridOption('quickFilterText', searchText);
@@ -3472,7 +3468,7 @@ You need to define at least one Layout!`);
     }
     setAgGridFindSearchValue(searchText) {
         if (!this.agGridModulesAdapter.isAgGridModuleRegistered('FindModule')) {
-            this.logger.consoleError(`Can NOT set Find Search text - missing required AG Grid module 'FindModule'`);
+            this.logger.consoleError('Find Search requires the AG Grid "FindModule". Please register this module to enable Find Search.');
             return;
         }
         this.agGridAdapter.setGridOption('findSearchValue', searchText);
@@ -4001,7 +3997,7 @@ You need to define at least one Layout!`);
         cellDataChangedInfos.forEach((cellDataChangedInfo) => {
             // if a Cell Data Change is undone, log to the Console
             if (cellDataChangedInfo.trigger === 'undo') {
-                this.logger.info(`Undo data change: PK(${cellDataChangedInfo.primaryKeyValue}) Col(${cellDataChangedInfo.column}) RevertedValue(${cellDataChangedInfo.oldValue}) OriginalValue(${cellDataChangedInfo.newValue})`);
+                this.logger.info(`Data change reverted: column="${cellDataChangedInfo.column}", primaryKey="${cellDataChangedInfo.primaryKeyValue}", from=${cellDataChangedInfo.newValue} to=${cellDataChangedInfo.oldValue}`);
             }
             // For Changes except Aggregations, reset cell selection if required
             if (cellDataChangedInfo.trigger === 'edit' ||
@@ -4206,7 +4202,7 @@ You need to define at least one Layout!`);
         this.api.layoutApi.createOrUpdateLayout(layout);
     }
     refreshAdaptableAfterLayoutChange(layout) {
-        this.logger.info('refreshAdaptableAfterLayoutChange()');
+        this.logger.info('Refreshing Adaptable after layout change.');
         const prevLayoutForRefresh = this.__prevLayoutForRefresh || this.api.layoutApi.getCurrentLayout();
         // see #on-regroup-expect-group-column-to-be-recomputed-and-setup-properly
         const rowGroupsChanged = this.isRowGroupDifferentInLayout(prevLayoutForRefresh, layout);
@@ -4252,7 +4248,7 @@ You need to define at least one Layout!`);
                 .forEach((colType) => {
                 const newValidCellDataType = mapOldTypeToDataType(colType);
                 if (newValidCellDataType) {
-                    this.logger.consoleWarn(`Column ${columnDefinition.colId ?? columnDefinition.field} has a type of ${colType} which is no longer in use${newValidCellDataType !== 'unknown' ? `. Please use 'colDef.cellDataType' instead: ${newValidCellDataType}` : ''}`);
+                    this.logger.consoleWarn(`Column "${columnDefinition.colId ?? columnDefinition.field}" uses deprecated type "${colType}"${newValidCellDataType !== 'unknown' ? `. Use colDef.cellDataType="${newValidCellDataType}" instead.` : '. This type is no longer supported.'}`);
                 }
             });
         });

package/src/agGrid/AgGridAdapter.js

@@ -145,7 +145,7 @@ export class AgGridAdapter {
                     }
                 }
                 options['columnDefs'] = colDefsWithSpecialColumns;
-                self.logger.info(`Added SpecialColumns on GridOptions.columnDefs update (source=${source})`);
+                self.logger.info(`Added special columns on GridOptions.columnDefs update (source="${source}").`);
             }
             // `context`
             const passedContext = options.context;
@@ -165,7 +165,7 @@ export class AgGridAdapter {
         }
         const agGridColumnFilterService = this.DANGER_getPrivateAgGridBeans()?.colFilter;
         if (!agGridColumnFilterService) {
-            this.logger.consoleError('Could not get hold of ColumnFilterService! This is a critical error and will prevent Adaptable from working correctly.');
+            this.logger.consoleError('Failed to initialize ColumnFilterService. Filtering and related features will not function correctly.');
             return;
         }
         const self = this;
@@ -211,7 +211,7 @@ export class AgGridAdapter {
     DANGER_getPrivateAgGridBeans() {
         const beans = DANGER_AG_GRID_BEANS_MAP[this._agGridId];
         if (!beans) {
-            this.logger.consoleError('Could not get hold of AgGridBeans! This is a critical error and will prevent Adaptable from working correctly.');
+            this.logger.consoleError('Failed to access AG Grid internal beans. Adaptable will not function correctly.');
         }
         return beans;
     }
@@ -355,7 +355,7 @@ export class AgGridAdapter {
                     if (isEndRowPin) {
                         shouldIncludeRange = false;
                     }
-                    this.logger.consoleWarn('Cannot select pinned rows in AG Grid.');
+                    this.logger.consoleWarn('Pinned rows cannot be selected in AG Grid.');
                 }
                 if (shouldIncludeRange) {
                     const y1 = Math.min(rangeSelection.startRow.rowIndex, rangeSelection.endRow.rowIndex);
@@ -525,7 +525,7 @@ export class AgGridAdapter {
     deriveAdaptableColumnDataType(agColumn, logWarning = true) {
         // Some columns can have no ID or Title. we return string as a consequence but it needs testing
         if (!agColumn) {
-            this.logger.warn(`Column is undefined, returning 'text' for Type`);
+            this.logger.warn('Column is undefined. Defaulting data type to "text".');
             return 'text';
         }
         if (this.adaptableApi.columnApi.isAutoRowGroupColumnForSingle(agColumn.getId())) {
@@ -568,7 +568,7 @@ export class AgGridAdapter {
         let row = this.getAgGridApi().getDisplayedRowAtIndex(0);
         if (row == null) {
             // possible that there will be no data.
-            this.logger.consoleError(`No data in grid, returning type "unknown" for Column: "${agColumn.getColId()}". This will impact several Adaptable features, such as Filters and ColumnFormats.`);
+            this.logger.consoleError(`No data in grid. Returning type "unknown" for column "${agColumn.getColId()}". This will affect features such as Filters and Column Formats.`);
             return 'unknown';
         }
         // // if it's a group we need the content of the group
@@ -610,7 +610,7 @@ export class AgGridAdapter {
                 dataType = arrayDataType;
             }
         }
-        this.logger.consoleWarn(`No defined type for column '${agColumn.getColId()}'. Defaulting to type of first row value: ${dataType}`);
+        this.logger.consoleWarn(`No explicit type for column "${agColumn.getColId()}". Inferred type from first row value: "${dataType}".`);
         return dataType;
     }
     isColumnReadonly(colDef) {
@@ -796,7 +796,7 @@ export class AgGridAdapter {
                 colDef.colId = colDef.field;
             }
             if (!colDef.colId) {
-                this.logger.warn('A column is missing the colId - please check ', colDef, 'Either pass a "field" property or a "colId" property.');
+                this.logger.warn('Column definition is missing colId. Provide either a "field" or "colId" property.', colDef);
             }
         };
         this.patchColDefs(colDefs, assignColId);

package/src/agGrid/AgGridColumnAdapter.d.ts

@@ -71,6 +71,7 @@ export declare class AgGridColumnAdapter {
     private getAlertCellStyle;
     private getFlashingCellStyle;
     private getCellHighlightStyle;
+    private getColumnHighlightStyle;
     isColGroupDef(columnDefinition: ColDef | ColGroupDef): columnDefinition is ColGroupDef;
     private getRelevantFormatColumnHeaderStyles;
 }

package/src/agGrid/AgGridColumnAdapter.js

@@ -398,6 +398,7 @@ export class AgGridColumnAdapter {
                     ...this.getAlertCellStyle(gridCell, params),
                     ...this.getFlashingCellStyle(gridCell, params),
                     ...this.getCellHighlightStyle(gridCell, params),
+                    ...this.getColumnHighlightStyle(gridCell, params),
                 };
                 return normalizeStyleForAgGrid(result);
             };
@@ -655,7 +656,7 @@ export class AgGridColumnAdapter {
             if (typeof original_filter !== 'boolean' &&
                 typeof original_filter?.handler !== 'function' &&
                 !pivotMode) {
-                this.adaptableApi.consoleError(`Column '${colId}' has a custom filter defined in colDef.filter, but Adaptable Filtering accepts only the TRUE/FALSE values!`);
+                this.adaptableApi.consoleError(`Column "${colId}" has a custom filter in colDef.filter, but Adaptable filtering only accepts boolean (true/false) values for this property.`);
                 return false;
             }
             return {
@@ -1289,14 +1290,24 @@ export class AgGridColumnAdapter {
                 : flashingCell.flashingCellDefinition.NeutralChangeStyle) ?? {});
     }
     getCellHighlightStyle(gridCell, params) {
-        const cellHightlight = this.adaptableApi.internalApi
+        const cellHighlight = this.adaptableApi.internalApi
             .getInternalState()
             .CellHighlightInfo.find((cellHighlightInfo) => {
             return (gridCell.column.columnId === cellHighlightInfo.columnId &&
                 cellHighlightInfo.primaryKeyValue === gridCell.primaryKeyValue);
         });
-        if (cellHightlight) {
-            return convertAdaptableStyleToCSS(cellHightlight.highlightStyle);
+        if (cellHighlight) {
+            return convertAdaptableStyleToCSS(cellHighlight.highlightStyle);
+        }
+    }
+    getColumnHighlightStyle(gridCell, params) {
+        const columnHighlight = this.adaptableApi.internalApi
+            .getInternalState()
+            .ColumnHighlightInfo.find((columnHighlightInfo) => {
+            return (gridCell.column.columnId === columnHighlightInfo.columnId);
+        });
+        if (columnHighlight) {
+            return convertAdaptableStyleToCSS(columnHighlight.highlightStyle);
         }
     }
     isColGroupDef(columnDefinition) {

package/src/agGrid/AgGridExportAdapter.js

@@ -66,7 +66,7 @@ export class AgGridExportAdapter {
         try {
             if (showProgressIndicator) {
                 this.adaptableApi.userInterfaceApi.showProgressIndicator({
-                    text: `${report.Name} Export in progress...`,
+                    text: `Exporting ${report.Name}...`,
                 });
                 // waitForTimeout required to give the ProgressIndicator rendering a head-start (see rAF in ProgressIndicator implementation)
                 // see #raf_progress_indicator
@@ -120,7 +120,7 @@ export class AgGridExportAdapter {
             };
         }
         catch (error) {
-            this.logger.consoleError(`Error exporting ${report.Name} in ${format} format to ${config.destination}`, error.stack);
+            this.logger.consoleError(`Export failed: report="${report.Name}", format="${format}", destination="${config.destination}".`, error.stack);
         }
         finally {
             /**
@@ -142,7 +142,7 @@ export class AgGridExportAdapter {
         const exportContext = this.buildExportProcessContext(config);
         let exportParams = this.buildExportParams(exportContext);
         if (typeof config?.customExportParams === 'function') {
-            this.logger.info(`Custom export params for ${config.report.Name} in ${config.format} format`);
+            this.logger.info(`Applying custom export params: report="${config.report.Name}", format="${config.format}".`);
             exportParams = config.customExportParams(exportParams);
         }
         exportContext.exportedColumnIds = exportParams.columnKeys;
@@ -508,7 +508,7 @@ export class AgGridExportAdapter {
             const columnId = column.getId();
             const adaptableColumn = getAdaptableColumnWithColumnId(columnId);
             if (!adaptableColumn) {
-                this.logger.warn(`Export Styling: Column with id ${columnId} not found in Adaptable`);
+                this.logger.warn(`Export styling: column "${columnId}" not found in Adaptable.`);
                 return;
             }
             let headerClassParams = {
@@ -571,7 +571,7 @@ export class AgGridExportAdapter {
                 const columnId = column.getId();
                 const adaptableColumn = getAdaptableColumnWithColumnId(columnId);
                 if (!adaptableColumn) {
-                    this.logger.warn(`Export Styling: Column with id ${columnId} not found in Adaptable`);
+                    this.logger.warn(`Export styling: column "${columnId}" not found in Adaptable.`);
                     return;
                 }
                 const isDateCellExportedAsFormattedValue = this.isDateCellExportedAsFormattedValue(adaptableColumn, exportContext);

package/src/agGrid/AgGridThemeAdapter.js

@@ -101,9 +101,9 @@ export class AgGridThemeAdapter {
             }
         }
         else {
-            this.logger.warn('No AgGrid container found, defaulting to ag-theme-balham for the light theme');
+            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');
+        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';
     }

package/src/components/OverlayTrigger/index.js

@@ -152,7 +152,7 @@ const OverlayTrigger = React.forwardRef((givenProps, ref) => {
             target = targetProp(target);
         }
         if (!target) {
-            adaptable.logger.warn('No OverlayTrigger target - make sure you render a child inside the OverlayTrigger, which will be the overlay target');
+            adaptable.logger.warn('OverlayTrigger target not found. Ensure a child element is rendered inside the OverlayTrigger component.');
             return;
         }
         targetRef.current = target;