@adaptabletools/adaptable 23.0.4 → 23.0.5-canary.0

package/src/View/FormatColumn/Wizard/FormatColumnFormatWizardSection.js

@@ -112,8 +112,6 @@ export const getFormatColumnDisplayFormatSummaryItems = (data) => {
     if (isAdaptableNumericFormatPreset(data.DisplayFormat)) {
         return [{ label: 'Preset', value: NUMERIC_PRESET_LABELS[data.DisplayFormat] }];
     }
-    // Wizard materialises an empty { Formatter, Options: {} } shell while editing;
-    // treat it as unset (same as handleFinish in FormatColumnWizard).
     if (data.DisplayFormat &&
         typeof data.DisplayFormat === 'object' &&
         isObjectEmpty(data.DisplayFormat.Options)) {
@@ -180,13 +178,10 @@ export const getFormatDisplayTypeForScope = (scope, api) => {
     if ('All' in scope) {
         return undefined;
     }
-    // Date-related data types that should all be treated as 'date'
     const dateTypes = api.columnApi.internalApi.getColumnDateTypes();
-    // Check for ColumnIds in scope
     if ('ColumnIds' in scope) {
         const columns = scope.ColumnIds.map((c) => api.columnApi.getColumnWithColumnId(c)).filter(Boolean);
         const columnDataTypes = uniq(columns.map((c) => c.dataType));
-        // If there's only one data type
         if (columnDataTypes.length === 1) {
             const dataType = columnDataTypes[0];
             if (dataType === 'number') {
@@ -199,14 +194,12 @@ export const getFormatDisplayTypeForScope = (scope, api) => {
                 return 'text';
             }
         }
-        // If there are multiple data types but they're all date types
         else if (columnDataTypes.length > 0 &&
             columnDataTypes.every((type) => dateTypes.includes(type))) {
             return 'date';
         }
         return undefined;
     }
-    // Check for DataTypes in scope
     if ('DataTypes' in scope && scope.DataTypes.length === 1) {
         const dataType = scope.DataTypes[0];
         if (dataType === 'number') {
@@ -219,34 +212,27 @@ export const getFormatDisplayTypeForScope = (scope, api) => {
             return 'text';
         }
     }
-    // If there are multiple data types but they're all date types
     else if ('DataTypes' in scope &&
         scope.DataTypes.length > 0 &&
         scope.DataTypes.every((type) => dateTypes.includes(type))) {
         return 'date';
     }
-    // Check for ColumnTypes in scope
     if ('ColumnTypes' in scope && scope.ColumnTypes.length) {
-        // If a Pivot Total Column Type is specified, we can safely assume that the data type is number
         const pivotTotalTypes = [
             PIVOT_ANY_TOTAL_COLUMN_TYPE,
             PIVOT_GRAND_TOTAL_COLUMN_TYPE,
             PIVOT_COLUMN_TOTAL_COLUMN_TYPE,
             PIVOT_AGGREGATION_TOTAL_COLUMN_TYPE,
         ];
-        // If all column types are pivot total types, we can safely assume that the data type is number
         if (scope.ColumnTypes.length > 0 &&
             scope.ColumnTypes.every((type) => pivotTotalTypes.includes(type))) {
             return 'number';
         }
-        // Check if all columns with this column type have the same data type
         const columns = scope.ColumnTypes.flatMap((columnType) => {
             return api.columnApi.getColumnsByColumnType(columnType);
         });
         if (columns.length) {
-            // Check if all columns have the same type
             const allSameType = columns.every((column) => column.dataType === columns[0].dataType);
-            // Handle specific data types
             if (allSameType) {
                 const dataType = columns[0].dataType;
                 if (dataType === 'number') {
@@ -259,7 +245,6 @@ export const getFormatDisplayTypeForScope = (scope, api) => {
                     return 'text';
                 }
             }
-            // Check if all columns are date types (even if they're different date types)
             const allDateTypes = columns.every((column) => dateTypes.includes(column.dataType));
             if (allDateTypes) {
                 return 'date';
@@ -329,10 +314,6 @@ const renderNumberFormat = (data, onChange, setFormatOption, scopedCustomFormatt
     if (resolved?.Formatter !== 'NumberFormatter') {
         return null;
     }
-    // A preset is now stored as a bare string ('Percentage', 'Dollar', ...)
-    // — selecting a preset just writes the name. The active preset is
-    // therefore a simple string comparison instead of the old fragile
-    // suffix/multiplier heuristic.
     const setPreset = (preset) => {
         onChange({ DisplayFormat: preset });
     };
@@ -431,23 +412,15 @@ export const FormatColumnFormatWizardSection = (props) => {
         props.onChange({ ...data, ...updated });
     };
     const setFormatOption = (key, value) => {
-        // If the user is currently on a preset (e.g. 'Dollar'), materialise
-        // it into a concrete AdaptableFormat *before* applying the edit so
-        // that tweaking any field "forks" the preset into a bespoke format.
         const current = resolveDisplayFormat(data.DisplayFormat);
         if (!current)
             return;
-        // The cast is safe: we copy `current.Formatter` and merge into
-        // `current.Options`, so the discriminant and the Options shape
-        // are always kept in sync — TypeScript just can't correlate the
-        // two ends of the discriminated union across a generic merge.
         const DisplayFormat = {
             Formatter: current.Formatter,
             Options: { ...current.Options, [key]: value },
         };
         update({ DisplayFormat });
     };
-    // A preset string is logically a NumberFormatter for this routing.
     const Type = isAdaptableNumericFormatPreset(data.DisplayFormat)
         ? 'NumberFormatter'
         : data.DisplayFormat && data.DisplayFormat.Formatter;

package/src/View/FormatColumn/Wizard/FormatColumnPreview.js

@@ -16,7 +16,6 @@ const toFormatColumnPreviewStyle = (formatColumn) => {
         textAlign: style.Alignment ? style.Alignment.toLowerCase() : undefined,
     };
 };
-/** Today's date as sample text — no Display Format options applied. */
 export const getFormatColumnPreviewDateSampleText = () => new Date().toLocaleDateString();
 export const getFormatColumnStylePreviewText = (formatColumn, api) => {
     const formatType = getFormatDisplayTypeForScope(formatColumn.Scope, api);

package/src/View/FormatColumn/Wizard/FormatColumnScopeWizardSection.js

@@ -22,10 +22,6 @@ const ROW_KIND_EXCLUDE_KEY = {
     Summary: 'ExcludeSummaryRows',
     Total: 'ExcludeTotalRows',
 };
-/**
- * Missing flags default to "include" — matches the long-standing
- * Format Column row-scope semantics.
- */
 const isRowKindIncluded = (data, kind) => {
     return !data.RowScope?.[ROW_KIND_EXCLUDE_KEY[kind]];
 };
@@ -79,9 +75,6 @@ const FormatColumnRowsScopeSummary = ({ data, }) => {
     return (_jsx(CollapsibleWizardValueSummary, { value: allKinds ? (_jsx(Tag, { children: "All" })) : includedKinds.length === 0 ? (_jsx(Tag, { children: "None" })) : (_jsx(Flex, { flexWrap: "wrap", className: "twa:gap-1", children: includedKinds.map((kind) => (_jsx(Tag, { children: ROW_KIND_LABELS[kind] }, kind))) })) }));
 };
 const FormatColumnRowScopePanel = ({ data, onRowKindChange, }) => (_jsx(Flex, { flexDirection: "column", className: "twa:items-start twa:px-1 twa:py-1", style: { rowGap: 'calc(var(--ab-base-space) * 2)' }, children: FORMAT_COLUMN_ROW_KINDS.map((kind) => (_jsx(CheckBox, { "data-name": `include-${kind.toLowerCase()}-rows-checkbox`, checked: isRowKindIncluded(data, kind), onChange: (checked) => onRowKindChange(kind, checked), children: ROW_KIND_LABELS[kind] }, kind))) }));
-// ---------------------------------------------------------------------------
-// Section
-// ---------------------------------------------------------------------------
 export const FormatColumnScopeWizardSection = (props) => {
     const { data, api } = useOnePageAdaptableWizardContext();
     const adaptable = useAdaptable();

package/src/View/FormatColumn/Wizard/FormatColumnSettingsWizardSection.js

@@ -15,17 +15,6 @@ export const renderFormatColumnSettingsSummary = (data) => {
 export const FormatColumnSettingsWizardSection = (props) => {
     const { data, api } = useOnePageAdaptableWizardContext();
     const nameInputRef = React.useRef(null);
-    // When the wizard is opened for a brand-new Format Column (no Name yet),
-    // park the cursor in the Name input so the user can start typing.
-    //
-    // We have to fight three earlier focus calls:
-    //   1. <Dialog> focuses itself on mount via useAutoFocus.
-    //   2. OnePageWizards focuses its section-nav pill in a useEffect.
-    //   3. The same wizard effect re-fires when its ResizeObserver-driven
-    //      `width` state updates (a tick or two after mount).
-    //
-    // Two animation frames takes us past (3); a small setTimeout fallback
-    // re-asserts focus in case layout thrashes again.
     React.useEffect(() => {
         if (data?.Name) {
             return;
@@ -44,7 +33,6 @@ export const FormatColumnSettingsWizardSection = (props) => {
     }, []);
     const currentTarget = data.Target ? data.Target : 'cell';
     const handleTargetChange = (target) => {
-        // Update the formatColumn object
         props.onChange({
             ...data,
             Target: target,

package/src/View/FormatColumn/Wizard/FormatColumnWizard.js

@@ -24,7 +24,6 @@ import { FormatColumnPreviewSummaryCard } from './FormatColumnPreview';
 const adjustDisplayFormat = (fc, api) => {
     const formatColumn = { ...fc };
     let formatDataType = getFormatDisplayTypeForScope(formatColumn.Scope, api);
-    // Add RowScope and Target defaults for Format Columns that have been created in Initial State
     if (!formatColumn.RowScope) {
         formatColumn.RowScope = {
             ExcludeDataRows: false,
@@ -37,16 +36,11 @@ const adjustDisplayFormat = (fc, api) => {
         formatColumn.Target = 'cell';
     }
     if (formatColumn.Target === 'columnHeader') {
-        // Column Headers are always text
         formatDataType = 'text';
     }
     if (!formatDataType && formatColumn.DisplayFormat) {
         formatColumn.DisplayFormat = undefined;
     }
-    // For shape-detection purposes a numeric preset name is equivalent to a
-    // NumberFormatter. We only need to materialise an object literal when
-    // there is no DisplayFormat at all (or the existing one targets a
-    // different Formatter type to the new scope).
     const resolvedFormatter = isAdaptableNumericFormatPreset(formatColumn.DisplayFormat)
         ? 'NumberFormatter'
         : formatColumn.DisplayFormat?.Formatter;
@@ -96,10 +90,6 @@ export function FormatColumnWizard(props) {
     }, []);
     const dispatch = useDispatch();
     const handleFinish = () => {
-        // If the user opened the wizard, picked a Formatter type, but never
-        // touched any options (and didn't pick a preset), drop the empty
-        // DisplayFormat object so we don't persist an empty shell. Preset
-        // strings are always meaningful and should never be removed here.
         const df = formatColumn?.DisplayFormat;
         if (df && !isAdaptableNumericFormatPreset(df) && df.Options && isObjectEmpty(df.Options)) {
             delete formatColumn.DisplayFormat;

package/src/View/GridFilter/GridFilterPopup.js

@@ -3,7 +3,6 @@ import { useGridFilterExpressionEditor } from './useGridFilterExpressionEditor';
 import { GridFilterPopupUI } from './GridFilterPopupUI';
 export const GridFilterPopup = () => {
     const hookData = useGridFilterExpressionEditor();
-    // Create callback handlers
     const handleExpressionChange = (value) => {
         hookData.setExpression(value);
     };

package/src/View/GridFilter/useGridFilterExpressionEditor.js

@@ -45,7 +45,6 @@ export const useGridFilterExpressionEditor = () => {
             columnId: col.columnId,
             label: col.friendlyName,
             onClick: () => {
-                // Bogdan - im sure there is a better way to do this check with fancy operators i dont know
                 const newExp = StringExtensions.IsNotNullOrEmpty(expression) ? expression : '';
                 setExpression(newExp + `[${col.columnId}]`);
             },
@@ -58,7 +57,6 @@ export const useGridFilterExpressionEditor = () => {
     const runQuery = (newExpression = expression) => {
         if (StringExtensions.IsNullOrEmpty(newExpression) &&
             StringExtensions.IsNotNullOrEmpty(gridFilter?.Expression)) {
-            // user pressed enter key with an empty input => clear existing expression
             adaptable.api.filterApi.gridFilterApi.setGridFilterExpression('');
             return;
         }

package/src/View/GridInfo/GridInfoPopup/GridInfoPopup.js

@@ -70,9 +70,6 @@ export const GridInfoPopup = (props) => {
         Key: { field: 'Key', header: 'Property', defaultFlex: 1 },
         Value: { field: 'Value', header: 'Value', defaultFlex: 3 },
     };
-    // Variant used by the Grid Options tab — the Value column allows newlines
-    // in the rendered string (preserved via `white-space: pre-line`), so a
-    // nested *Options object can show one `"key": value` per line.
     const optionsColumnsMap = {
         Key: { field: 'Key', header: 'Property', defaultFlex: 1 },
         Value: {
@@ -90,16 +87,10 @@ export const GridInfoPopup = (props) => {
                 }, children: String(value ?? '') })),
         },
     };
-    // ---- Grid Options tab: build one InfiniteTableGrid section per non-empty
-    // ---- options group on the AdaptableOptions object.
     const titleCase = (camel) => camel
         .replace(/([a-z])([A-Z])/g, '$1 $2')
         .replace(/^./, (c) => c.toUpperCase())
         .replace(/\s./g, (c) => c.toUpperCase());
-    // Renders a single primitive / function / array / nested-object value
-    // (used for the inner key:value pairs when expanding a nested options
-    // object). Short objects/arrays stay on one line; longer ones get
-    // pretty-printed across multiple lines so they don't blow past the cell.
     const ONE_LINE_MAX = 60;
     const fnReplacer = (_k, val) => typeof val === 'function' ? '[Function]' : val;
     const formatValuePart = (v) => {
@@ -132,14 +123,9 @@ export const GridInfoPopup = (props) => {
         if (v === null)
             return '(null)';
         if (Array.isArray(v)) {
-            // Top-level array values (e.g. `colorPalette`, `systemFilterPredicates`).
-            // Short arrays stay on one line; longer ones get pretty-printed across
-            // multiple lines so they don't blow past the cell.
             return formatValuePart(v);
         }
         if (typeof v === 'object') {
-            // Nested options-style object → render as one `key: value` per line.
-            // `undefined` entries are filtered out so empty defaults don't show up.
             const entries = Object.entries(v).filter(([, val]) => val !== undefined);
             if (entries.length === 0)
                 return '{}';
@@ -152,11 +138,6 @@ export const GridInfoPopup = (props) => {
         const defaults = getDefaultAdaptableOptions();
         const baseRows = [];
         const groupSections = [];
-        // A function value should only be displayed when it has been explicitly
-        // provided by the user. AdapTable ships a handful of identity-function
-        // defaults (e.g. `stateOptions.applyState`) that are not interesting to
-        // surface here — we suppress those by reference-comparing against the
-        // values returned from `getDefaultAdaptableOptions()`.
         const isUserProvidedFunction = (groupKey, propKey, fn) => {
             if (typeof fn !== 'function')
                 return false;
@@ -175,11 +156,8 @@ export const GridInfoPopup = (props) => {
                 const rows = Object.keys(value)
                     .filter((sk) => {
                     const sv = value[sk];
-                    // skip empties (undefined / null)
                     if (sv == null)
                         return false;
-                    // suppress AdapTable's default callback implementations — keep
-                    // only user-provided overrides
                     if (typeof sv === 'function' && !isUserProvidedFunction(key, sk, sv)) {
                         return false;
                     }
@@ -191,8 +169,6 @@ export const GridInfoPopup = (props) => {
                 groupSections.push({ title: titleCase(key), rows });
             }
             else {
-                // top-level function defaults aren't a concern (none exist on the
-                // root of `AdaptableOptions`), but apply the same rule for safety
                 if (typeof value === 'function' && defaults[key] === value)
                     return;
                 baseRows.push({ Key: key, Value: displayValue(value) });
@@ -204,10 +180,7 @@ export const GridInfoPopup = (props) => {
         sections.push(...groupSections);
         return sections;
     })();
-    // Variable row height for Grid Options sections — counts the number of
-    // lines in the formatted value string (including visual wraps for very
-    // long unbroken lines) and grows the row to fit.
-    const VALUE_CHARS_PER_LINE = 70; // rough estimate at the Value column width
+    const VALUE_CHARS_PER_LINE = 70;
     const visualLineCount = (text) => {
         if (!text)
             return 1;
@@ -223,11 +196,8 @@ export const GridInfoPopup = (props) => {
     };
     const sectionHeight = (rows) => {
         const total = rows.reduce((acc, r) => acc + calculateRowHeight(r), 0);
-        // 40px header, capped so a single huge group can't dominate the popup
         return Math.min(total + 40, 400);
     };
-    // Which tabs the user has opted in to. Falls back to all three (matching
-    // the default in `DefaultAdaptableOptions`) when the option isn't set.
     const enabledTabs = api.optionsApi.getSettingsPanelOptions()?.gridInfoTabs ??
         ['Grid Options', 'Grid Summary', 'Grid State'];
     const showSummary = enabledTabs.includes('Grid Summary');

package/src/View/Layout/LayoutViewPanel.js

@@ -16,7 +16,6 @@ import { NewDropdownButton } from '../../components/DropdownButton';
 import { Icon } from '../../components/icons';
 import { Flex } from '../../components/Flex';
 import { SingleSelect } from '../../components/NewSelect';
-// import { SingleCombobox } from '../../components/Combobox';
 export const COMPONENT_LAYOUT_POPUP_NAME = 'LayoutEditorStandalonePopup';
 const LayoutViewPanelComponent = (props) => {
     const { Layouts, CurrentLayoutName, accessLevel, viewType, api, onSelectLayout, showMissingLayoutsError, } = props;
@@ -27,7 +26,6 @@ const LayoutViewPanelComponent = (props) => {
         }
     }, [isErrorLayout]);
     const layoutEntity = Layouts.find((x) => x.Name === CurrentLayoutName || x.Uuid === CurrentLayoutName);
-    // the global access level (Layout Entitlement)
     const cloneAccessLevel = accessLevel;
     const newAccessLevel = accessLevel;
     const entityAccessLevel = AdaptableHelper.getAccessLevelForObject(layoutEntity, accessLevel);

package/src/View/Layout/PivotDetailsPopoup.js

@@ -39,8 +39,6 @@ export const PivotDetailsPopoup = (props) => {
             [primaryKey]: { field: primaryKey, header: getFriendlyName(primaryKey) },
         };
         if (hasPivotValue) {
-            // pivot_license_MIT License_stargazers_count
-            // splitting by _ does not work
             columns = {
                 ...columns,
                 [aggColumn]: { field: aggColumn, header: getFriendlyName(aggColumn) },

package/src/View/Layout/TransposedPopup.js

@@ -11,9 +11,7 @@ import { ExportModuleId } from '../../Utilities/Constants/ModuleConstants';
 import StringExtensions from '../../Utilities/Extensions/StringExtensions';
 const ADAPTABLE_CONTAINER_ID = 'transposed-adaptable-container';
 const AG_GRID_CONTAINER_ID = 'transposed-adaptable-ag-grid-container';
-/** Field used for the first column in transposed grid (hidden, holds column id). */
 const TRANSPOSED_FIRST_COLUMN_FIELD = '_transposed_column_value';
-/** Header used for the first visible column in transposed grid (shows friendly name). */
 const TRANSPOSED_FIRST_COLUMN_HEADER = '_transposed_column_header';
 function buildTransposedAdaptableOptions({ hostOptions, transposedRowsAndColumns, currentTheme, }) {
     return {
@@ -129,11 +127,6 @@ export const TransposedPopup = (props) => {
             .map((c) => adaptable.api.columnApi.getColumnWithColumnId(c))
             .filter((col) => col != null);
     }, [transposeConfig.columnsToTranspose, adaptable.api]);
-    /**
-     * Build transposed structure: original rows become columns (colId = primaryKey value),
-     * original columns become rows. Each transposed row has the column id/header plus
-     * one cell per original row keyed by primaryKey value.
-     */
     const transposedRowsAndColumns = React.useMemo(() => {
         const transposedColumns = [];
         const transposedRows = [];

package/src/View/Layout/Wizard/LayoutWizard.js

@@ -197,7 +197,6 @@ export const LayoutWizard = (props) => {
                 isValid: (data) => isAggregationsSectionValid(data),
                 render: () => (_jsx(Box, { className: "twa:p-2 twa:h-full", children: _jsx(AggregationsSection, { layoutWizardMode: layoutWizardMode, onChange: (layout) => {
                             let newLayout = cloneObject(layout);
-                            // if we do not have an weighted avg col, we need to clear the row summary if one exists
                             if (newLayout.RowSummaries) {
                                 const aggColsMap = (newLayout.TableAggregationColumns || []).reduce((acc, { ColumnId, AggFunc }) => {
                                     acc[ColumnId] = AggFunc;
@@ -207,14 +206,10 @@ export const LayoutWizard = (props) => {
                                     return {
                                         ...rowSummary,
                                         ColumnsMap: Object.entries(rowSummary.ColumnsMap).reduce((acc, [columnId, aggFunc]) => {
-                                            if (
-                                            // see if it is weighted avg
-                                            aggFunc === WEIGHTED_AVERAGE_AGGREGATED_FUNCTION &&
-                                                // see if we have a weight in the agg columns
+                                            if (aggFunc === WEIGHTED_AVERAGE_AGGREGATED_FUNCTION &&
                                                 aggColsMap[columnId] &&
                                                 (typeof aggColsMap[columnId] !== 'object' ||
                                                     aggColsMap[columnId].weightColumnId)) {
-                                                // need to remove the row summary
                                                 return acc;
                                             }
                                             acc[columnId] = aggFunc;

package/src/View/Layout/Wizard/sections/AggregationsSection.js

@@ -163,8 +163,6 @@ export const AggregationsSection = (props) => {
     const numberColumns = adaptable.api.columnApi.getNumericColumns();
     const allAggregatableColumnIds = allAggregableColumns.map((col) => col.columnId);
     (layout.TableAggregationColumns || []).forEach((agg) => {
-        // we need to also display the columns currently aggregated,
-        // even if they are not aggregatable
         if (!allAggregatableColumnIds.includes(agg.ColumnId)) {
             allAggregatableColumnIds.push(agg.ColumnId);
         }

package/src/View/Layout/Wizard/sections/ColumnsSection.js

@@ -32,7 +32,6 @@ export const ColumnsSectionSummary = (props) => {
     const { data: contextLayout } = useOnePageAdaptableWizardContext();
     const layout = props.layout ?? contextLayout;
     const groups = getTableColumnsSummaryGroups(layout, adaptable.api, TABLE_COLUMNS_WIZARD_SUBSECTION_LABELS);
-    // Wizard summary lists every column — no truncation (settings panel uses truncateColumnSummaryValues).
     return (_jsx(Flex, { flexDirection: "column", className: "twa:gap-3", children: groups.map(({ label, values, emptyLabel }) => (_jsxs(Box, { children: [_jsx(Box, { className: "twa:text-xs twa:font-medium twa:mb-1", children: label }), values.length ? renderSummaryStringTags(values) : _jsx(Tag, { children: emptyLabel })] }, label))) }));
 };
 const isSelectionColumn = (columnId) => columnId === AG_GRID_SELECTION_COLUMN;
@@ -66,24 +65,15 @@ export const ColumnsSection = (props) => {
     const { data: layout } = useOnePageAdaptableWizardContext();
     const [searchInputValue, setSearchInputValue] = React.useState('');
     const [selectedColumnId, setSelectedColumnId] = React.useState(null);
-    // the selection column is a special column managed by AG Grid - it is not
-    // returned by `getUIAvailableColumns`, but we still want to expose it in the
-    // column list (similar to auto row group columns) so users can reorder/pin it
     const hasSelectionColumn = adaptable.api.columnApi
         .getColumns()
         .some((col) => col.isGeneratedSelectionColumn);
     const allColumns = adaptable.api.columnApi
         .getUIAvailableColumns()
         .filter((col) => {
-        // since the "Row Groups" section of the editor can determine a change
-        // which is not reflected into AG Grid until we hit finish
-        // so we only rely on non-generated columns, which are the same
         return !col.isGeneratedRowGroupColumn;
     })
-        // if the current Layout is a PivotLayout, then we also filter out current Pivot Result Columns
         .filter((col) => !col.isGeneratedPivotResultColumn)
-        // selection columns are not returned by `getUIAvailableColumns` but guard
-        // against any future change
         .filter((col) => !col.isGeneratedSelectionColumn);
     const onChange = (data) => {
         if (hasSelectionColumn &&
@@ -93,9 +83,6 @@ export const ColumnsSection = (props) => {
         }
         props.onChange(data);
     };
-    // however, changes in RowGroupedColumns done in the previous step
-    // are reflected into the layout, so we use the latest layout.RowGroupedColumns
-    // to create new columns
     if (layout.RowGroupedColumns && layout.RowGroupedColumns.length > 0) {
         if (layout.RowGroupDisplayType === 'single') {
             allColumns.unshift(generateAutoRowGroupSingleColumn());
@@ -121,9 +108,6 @@ export const ColumnsSection = (props) => {
         return acc;
     }, {});
     let TableColumns = layout.TableColumns;
-    // when the selection column exists but is not explicitly listed in
-    // TableColumns, `sortArrayWithOrder` would push it to the end - instead we
-    // want it at the start (consistent with AG Grid's default position for it)
     if (hasSelectionColumn && !TableColumns.includes(AG_GRID_SELECTION_COLUMN)) {
         TableColumns = [AG_GRID_SELECTION_COLUMN, ...TableColumns];
     }
@@ -136,16 +120,7 @@ export const ColumnsSection = (props) => {
     const currentOrderIds = currentOrder.map((col) => col.columnId);
     const handleColumnsChange = (newColumns) => {
         if (searchInputValue) {
-            // there's a search in-place
-            // so the current `newColumns` does not actually contain all the columns
-            // so we need to recompute it
-            const newOrder = sortColumnIdsByOrder(
-            // given those are all the columns
-            ColumnOrderAllColumns.map((col) => col.columnId), 
-            // we want to sort them to have this new order
-            // - this sorting will leave all ids not found in this new order untouched
-            newColumns.map((col) => col.columnId));
-            // so basically we now have the correct order, with all columns
+            const newOrder = sortColumnIdsByOrder(ColumnOrderAllColumns.map((col) => col.columnId), newColumns.map((col) => col.columnId));
             newColumns = newOrder.map((colId) => colIdToCol[colId]);
         }
         const oldColumns = ColumnOrderAllColumns;
@@ -192,15 +167,9 @@ export const ColumnsSection = (props) => {
                 if (isCurrent) {
                     return true;
                 }
-                // include all columns past the current column order
-                // that reach until the index of the currently checked column
                 const shouldInclude = idsToIndexes.get(colId) <= biggestIndex;
                 if (shouldInclude) {
                     if (!columnIdSet.has(colId) && visible) {
-                        // this is a column that's between the current column order limit
-                        // and the currently checked column
-                        // so we need to include it in order
-                        // but make it invisible
                         ColumnVisibility[colId] = false;
                     }
                 }

package/src/View/Layout/Wizard/sections/RowSummarySection.js

@@ -47,7 +47,6 @@ const getAvailableExpressionsForColumnType = (columnType, availableScalarExpress
     const expressions = Object.keys(availableScalarExpressions)
         .filter((availableExpression) => {
         if (!aggregatedExpressionFunctions.includes(availableExpression)) {
-            // is custom
             return true;
         }
         return Boolean(summarySupportedExpressions.includes(availableExpression));
@@ -55,12 +54,10 @@ const getAvailableExpressionsForColumnType = (columnType, availableScalarExpress
         .map((expression) => {
         const expressionDef = availableScalarExpressions[expression];
         let firstArg = null;
-        // filter out expressions without inputs defined
         if (!expressionDef?.inputs) {
             return null;
         }
         if (Array.isArray(expressionDef?.inputs?.[0])) {
-            // @ts-ignore
             firstArg = expressionDef.inputs.find((input) => input.includes(columnInputType))?.[0];
         }
         else {

package/src/View/Layout/Wizard/sections/columnLayoutHelpers.js

@@ -14,7 +14,6 @@ export function getLayoutPinnedColumnIds(layout, side) {
         .filter(([, pin]) => pin === side)
         .map(([columnId]) => columnId);
 }
-/** Whether this column has sizing stored in the Layout that differs from grid defaults. */
 export function hasExplicitLayoutSizing(columnId, sizing, resolveDefaultWidth) {
     if (!sizing) {
         return false;
@@ -57,7 +56,6 @@ export function formatColumnSizingBrief(columnId, sizing, resolveDefaultWidth) {
     }
     return parts.join(', ');
 }
-/** Width/flex values to show in the sizing editor (default grid widths are omitted). */
 export function getExplicitSizingDisplayValues(columnId, sizing, resolveDefaultWidth) {
     if (!sizing) {
         return {

package/src/View/Layout/Wizard/sections/layoutWizardColumns.js

@@ -2,13 +2,9 @@ import { generateAutoRowGroupColumnForColumn, generateAutoRowGroupSingleColumn,
 import ArrayExtensions from '../../../../Utilities/Extensions/ArrayExtensions';
 import { AG_GRID_GROUPED_COLUMN, AG_GRID_SELECTION_COLUMN } from '../../../../Utilities/Constants/GeneralConstants';
 import { WIZARD_COLUMN_LIST_HEADER_CLASS, WIZARD_COLUMN_LIST_OPTION_CLASS, WIZARD_COLUMN_LIST_SEARCH_CLASS, } from '../../../Components/wizardColumnListStyles';
-/** ValueSelector row sizing — matches Visibility and Order list rows. */
 export const LAYOUT_WIZARD_COLUMN_LIST_OPTION_CLASS = WIZARD_COLUMN_LIST_OPTION_CLASS;
-/** Compact list header — spacing between Select All, Show Selected Only, and search. */
 export const LAYOUT_WIZARD_COLUMN_LIST_HEADER_CLASS = WIZARD_COLUMN_LIST_HEADER_CLASS;
-/** Narrow search field (matches Visibility list column width, not full card width). */
 export const LAYOUT_WIZARD_COLUMN_LIST_SEARCH_CLASS = WIZARD_COLUMN_LIST_SEARCH_CLASS;
-/** Columns shown in Layout wizard column steps, in layout table order. */
 export function getLayoutWizardOrderedColumns(adaptable, layout) {
     const hasSelectionColumn = adaptable.api.columnApi
         .getColumns()

package/src/View/Note/NotePopup.js

@@ -13,7 +13,6 @@ const tableDOMProps = {
     style: {
         height: '100%',
         minWidth: '10rem',
-        // minHeight: 600,
     },
 };
 const columnTypes = {

package/src/View/QuickSearch/QuickSearchPopup.js

@@ -32,7 +32,6 @@ function mapStateToProps(state, ownProps) {
         QuickSearchText: quickSearch.QuickSearchText,
         QuickSearchTextMatchStyle: resolveQuickSearchTextMatchStyle(quickSearch),
         QuickSearchCurrentTextMatchStyle: resolveQuickSearchCurrentTextMatchStyle(quickSearch),
-        // Cell match: show only what is in state — defaults are not applied until explicitly set
         QuickSearchCellMatchStyle: isQuickSearchStyleUnset(quickSearch.CellMatchStyle)
             ? {}
             : quickSearch.CellMatchStyle,

package/src/View/StatusBar/AdaptableStatusBar.js

@@ -23,14 +23,10 @@ export const AdaptableStatusBar = (props) => {
             if (!module?.isModuleVisible()) {
                 return _jsx(React.Fragment, {}, subPanel);
             }
-            /**
-             * Both view properties and status panel are optional.
-             */
             const statusBarPanelProps = module.getViewProperties?.()?.getStatusBarPanelProps?.() ?? {};
             const moduleInfo = module.moduleInfo;
             const onAction = statusBarPanelProps.onAction ??
                 (() => {
-                    // #by-default-open-settings-panel-for-module
                     const menuItem = allMenuItems.find((menuItem) => menuItem.category === subPanel);
                     if (menuItem?.reduxAction) {
                         dispatch(menuItem.reduxAction);

package/src/View/StatusBar/StatusBarPanel.js

@@ -5,12 +5,6 @@ import ArrayExtensions from '../../Utilities/Extensions/ArrayExtensions';
 import { AdaptablePopover } from '../AdaptablePopover';
 import { Flex, Box } from '../../components/Flex';
 import { cn } from '../../lib/utils';
-/**
- * Statusbar sub panel are similar to vs code:
- * - text
- * - action
- * - icon
- */
 export const StatusBarPanel = (props) => {
     const { icon, content, popover, popoverMinWidth, view, onAction, extraActions, triggerActionOnWrapperClick = true, tooltip, ...flexProps } = props;
     const handleAction = () => onAction();
@@ -24,12 +18,10 @@ export const StatusBarPanel = (props) => {
     }, []);
     const shouldTriggerActionOnWrapperClick = triggerActionOnWrapperClick && !popover;
     const handleWrapperClick = React.useCallback(() => {
-        // handle action only if there is no popover
         if (shouldTriggerActionOnWrapperClick) {
             handleAction();
         }
     }, [popover, triggerActionOnWrapperClick]);
-    // popover
     const popoverContent = typeof popover === 'function'
         ? React.createElement(popover)
         : popover;

package/src/View/StatusBar/StatusBarPopup.js

@@ -47,8 +47,6 @@ export const StatusBarPopup = (props) => {
                 moduleShortcutIds.push(item.Id);
             }
         });
-        // Roughly balance the two boxes by item count (~10 panels vs ~17 shortcuts)
-        // so the wider list gets the extra room and they feel more even.
         return [
             {
                 listId: 'UNUSED-PANELS',