@adaptabletools/adaptable 23.0.4 → 23.0.5-canary.1

package/src/View/Components/ToolPanel/AdaptableToolPanel.js

@@ -25,12 +25,10 @@ const AdaptableToolPanelComponent = (props) => {
     const toolPanelOptions = adaptableOptions.toolPanelOptions;
     const settingsPanelOptions = adaptableOptions.settingsPanelOptions;
     if (props.api.entitlementApi.isModuleHiddenEntitlement('ToolPanel')) {
-        // do NOT show any toolPanel content if the required entitlements are missing
         return _jsx(HelpBlock, { className: "twa:my-2 twa:p-2 twa:text-3", children: "Not enough rights" });
     }
     const availableModuleItems = props.MainMenuItems.filter((menuItem) => menuItem.isVisible);
     const availableModules = availableModuleItems.map((menuItem) => menuItem.category);
-    // 'Dashboard' is a special case because it's not available in the dashboard menu items, s we have to add it manually
     if (!props.api.entitlementApi.isModuleHiddenEntitlement('Dashboard')) {
         availableModules.push('Dashboard');
     }
@@ -58,15 +56,12 @@ const AdaptableToolPanelComponent = (props) => {
     const visibleToolPanels = [];
     const visibleToolPanelControls = (props.ToolPanels ?? []).map((toolPanelDefinition) => {
         const visibilityMode = toolPanelDefinition.VisibilityMode ?? 'collapsed';
-        // 1. check if it is a custom toolPanel
         const customToolPanel = props.api.toolPanelApi.getCustomToolPanelByName(toolPanelDefinition.Name);
         if (customToolPanel) {
             visibleToolPanels.push(customToolPanel.name);
             return (_jsx(ToolPanelWrapper, { customToolPanel: customToolPanel, visibilityMode: visibilityMode, onVisibilityModeChange: (state) => onStateChange(customToolPanel.name, state) }, customToolPanel.name));
         }
-        // 2. check if it's an available module toolPanel
         if (availableModuleToolPanels.some((module) => module === toolPanelDefinition.Name)) {
-            // assertion is safe as if just checked it in the if clause
             const moduleToolPanel = toolPanelDefinition.Name;
             visibleToolPanels.push(moduleToolPanel);
             return (_jsx(ToolPanelWrapper, { adaptableToolPanel: moduleToolPanel, visibilityMode: visibilityMode, onVisibilityModeChange: (state) => onStateChange(moduleToolPanel, state) }, moduleToolPanel));
@@ -95,17 +90,14 @@ const AdaptableToolPanelComponent = (props) => {
                 label: (_jsx(CheckBox, { className: "ab-dd-checkbox twa:my-0", value: toolPanel, checked: isVisible, onMouseDown: preventDefault, children: toolPanel }, toolPanel)),
             };
         };
-        // 1. process custom tool panels
         props.api.toolPanelApi.getCustomToolPanels().forEach((customToolPanel) => {
             const customToolPanelName = customToolPanel.name;
             toolpanelItems.push(buildDropdownButtonItem(customToolPanelName, customToolPanelName, isItemVisible(customToolPanelName)));
         });
-        // 2. process module tool panels
         availableModuleToolPanels.forEach((toolPanel) => {
             let moduleName = moduleService.getModuleInfoByModule(toolPanel).FriendlyName;
             toolpanelItems.push(buildDropdownButtonItem(toolPanel, moduleName, isItemVisible(toolPanel)));
         });
-        // 3. sort the dropdown items alphabetically
         toolpanelItems.sort((first, second) => {
             if (first.label < second.label) {
                 return -1;
@@ -162,13 +154,7 @@ const AdaptableToolPanelComponent = (props) => {
             } }));
     };
     const toolPanelButtons = renderToolPanelButtons();
-    return (_jsxs(Flex // Cannot have the with of the parent, on windows, when you have scrollbars
-    // the inner-width is smaller
-    // minWidth: 'var(--ab-cmp-toolpanel__width)',
-    , { 
-        // the inner-width is smaller
-        // minWidth: 'var(--ab-cmp-toolpanel__width)',
-        className: "ab-ToolPanel twa:p-2 twa:w-full", "data-name": "adaptable-tool-panel", flexDirection: "column", justifyContent: "center", children: [_jsxs(Flex, { className: "ab-ToolPanel__header twa:p-1 twa:w-full twa:justify-start", flexDirection: "row", flexWrap: "wrap", children: [isToolPanelModuleConfigurable && renderToolPanelDropdowns(), toolPanelButtons, isToolPanelModuleConfigurable && renderToolPanelConfigureButton()] }), Boolean(props.api.toolPanelApi.getCustomToolPanelButtons().length) && (_jsx(Flex, { className: "ab-ToolPanel__header__buttons twa:p-1 twa:justify-start", flexDirection: "row", flexWrap: "wrap", children: renderCustomToolPanelButtons() })), visibleToolPanelControls] }));
+    return (_jsxs(Flex, { className: "ab-ToolPanel twa:p-2 twa:w-full", "data-name": "adaptable-tool-panel", flexDirection: "column", justifyContent: "center", children: [_jsxs(Flex, { className: "ab-ToolPanel__header twa:p-1 twa:w-full twa:justify-start", flexDirection: "row", flexWrap: "wrap", children: [isToolPanelModuleConfigurable && renderToolPanelDropdowns(), toolPanelButtons, isToolPanelModuleConfigurable && renderToolPanelConfigureButton()] }), Boolean(props.api.toolPanelApi.getCustomToolPanelButtons().length) && (_jsx(Flex, { className: "ab-ToolPanel__header__buttons twa:p-1 twa:justify-start", flexDirection: "row", flexWrap: "wrap", children: renderCustomToolPanelButtons() })), visibleToolPanelControls] }));
 };
 function mapStateToProps(state) {
     return {
@@ -210,10 +196,8 @@ export const getAdaptableToolPanelAgGridComponent = (adaptable) => {
             const api = adaptable.api;
             this.gui = document.createElement('div');
             this.gui.id = getContainerId();
-            // preserve AG Grid naming convention
             this.gui.className = 'ag-adaptable-panel';
             Object.keys(toolPanelContainerStyle).forEach((key) => {
-                //@ts-ignore
                 this.gui.style[key] = toolPanelContainerStyle[key];
             });
             this.unmountReactRoot = adaptable.renderReactRoot(renderWithAdaptableContext(_jsx(ConnectedAdaptableToolPanel, { api: api, teamSharingActivated: false }), adaptable), this.gui);
@@ -225,7 +209,6 @@ export const getAdaptableToolPanelAgGridComponent = (adaptable) => {
             return this.gui;
         }
         refresh() {
-            // no refresh logic needed
         }
         destroy() {
             this.unmountReactRoot?.();

package/src/View/Components/ToolPanel/ToolPanelPopupSections.js

@@ -14,10 +14,6 @@ export var ToolPanelConfigView;
 })(ToolPanelConfigView || (ToolPanelConfigView = {}));
 export const ToolPanelPopupSections = (props) => {
     const { api } = useAdaptable();
-    // Start with both cards collapsed so the user sees the popup's full
-    // summary at a glance and explicitly chooses which to edit. `initialTab`
-    // is still accepted on the props for backwards compatibility but no
-    // longer pre-expands a card.
     const { bindCard, hasExpandedCard, expandedFillsSpace } = useWizardCardAccordion(null);
     const selectedModuleButtons = [];
     if (ArrayExtensions.IsNotNullOrEmpty(props.ToolPanelState.ModuleButtons)) {

package/src/View/Components/ToolPanel/ToolPanelWrapper.js

@@ -25,7 +25,6 @@ export const ToolPanelWrapper = (props) => {
             ];
         }
         else {
-            // custom toolPanel
             return [
                 customToolPanel.name,
                 customToolPanel?.title ?? customToolPanel.name,
@@ -35,9 +34,7 @@ export const ToolPanelWrapper = (props) => {
     };
     const [toolPanelName, toolPanelHeader, toolPanelContent, moduleInfo] = getToolPanelConfiguration();
     const isMinimised = visibilityMode === 'collapsed';
-    const isConfigurable = 
-    // only module toolPanels are configurable
-    !!moduleInfo && toolPanelName !== 'OpenFin' && toolPanelName !== 'IPushPull';
+    const isConfigurable = !!moduleInfo && toolPanelName !== 'OpenFin' && toolPanelName !== 'IPushPull';
     const isToolPanelModuleConfigurable = api.internalApi
         .getModuleService()
         .getModuleById(ToolPanelModuleId)

package/src/View/Components/ValueSelector/index.js

@@ -32,7 +32,6 @@ function useValuesMap({ options, value, toIdentifier, selectedMap, }) {
     return { selectedMap: result, optionsMap };
 }
 const baseClassName = 'ab-ValueSelector';
-/** Clicks on these targets must not toggle row selection (`toggleSelectionOnRowClick`). */
 const ROW_CLICK_IGNORE_SELECTOR = 'button, input, select, textarea, a, [data-drag-handle], [data-slot=select-trigger], [data-slot=select-content], [data-slot=select-item], .ab-Select';
 const ROW_TOGGLE_SUPPRESS_MS = 150;
 export function ValueSelector(props) {
@@ -241,9 +240,6 @@ export const renderSelectionSection = (props) => {
                         props.onClear();
                     }
                 }, children: "Select All" })), hideShowSelectedOnly ? null : (_jsx(Box, { className: "twa:shrink-0 twa:text-2 twa:font-normal", children: props.showOnlySelectedCheckbox })), showFilterInput && filter ? (_jsxs(_Fragment, { children: [compactFilterUsesSpacer ? _jsx(Box, { className: "twa:flex-1 twa:min-w-0" }) : null, _jsx(AdaptableFormControlTextClear, { value: searchInputValue ?? '', OnTextChange: setSearchInputValue ?? (() => undefined), placeholder: filterPlaceholder ?? 'Search...', className: compactFilterClass })] })) : null] })) : (_jsx(Box, { className: "twa:flex twa:flex-col twa:gap-2 twa:py-2", children: _jsxs(Box, { className: headerRowClassName, children: [singleSelect ? (_jsx(Box, { className: "twa:flex twa:flex-row twa:text-3 twa:font-normal twa:items-center", children: (() => {
-                        // For single-select we show the *name* of the selected option
-                        // rather than a (1 of N) counter — useful when the list is
-                        // long enough to scroll past the chosen row.
                         if (!value.length) {
                             return _jsx("span", { className: "twa:opacity-70", children: "(none selected)" });
                         }

package/src/View/Components/wizardColumnListStyles.js

@@ -1,10 +1,5 @@
-/** ValueSelector row sizing — matches Visibility and Order list rows. */
 export const WIZARD_COLUMN_LIST_OPTION_CLASS = 'twa:!py-1 twa:!px-1.5';
-/** Single-select column lists — ~50% taller than multi-select compact rows. */
 export const WIZARD_COLUMN_LIST_OPTION_CLASS_SINGLE_SELECT = 'twa:!py-1.5 twa:!px-1.5';
-/** Taller rows when ultra-compact one-line cards are not required (e.g. Custom Sort wizard). */
 export const WIZARD_COLUMN_LIST_OPTION_CLASS_RELAXED = 'twa:!py-2 twa:!px-2';
-/** Compact list header — spacing between Select All, Show Selected Only, and search. */
 export const WIZARD_COLUMN_LIST_HEADER_CLASS = 'twa:gap-4';
-/** Narrow search field (matches Visibility list column width, not full card width). */
 export const WIZARD_COLUMN_LIST_SEARCH_CLASS = 'twa:w-[12rem] twa:shrink-0 twa:min-w-0';

package/src/View/Dashboard/CustomToolbar.js

@@ -7,7 +7,6 @@ import { AdaptableFormComponent } from '../../components/AdaptableFormComponent/
 import { getDefaultAdaptableFormData, validateAdaptableForm, } from '../../AdaptableState/Common/AdaptableForm';
 export const CustomToolbarCmp = (props) => {
     const { api } = useAdaptable();
-    // dummy object which is used to force a re-render of the toolbar
     const [componentRevision, setComponentRevision] = useState(1);
     const renderContentContainerId = api.dashboardApi.internalApi.getCustomToolbarRenderContainerId(props.customToolbar.name);
     const renderContentContainerRef = useRef(null);
@@ -16,17 +15,9 @@ export const CustomToolbarCmp = (props) => {
     const componentContainerId = api.dashboardApi.internalApi.getCustomToolbarComponentContainerId(props.customToolbar.name);
     const componentContainerRef = useRef(null);
     const formContainerId = `${buttonsContainerId}-form`;
-    // Local form state for `customToolbar.toolbarForm`.
-    // Defaults are seeded from the form's field `defaultValue`s (if any) on
-    // first mount; subsequent definition swaps (different toolbar instance) are
-    // re-seeded too because the dependency includes the form definition.
     const [toolbarFormData, setToolbarFormData] = useState(() => props.customToolbar.toolbarForm
         ? getDefaultAdaptableFormData(props.customToolbar.toolbarForm)
         : {});
-    // Register a reset callback so `dashboardApi.resetCustomToolbarFormData(name)`
-    // can wipe this toolbar's state back to its declared defaults without
-    // forcing a re-mount. The unregister guard inside the internal API
-    // ensures a stale unmount can't clobber a fresh registration.
     useEffect(() => {
         if (!props.customToolbar.toolbarForm) {
             return;
@@ -34,9 +25,6 @@ export const CustomToolbarCmp = (props) => {
         const resetter = () => {
             const next = getDefaultAdaptableFormData(props.customToolbar.toolbarForm);
             setToolbarFormData(next);
-            // Mirror the public form-change callback so consumer code (e.g. a
-            // grid-filter helper) can rebuild itself from the cleared data
-            // without re-implementing default-seeding.
             if (props.customToolbar.onToolbarFormChange) {
                 props.customToolbar.onToolbarFormChange(next, buildToolbarFormContext(next));
             }
@@ -45,8 +33,6 @@ export const CustomToolbarCmp = (props) => {
         return () => {
             api.dashboardApi.internalApi.unregisterCustomToolbarFormResetter(props.customToolbar.name, resetter);
         };
-        // We deliberately depend on the toolbar identity + form definition so
-        // that a re-render with the same toolbar doesn't churn the registration.
     }, [api, props.customToolbar.name, props.customToolbar.toolbarForm]);
     const buildToolbarFormContext = (data) => {
         const baseCtx = {
@@ -55,9 +41,6 @@ export const CustomToolbarCmp = (props) => {
             formData: data,
             ...api.internalApi.buildBaseContext(),
         };
-        // Validate against the same context the consumer will see, then attach
-        // the results so button hidden/disabled/onClick callbacks can gate on
-        // form validity (e.g. a "Submit" button disabled while invalid).
         const formErrors = props.customToolbar.toolbarForm
             ? validateAdaptableForm(props.customToolbar.toolbarForm, data, baseCtx)
             : {};
@@ -80,7 +63,6 @@ export const CustomToolbarCmp = (props) => {
     useLayoutEffect(() => {
         const element = renderContentContainerRef.current;
         const adaptableApi = api;
-        // cleanup runs in a separate effect, this way it's executed only once on unmount
         return () => {
             if (props.customToolbar.render) {
                 props.customToolbar.render({
@@ -116,13 +98,7 @@ export const CustomToolbarCmp = (props) => {
                         ...api.internalApi.buildBaseContext(),
                     };
                     return (_jsx(AdaptableButtonView, { button: button, context: dashboardContext, api: api, defaults: ADAPTABLE_BUTTON_SURFACE_DEFAULTS.customToolbar, rerenderOnClick: false, style: { marginLeft: index ? 'var(--ab-base-space)' : 0 }, onAfterClick: () => setComponentRevision(componentRevision + 1) }, index));
-                }) })), props.customToolbar.toolbarForm && (_jsx("div", { id: formContainerId, className: `ab-CustomToolbar__form ab-CustomToolbar--${props.customToolbar.name}`, style: { minHeight: 22 }, children: _jsx(AdaptableFormComponent
-                // ensure the form uses the inline (toolbar-friendly) layout even
-                // if the consumer forgot to set it explicitly
-                , { 
-                    // ensure the form uses the inline (toolbar-friendly) layout even
-                    // if the consumer forgot to set it explicitly
-                    formDef: { ...props.customToolbar.toolbarForm, layout: 'inline' }, data: toolbarFormData, displayTitle: false, api: api, context: buildToolbarFormContext(toolbarFormData), focusFirstButton: false, onChange: (newData) => {
+                }) })), props.customToolbar.toolbarForm && (_jsx("div", { id: formContainerId, className: `ab-CustomToolbar__form ab-CustomToolbar--${props.customToolbar.name}`, style: { minHeight: 22 }, children: _jsx(AdaptableFormComponent, { formDef: { ...props.customToolbar.toolbarForm, layout: 'inline' }, data: toolbarFormData, displayTitle: false, api: api, context: buildToolbarFormContext(toolbarFormData), focusFirstButton: false, onChange: (newData) => {
                         setToolbarFormData(newData);
                         if (props.customToolbar.onToolbarFormChange) {
                             props.customToolbar.onToolbarFormChange(newData, buildToolbarFormContext(newData));

package/src/View/Dashboard/PinnedToolbarsSelector.js

@@ -26,7 +26,6 @@ export const PinnedToolbarsSelector = (props) => {
     const tabs = React.useMemo(() => {
         return [
             {
-                // Name: 'Pinned Toolbars',
                 Name: '',
                 Items: pinnedToolbars ?? [],
             },

package/src/View/DataImport/DataImportWizard/DataImportWizard.js

@@ -16,9 +16,6 @@ export const DataImportWizard = (props) => {
     const [rowData, setRowData] = React.useState(null);
     const [file, setFile] = React.useState(null);
     const [text, setText] = React.useState('');
-    // This needs to be called only when:
-    // - columnsMap changes
-    // - new data is loaded
     const [importType, setImportType] = React.useState('file');
     const primaryKey = adaptableApi.optionsApi.getPrimaryKey();
     const hasDynamicallyAddedPrimaryKey = typeof dataImportOptions._getPrimaryKeyValue === 'function';
@@ -38,7 +35,6 @@ export const DataImportWizard = (props) => {
                 rowData,
             });
         }
-        // by default just trim the keys
         let processedRowData = {};
         if (rowData) {
             processedRowData = Object.keys(rowData).reduce((acc, key) => {
@@ -48,11 +44,9 @@ export const DataImportWizard = (props) => {
         }
         return processedRowData;
     };
-    // ---- COLUMNS ----
     const [columnsMap, setColumnsMap] = React.useState(null);
     const mappedRowDataToColumns = React.useMemo(() => {
         return (rowData ?? []).map((row) => {
-            // pick only included columns, and need to map to abColumn.field
             return columnsMap?.reduce((acc, colMap) => {
                 if (colMap.include && colMap.abColumn && colMap.abColumn.field) {
                     acc[colMap.abColumn.field] = row[colMap.field];
@@ -84,24 +78,19 @@ export const DataImportWizard = (props) => {
             const allAbColumns = adaptableApi.columnApi.getColumns();
             const columnsMap = Array.from(fields).map((field) => {
                 const abColumn = allAbColumns.find((c) => {
-                    // exact field match
                     if (c.field === field) {
                         return true;
                     }
-                    // based on friendly name
                     const friendlyName = c.friendlyName;
                     if (typeof friendlyName !== 'string') {
                         return false;
                     }
-                    // exact match
                     if (friendlyName === field) {
                         return true;
                     }
-                    // match without case
                     if (friendlyName.toLowerCase() === field.toLowerCase()) {
                         return true;
                     }
-                    // match without spaces and case, and special characters: ' ', '-', '_'
                     if (friendlyName.replace(/\s|-|_/g, '').toLowerCase() === field.toLowerCase()) {
                         return true;
                     }
@@ -126,12 +115,10 @@ export const DataImportWizard = (props) => {
     }, []);
     const handleChangeImportType = React.useCallback((importType) => {
         setImportType(importType);
-        // clear data
         handleNewRowData(null);
         setFile(null);
         setText('');
     }, []);
-    // ---- FILE HANDLERS ----
     const fileHandlers = React.useMemo(() => {
         const userDefinedHandlers = adaptableApi.optionsApi.getDataImportOptions()?.fileHandlers ?? [];
         return [...userDefinedHandlers, ...systemFileHandlers];
@@ -158,13 +145,10 @@ export const DataImportWizard = (props) => {
         }
     }, [adaptable]);
     const fileMessage = file ? (_jsxs(HelpBlock, { className: "twa:mb-2", children: ["File ", _jsx(Tag, { children: file.name }), " is ready to be imported."] })) : (_jsxs(HelpBlock, { className: "twa:mb-2", children: ["Supported file types: ", supportedFileFormats] }));
-    // ---- TEXT IMPORT ----
     const [textError, setTextError] = React.useState(null);
     const handleTextChange = React.useCallback(async (text) => {
-        //
         let textHandler = adaptableApi.optionsApi.getDataImportOptions()?.textHandler;
         if (!textHandler) {
-            // regex to test for json
             const jsonRegex = /^\s*[\[{]/;
             if (text && jsonRegex.test(text)) {
                 textHandler = JSON.parse;
@@ -196,7 +180,6 @@ export const DataImportWizard = (props) => {
         }
     }, []);
     const textMessage = textError ?? (!rowData ? 'Paste your data here' : '');
-    // ---- VALIDATION ----
     const errors = React.useMemo(() => {
         if (!dataImportOptions?.validate) {
             return null;
@@ -237,11 +220,7 @@ export const DataImportWizard = (props) => {
                 },
                 details: 'Upload the data',
                 render: () => {
-                    return (_jsx(UploadSection, { importType: importType, onImportTypeChange: handleChangeImportType, 
-                        // FILE
-                        readFile: readFile, supportedFileFormats: supportedFileFormats, fileMessage: fileMessage, 
-                        // TEXT
-                        text: text, onTextChange: handleTextChange, textMessage: textMessage }));
+                    return (_jsx(UploadSection, { importType: importType, onImportTypeChange: handleChangeImportType, readFile: readFile, supportedFileFormats: supportedFileFormats, fileMessage: fileMessage, text: text, onTextChange: handleTextChange, textMessage: textMessage }));
                 },
             },
             {

package/src/View/DataImport/DataImportWizard/sections/DataPreview.js

@@ -107,7 +107,6 @@ export const DataPreview = (props) => {
                     return isExistingRow ? 'Update' : 'New';
                 }
                 catch (error) {
-                    // empty
                 }
                 return _jsx(_Fragment, {});
             },

package/src/View/DataImport/systemFileHandlers.js

@@ -16,7 +16,7 @@ export const parseCSV = (content) => {
             const item = {};
             for (let j = 0; j < headers.length; j++) {
                 const value = values[j];
-                item[headers[j]] = value === '' || isNaN(Number(value)) ? value : Number(value); // '0012345'
+                item[headers[j]] = value === '' || isNaN(Number(value)) ? value : Number(value);
             }
             data.push(item);
         }

package/src/View/FlashingCell/Wizard/isValidFlashingCellRules.js

@@ -7,7 +7,6 @@ export const isValidFlashingCellRules = (flashingCell, api, context) => {
     if (typeof isRuleValid === 'string') {
         return isRuleValid;
     }
-    // boolean expressions may also be saved as named queries
     if (context.namedQuery != false) {
         const isValidName = api.namedQueryApi.isValidNamedQuery(context.namedQuery);
         if (!isValidName.valid) {

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;
         }