@adaptabletools/adaptable-cjs 23.0.4-canary.0 → 23.0.5-canary.0

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

@@ -116,8 +116,6 @@ const getFormatColumnDisplayFormatSummaryItems = (data) => {
     if ((0, AdaptableFormatPresets_1.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' &&
         (0, ObjectExtensions_1.isObjectEmpty)(data.DisplayFormat.Options)) {
@@ -188,13 +186,10 @@ 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 = (0, ArrayExtensions_1.uniq)(columns.map((c) => c.dataType));
-        // If there's only one data type
         if (columnDataTypes.length === 1) {
             const dataType = columnDataTypes[0];
             if (dataType === 'number') {
@@ -207,14 +202,12 @@ 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') {
@@ -227,34 +220,27 @@ 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 = [
             AdaptableColumn_1.PIVOT_ANY_TOTAL_COLUMN_TYPE,
             AdaptableColumn_1.PIVOT_GRAND_TOTAL_COLUMN_TYPE,
             AdaptableColumn_1.PIVOT_COLUMN_TOTAL_COLUMN_TYPE,
             AdaptableColumn_1.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') {
@@ -267,7 +253,6 @@ 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';
@@ -338,10 +323,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 });
     };
@@ -440,23 +421,15 @@ 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 = (0, AdaptableFormatPresets_1.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 = (0, AdaptableFormatPresets_1.isAdaptableNumericFormatPreset)(data.DisplayFormat)
         ? 'NumberFormatter'
         : data.DisplayFormat && data.DisplayFormat.Formatter;

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

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

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

@@ -25,10 +25,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]];
 };
@@ -83,9 +79,6 @@ const FormatColumnRowsScopeSummary = ({ data, }) => {
     return ((0, jsx_runtime_1.jsx)(CollapsibleWizardCard_1.CollapsibleWizardValueSummary, { value: allKinds ? ((0, jsx_runtime_1.jsx)(Tag_1.Tag, { children: "All" })) : includedKinds.length === 0 ? ((0, jsx_runtime_1.jsx)(Tag_1.Tag, { children: "None" })) : ((0, jsx_runtime_1.jsx)(Flex_1.Flex, { flexWrap: "wrap", className: "twa:gap-1", children: includedKinds.map((kind) => ((0, jsx_runtime_1.jsx)(Tag_1.Tag, { children: ROW_KIND_LABELS[kind] }, kind))) })) }));
 };
 const FormatColumnRowScopePanel = ({ data, onRowKindChange, }) => ((0, jsx_runtime_1.jsx)(Flex_1.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) => ((0, jsx_runtime_1.jsx)(CheckBox_1.CheckBox, { "data-name": `include-${kind.toLowerCase()}-rows-checkbox`, checked: isRowKindIncluded(data, kind), onChange: (checked) => onRowKindChange(kind, checked), children: ROW_KIND_LABELS[kind] }, kind))) }));
-// ---------------------------------------------------------------------------
-// Section
-// ---------------------------------------------------------------------------
 const FormatColumnScopeWizardSection = (props) => {
     const { data, api } = (0, OnePageAdaptableWizard_1.useOnePageAdaptableWizardContext)();
     const adaptable = (0, AdaptableContext_1.useAdaptable)();

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

@@ -20,17 +20,6 @@ exports.renderFormatColumnSettingsSummary = renderFormatColumnSettingsSummary;
 const FormatColumnSettingsWizardSection = (props) => {
     const { data, api } = (0, OnePageAdaptableWizard_1.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;
@@ -49,7 +38,6 @@ 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

@@ -28,7 +28,6 @@ const FormatColumnPreview_1 = require("./FormatColumnPreview");
 const adjustDisplayFormat = (fc, api) => {
     const formatColumn = { ...fc };
     let formatDataType = (0, FormatColumnFormatWizardSection_1.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,
@@ -41,16 +40,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 = (0, AdaptableFormatPresets_1.isAdaptableNumericFormatPreset)(formatColumn.DisplayFormat)
         ? 'NumberFormatter'
         : formatColumn.DisplayFormat?.Formatter;
@@ -100,10 +94,6 @@ function FormatColumnWizard(props) {
     }, []);
     const dispatch = (0, react_redux_2.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 && !(0, AdaptableFormatPresets_1.isAdaptableNumericFormatPreset)(df) && df.Options && (0, ObjectExtensions_1.isObjectEmpty)(df.Options)) {
             delete formatColumn.DisplayFormat;

package/src/View/GridFilter/GridFilterPopup.js

@@ -6,7 +6,6 @@ const useGridFilterExpressionEditor_1 = require("./useGridFilterExpressionEditor
 const GridFilterPopupUI_1 = require("./GridFilterPopupUI");
 const GridFilterPopup = () => {
     const hookData = (0, useGridFilterExpressionEditor_1.useGridFilterExpressionEditor)();
-    // Create callback handlers
     const handleExpressionChange = (value) => {
         hookData.setExpression(value);
     };

package/src/View/GridFilter/useGridFilterExpressionEditor.js

@@ -49,7 +49,6 @@ 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_1.default.IsNotNullOrEmpty(expression) ? expression : '';
                 setExpression(newExp + `[${col.columnId}]`);
             },
@@ -62,7 +61,6 @@ const useGridFilterExpressionEditor = () => {
     const runQuery = (newExpression = expression) => {
         if (StringExtensions_1.default.IsNullOrEmpty(newExpression) &&
             StringExtensions_1.default.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

@@ -74,9 +74,6 @@ 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: {
@@ -94,16 +91,10 @@ 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) => {
@@ -136,14 +127,9 @@ 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 '{}';
@@ -156,11 +142,6 @@ const GridInfoPopup = (props) => {
         const defaults = (0, DefaultAdaptableOptions_1.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;
@@ -179,11 +160,8 @@ 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;
                     }
@@ -195,8 +173,6 @@ 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) });
@@ -208,10 +184,7 @@ 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;
@@ -227,11 +200,8 @@ 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

@@ -20,7 +20,6 @@ const DropdownButton_1 = require("../../components/DropdownButton");
 const icons_1 = require("../../components/icons");
 const Flex_1 = require("../../components/Flex");
 const NewSelect_1 = require("../../components/NewSelect");
-// import { SingleCombobox } from '../../components/Combobox';
 exports.COMPONENT_LAYOUT_POPUP_NAME = 'LayoutEditorStandalonePopup';
 const LayoutViewPanelComponent = (props) => {
     const { Layouts, CurrentLayoutName, accessLevel, viewType, api, onSelectLayout, showMissingLayoutsError, } = props;
@@ -31,7 +30,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_1.default.getAccessLevelForObject(layoutEntity, accessLevel);

package/src/View/Layout/PivotDetailsPopoup.js

@@ -43,8 +43,6 @@ 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

@@ -15,9 +15,7 @@ const ModuleConstants_1 = require("../../Utilities/Constants/ModuleConstants");
 const StringExtensions_1 = tslib_1.__importDefault(require("../../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 {
@@ -133,11 +131,6 @@ 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

@@ -201,7 +201,6 @@ const LayoutWizard = (props) => {
                 isValid: (data) => (0, AggregationsSection_1.isAggregationsSectionValid)(data),
                 render: () => ((0, jsx_runtime_1.jsx)(Flex_1.Box, { className: "twa:p-2 twa:h-full", children: (0, jsx_runtime_1.jsx)(AggregationsSection_1.AggregationsSection, { layoutWizardMode: layoutWizardMode, onChange: (layout) => {
                             let newLayout = (0, Helper_1.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;
@@ -211,14 +210,10 @@ const LayoutWizard = (props) => {
                                     return {
                                         ...rowSummary,
                                         ColumnsMap: Object.entries(rowSummary.ColumnsMap).reduce((acc, [columnId, aggFunc]) => {
-                                            if (
-                                            // see if it is weighted avg
-                                            aggFunc === RowSummary_1.WEIGHTED_AVERAGE_AGGREGATED_FUNCTION &&
-                                                // see if we have a weight in the agg columns
+                                            if (aggFunc === RowSummary_1.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

@@ -173,8 +173,6 @@ 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

@@ -36,7 +36,6 @@ const ColumnsSectionSummary = (props) => {
     const { data: contextLayout } = (0, OnePageAdaptableWizard_1.useOnePageAdaptableWizardContext)();
     const layout = props.layout ?? contextLayout;
     const groups = (0, columnsSummaryHelpers_1.getTableColumnsSummaryGroups)(layout, adaptable.api, columnsSummaryHelpers_1.TABLE_COLUMNS_WIZARD_SUBSECTION_LABELS);
-    // Wizard summary lists every column — no truncation (settings panel uses truncateColumnSummaryValues).
     return ((0, jsx_runtime_1.jsx)(Flex_1.Flex, { flexDirection: "column", className: "twa:gap-3", children: groups.map(({ label, values, emptyLabel }) => ((0, jsx_runtime_1.jsxs)(Flex_1.Box, { children: [(0, jsx_runtime_1.jsx)(Flex_1.Box, { className: "twa:text-xs twa:font-medium twa:mb-1", children: label }), values.length ? (0, SummaryColorTag_1.renderSummaryStringTags)(values) : (0, jsx_runtime_1.jsx)(Tag_1.Tag, { children: emptyLabel })] }, label))) }));
 };
 exports.ColumnsSectionSummary = ColumnsSectionSummary;
@@ -71,24 +70,15 @@ const ColumnsSection = (props) => {
     const { data: layout } = (0, OnePageAdaptableWizard_1.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 &&
@@ -98,9 +88,6 @@ 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((0, ColumnApiImpl_1.generateAutoRowGroupSingleColumn)());
@@ -126,9 +113,6 @@ 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(GeneralConstants_1.AG_GRID_SELECTION_COLUMN)) {
         TableColumns = [GeneralConstants_1.AG_GRID_SELECTION_COLUMN, ...TableColumns];
     }
@@ -141,16 +125,7 @@ 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 = (0, sortColumnIdsByOrder_1.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 = (0, sortColumnIdsByOrder_1.sortColumnIdsByOrder)(ColumnOrderAllColumns.map((col) => col.columnId), newColumns.map((col) => col.columnId));
             newColumns = newOrder.map((colId) => colIdToCol[colId]);
         }
         const oldColumns = ColumnOrderAllColumns;
@@ -197,15 +172,9 @@ 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

@@ -52,7 +52,6 @@ const getAvailableExpressionsForColumnType = (columnType, availableScalarExpress
     const expressions = Object.keys(availableScalarExpressions)
         .filter((availableExpression) => {
         if (!aggregatedScalarExpressionFunctions_1.aggregatedExpressionFunctions.includes(availableExpression)) {
-            // is custom
             return true;
         }
         return Boolean(RowSummary_1.summarySupportedExpressions.includes(availableExpression));
@@ -60,12 +59,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

@@ -29,7 +29,6 @@ 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. */
 function hasExplicitLayoutSizing(columnId, sizing, resolveDefaultWidth) {
     if (!sizing) {
         return false;
@@ -72,7 +71,6 @@ function formatColumnSizingBrief(columnId, sizing, resolveDefaultWidth) {
     }
     return parts.join(', ');
 }
-/** Width/flex values to show in the sizing editor (default grid widths are omitted). */
 function getExplicitSizingDisplayValues(columnId, sizing, resolveDefaultWidth) {
     if (!sizing) {
         return {

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

@@ -7,13 +7,9 @@ const ColumnApiImpl_1 = require("../../../../Api/Implementation/ColumnApiImpl");
 const ArrayExtensions_1 = tslib_1.__importDefault(require("../../../../Utilities/Extensions/ArrayExtensions"));
 const GeneralConstants_1 = require("../../../../Utilities/Constants/GeneralConstants");
 const wizardColumnListStyles_1 = require("../../../Components/wizardColumnListStyles");
-/** ValueSelector row sizing — matches Visibility and Order list rows. */
 exports.LAYOUT_WIZARD_COLUMN_LIST_OPTION_CLASS = wizardColumnListStyles_1.WIZARD_COLUMN_LIST_OPTION_CLASS;
-/** Compact list header — spacing between Select All, Show Selected Only, and search. */
 exports.LAYOUT_WIZARD_COLUMN_LIST_HEADER_CLASS = wizardColumnListStyles_1.WIZARD_COLUMN_LIST_HEADER_CLASS;
-/** Narrow search field (matches Visibility list column width, not full card width). */
 exports.LAYOUT_WIZARD_COLUMN_LIST_SEARCH_CLASS = wizardColumnListStyles_1.WIZARD_COLUMN_LIST_SEARCH_CLASS;
-/** Columns shown in Layout wizard column steps, in layout table order. */
 function getLayoutWizardOrderedColumns(adaptable, layout) {
     const hasSelectionColumn = adaptable.api.columnApi
         .getColumns()

package/src/View/Note/NotePopup.js

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

package/src/View/QuickSearch/QuickSearchPopup.js

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

package/src/View/StatusBar/AdaptableStatusBar.js

@@ -27,14 +27,10 @@ const AdaptableStatusBar = (props) => {
             if (!module?.isModuleVisible()) {
                 return (0, jsx_runtime_1.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

@@ -9,12 +9,6 @@ const ArrayExtensions_1 = tslib_1.__importDefault(require("../../Utilities/Exten
 const AdaptablePopover_1 = require("../AdaptablePopover");
 const Flex_1 = require("../../components/Flex");
 const utils_1 = require("../../lib/utils");
-/**
- * Statusbar sub panel are similar to vs code:
- * - text
- * - action
- * - icon
- */
 const StatusBarPanel = (props) => {
     const { icon, content, popover, popoverMinWidth, view, onAction, extraActions, triggerActionOnWrapperClick = true, tooltip, ...flexProps } = props;
     const handleAction = () => onAction();
@@ -28,12 +22,10 @@ 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

@@ -51,8 +51,6 @@ 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',