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

package/src/Utilities/Helpers/StyleHelper.js

@@ -4,10 +4,6 @@ exports.getAutoContrastTextColor = exports.DEFAULT_CELL_SURFACE_COLOR = exports.
 const tslib_1 = require("tslib");
 const tinycolor2_1 = tslib_1.__importDefault(require("tinycolor2"));
 const EnumExtensions_1 = require("../Extensions/EnumExtensions");
-/**
- * Keys belonging to the "Font" slice (text rendering + alignment).
- * See {@link CellFontStyle}.
- */
 exports.CELL_FONT_STYLE_KEYS = [
     'ForeColor',
     'FontWeight',
@@ -16,10 +12,6 @@ exports.CELL_FONT_STYLE_KEYS = [
     'FontSize',
     'Alignment',
 ];
-/**
- * Keys belonging to the "Cell" slice (cell chrome — background and border).
- * See {@link CellBoxStyle}.
- */
 exports.CELL_BOX_STYLE_KEYS = [
     'BackColor',
     'BorderColor',
@@ -31,19 +23,10 @@ const hasAtLeastOneKey = (obj, keys) => {
     }
     return keys.some((k) => obj[k] !== undefined && obj[k] !== null);
 };
-/**
- * Returns true when the given style has any property in the Font slice set.
- */
 const hasCellFontStyle = (style) => hasAtLeastOneKey(style, exports.CELL_FONT_STYLE_KEYS);
 exports.hasCellFontStyle = hasCellFontStyle;
-/**
- * Returns true when the given style has any property in the Cell (box) slice set.
- */
 const hasCellBoxStyle = (style) => hasAtLeastOneKey(style, exports.CELL_BOX_STYLE_KEYS);
 exports.hasCellBoxStyle = hasCellBoxStyle;
-/**
- * Returns a copy of `style` containing only Font-slice keys.
- */
 const pickCellFontStyle = (style) => {
     const out = {};
     if (!style) {
@@ -58,9 +41,6 @@ const pickCellFontStyle = (style) => {
     return out;
 };
 exports.pickCellFontStyle = pickCellFontStyle;
-/**
- * Returns a copy of `style` containing only Cell-slice (box) keys.
- */
 const pickCellBoxStyle = (style) => {
     const out = {};
     if (!style) {
@@ -75,25 +55,18 @@ const pickCellBoxStyle = (style) => {
     return out;
 };
 exports.pickCellBoxStyle = pickCellBoxStyle;
-/**
- * Returns a copy of `style` with all Font-slice keys removed.
- */
 const omitCellFontStyle = (style) => {
     const out = { ...(style ?? {}) };
     exports.CELL_FONT_STYLE_KEYS.forEach((k) => delete out[k]);
     return out;
 };
 exports.omitCellFontStyle = omitCellFontStyle;
-/**
- * Returns a copy of `style` with all Cell-slice (box) keys removed.
- */
 const omitCellBoxStyle = (style) => {
     const out = { ...(style ?? {}) };
     exports.CELL_BOX_STYLE_KEYS.forEach((k) => delete out[k]);
     return out;
 };
 exports.omitCellBoxStyle = omitCellBoxStyle;
-// required for hacky fix for https://github.com/AdaptableTools/adaptable/issues/2119
 exports.AgGridCellStyleProperties = [
     'backgroundColor',
     'color',
@@ -101,9 +74,6 @@ exports.AgGridCellStyleProperties = [
     'fontStyle',
     'fontSize',
     'borderColor',
-    // needed for quick search
-    // as otherwise, the currentTextMatchStyle is not removed correctly
-    // from the cell when looping through the values, and the cell is no longer the current match
     '--ab-dynamic-background-color',
     '--ab-dynamic-color',
     '--ab-dynamic-font-weight',
@@ -113,9 +83,6 @@ exports.AgGridCellStyleProperties = [
     '--ab-dynamic-border-radius',
     '--ab-dynamic-border-color',
 ];
-// see https://github.com/AdaptableTools/adaptable/issues/2119
-// since v29 AG Grid ignores style properties with null or undefined values
-// Fix: pass empty string instead of null/undefined
 const normalizeStyleForAgGrid = (style) => {
     const normalizedStyle = { ...style };
     exports.AgGridCellStyleProperties.forEach((cellStylePropKey) => {
@@ -126,7 +93,6 @@ const normalizeStyleForAgGrid = (style) => {
     return normalizedStyle;
 };
 exports.normalizeStyleForAgGrid = normalizeStyleForAgGrid;
-// the inverse of normalizeStyleForAgGrid, see comments above
 const sanitizeStyle = (style) => {
     const sanitizedStyle = {};
     Object.keys(style).forEach((key) => {
@@ -190,11 +156,9 @@ const convertAdaptableStyleToCSS = (style) => {
                 break;
         }
     }
-    // assertion added to comply with the ag-Grid types
     return result;
 };
 exports.convertAdaptableStyleToCSS = convertAdaptableStyleToCSS;
-// based on http://jerekdain.com/fontconversion.html
 const convertCSSAbsoluteFontSizeToPt = (fontSize) => {
     switch (fontSize) {
         case 'x-large':
@@ -232,28 +196,8 @@ const getVariableColor = (variable) => {
     return getComputedStyle(document.documentElement).getPropertyValue(match[1]).trim();
 };
 exports.getVariableColor = getVariableColor;
-/**
- * Default surface colour assumed to sit *behind* a semi-transparent cell tint
- * when computing auto-contrast text. Gradient cells paint a base colour at a
- * value-driven alpha, so the colour the user actually perceives is that tint
- * composited over the grid cell background (white in the default light theme).
- */
 exports.DEFAULT_CELL_SURFACE_COLOR = '#ffffff';
-/**
- * Luminance above which a (composited, opaque) background is considered "light"
- * enough to need dark text.
- */
 const AUTO_CONTRAST_LUMINANCE_THRESHOLD = 0.45;
-/**
- * Picks a readable text colour (near-black or white) for a cell whose
- * background is `baseColor` painted at `alpha` over `surfaceColor`.
- *
- * The tint must be composited over the surface *before* measuring luminance:
- * `tinycolor.getLuminance()` ignores the alpha channel, so measuring the raw
- * base colour would yield the same result for a faint cell and a fully-opaque
- * one — making auto-contrast text identical on every row instead of adapting
- * per cell.
- */
 const getAutoContrastTextColor = (baseColor, alpha, surfaceColor = exports.DEFAULT_CELL_SURFACE_COLOR) => {
     const composited = tinycolor2_1.default.mix(surfaceColor, baseColor, alpha * 100);
     return composited.getLuminance() > AUTO_CONTRAST_LUMINANCE_THRESHOLD ? '#111111' : '#ffffff';

package/src/Utilities/Helpers/StyledColumns/BarStylesHelper.js

@@ -1,7 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.mountBarStyleCellText = exports.formatBarStyleCellTextLayoutSummary = exports.getBarStyleCellTextSlotPresence = exports.resolveBarStyleCellTextSlots = exports.joinBarStyleCellTextLabels = exports.hasBarStyleCellTextLabels = exports.buildBarStyleCellTextLabels = exports.toggleBarStyleCellTextToken = exports.patchBarStyleCellTextPlacement = exports.setBarStyleCellTextPlacement = exports.mergeBarStyleCellTextProperties = exports.getActiveBarStyleCellTextTokens = exports.hasBarStyleCellTextConfigured = exports.resolveBarStyleCellTextLayout = void 0;
-/** Default placement applied to a value whose Horizontal/Vertical were omitted. */
 const DEFAULT_PLACEMENT = {
     Horizontal: 'Left',
     Vertical: 'Below',
@@ -10,14 +9,12 @@ const HORIZONTAL_VALUES = ['Left', 'Center', 'Right'];
 const VERTICAL_VALUES = ['Above', 'Below', 'Merged'];
 const isHorizontal = (value) => typeof value === 'string' && HORIZONTAL_VALUES.includes(value);
 const isVertical = (value) => typeof value === 'string' && VERTICAL_VALUES.includes(value);
-/** Normalise a placement, applying defaults for missing axes. */
 const sanitizePlacement = (placement) => ({
     Horizontal: isHorizontal(placement?.Horizontal)
         ? placement.Horizontal
         : DEFAULT_PLACEMENT.Horizontal,
     Vertical: isVertical(placement?.Vertical) ? placement.Vertical : DEFAULT_PLACEMENT.Vertical,
 });
-/** Returns a layout with only the values that are actually present, defaults filled in. */
 const sanitizeCellTextLayout = (layout) => {
     const result = {};
     if (layout?.CellValue) {
@@ -30,13 +27,11 @@ const sanitizeCellTextLayout = (layout) => {
 };
 const resolveBarStyleCellTextLayout = (props) => sanitizeCellTextLayout(props?.CellTextLayout);
 exports.resolveBarStyleCellTextLayout = resolveBarStyleCellTextLayout;
-/** True when any value is configured for display. */
 const hasBarStyleCellTextConfigured = (props) => {
     const layout = (0, exports.resolveBarStyleCellTextLayout)(props);
     return Boolean(layout.CellValue || layout.PercentValue);
 };
 exports.hasBarStyleCellTextConfigured = hasBarStyleCellTextConfigured;
-/** Set of value tokens that currently have a placement. */
 const getActiveBarStyleCellTextTokens = (props) => {
     const layout = (0, exports.resolveBarStyleCellTextLayout)(props);
     const tokens = [];
@@ -63,9 +58,6 @@ const mergeBarStyleCellTextProperties = (current, patch) => {
     return merged;
 };
 exports.mergeBarStyleCellTextProperties = mergeBarStyleCellTextProperties;
-/**
- * Set or clear a value's placement. Passing `placement: undefined` removes the value.
- */
 const setBarStyleCellTextPlacement = (current, token, placement) => {
     const layout = (0, exports.resolveBarStyleCellTextLayout)(current);
     const next = { ...layout };
@@ -79,10 +71,6 @@ const setBarStyleCellTextPlacement = (current, token, placement) => {
     return (0, exports.mergeBarStyleCellTextProperties)(current, { CellTextLayout: next });
 };
 exports.setBarStyleCellTextPlacement = setBarStyleCellTextPlacement;
-/**
- * Patch a single axis of a value's placement (creating the value with defaults if
- * needed). Used by the wizard's placement pickers.
- */
 const patchBarStyleCellTextPlacement = (current, token, patch) => {
     const layout = (0, exports.resolveBarStyleCellTextLayout)(current);
     const key = token === 'CellValue' ? 'CellValue' : 'PercentValue';
@@ -94,10 +82,6 @@ const patchBarStyleCellTextPlacement = (current, token, patch) => {
     return (0, exports.setBarStyleCellTextPlacement)(current, token, nextPlacement);
 };
 exports.patchBarStyleCellTextPlacement = patchBarStyleCellTextPlacement;
-/**
- * Toggle whether a value is displayed. When turning on a previously-hidden
- * value we use the default placement (Left / Below).
- */
 const toggleBarStyleCellTextToken = (current, token, show) => {
     const layout = (0, exports.resolveBarStyleCellTextLayout)(current);
     const key = token === 'CellValue' ? 'CellValue' : 'PercentValue';
@@ -128,7 +112,6 @@ const buildBarStyleCellTextLabels = (props, cellValueLabel, percentageLabel) =>
 exports.buildBarStyleCellTextLabels = buildBarStyleCellTextLabels;
 const hasBarStyleCellTextLabels = (labels) => Boolean(labels.cellValue || labels.percentage);
 exports.hasBarStyleCellTextLabels = hasBarStyleCellTextLabels;
-/** Join active labels with a space, Cell Value first then Percent Value. */
 const joinBarStyleCellTextLabels = (labels) => {
     const parts = [];
     if (labels.cellValue)
@@ -138,16 +121,8 @@ const joinBarStyleCellTextLabels = (labels) => {
     return parts.join(' ');
 };
 exports.joinBarStyleCellTextLabels = joinBarStyleCellTextLabels;
-/**
- * Resolve which placements are populated and what text goes in each cell of
- * the 3 × 3 grid. Empty slots are omitted. Two values landing on the same
- * (vertical, horizontal) slot are concatenated with a single space, Cell
- * Value before Percent Value.
- */
 const resolveBarStyleCellTextSlots = (props, labels) => {
     const layout = (0, exports.resolveBarStyleCellTextLayout)(props);
-    // Iteration order matters when both values share a slot: Cell Value first,
-    // then Percent Value (matches the legacy "joined" behaviour).
     const candidates = [];
     if (layout.CellValue && labels.cellValue) {
         candidates.push({
@@ -195,9 +170,6 @@ const getBarStyleCellTextSlotPresence = (props) => {
     };
 };
 exports.getBarStyleCellTextSlotPresence = getBarStyleCellTextSlotPresence;
-// ---------------------------------------------------------------------------
-// Summary formatting
-// ---------------------------------------------------------------------------
 const HORIZONTAL_LABEL = {
     Left: 'Left',
     Center: 'Center',
@@ -224,9 +196,6 @@ const formatBarStyleCellTextLayoutSummary = (layout) => {
     return parts.length ? parts.join('; ') : undefined;
 };
 exports.formatBarStyleCellTextLayoutSummary = formatBarStyleCellTextLayoutSummary;
-// ---------------------------------------------------------------------------
-// DOM mount
-// ---------------------------------------------------------------------------
 const HORIZONTAL_FLEX_JUSTIFY = {
     Left: 'flex-start',
     Center: 'center',
@@ -244,10 +213,6 @@ const createCellTextRow = (slots, textClassName, compact) => {
     row.style.flexDirection = 'row';
     row.style.alignItems = 'center';
     row.style.width = '100%';
-    // When `Above` AND `Below` are both populated, two default-size text rows
-    // would squeeze the bar to 0 on a standard ~28-40px cell, so the caller
-    // asks for a compact band (smaller font + tighter line-height).
-    // Otherwise we render at the inherited cell font for legibility.
     if (compact) {
         row.style.fontSize = '0.75em';
         row.style.lineHeight = '1';
@@ -257,8 +222,6 @@ const createCellTextRow = (slots, textClassName, compact) => {
     }
     row.style.flex = '0 0 auto';
     row.style.gap = '4px';
-    // Render fixed Left / Center / Right cells so spacing is stable even when
-    // only one or two slots are present.
     for (const horizontal of HORIZONTAL_VALUES) {
         const slot = slots.find((s) => s.horizontal === horizontal);
         const cell = document.createElement('span');
@@ -273,11 +236,6 @@ const createCellTextRow = (slots, textClassName, compact) => {
         }
         row.append(cell);
     }
-    // Suppress the dummy row in the rare case where every slot for this band is
-    // empty (e.g. the only label is empty string). `String(...)` because text
-    // may arrive as a number when `params.formatValue` is undefined for a
-    // numeric column — `(11).length` is `undefined`, which would otherwise
-    // wrongly trip the empty-row branch.
     if (!slots.some((s) => String(s.text ?? '').length)) {
         row.style.display = 'none';
     }
@@ -299,10 +257,6 @@ const mountBarStyleCellText = (args) => {
         Below: slots.filter((s) => s.vertical === 'Below'),
         Merged: slots.filter((s) => s.vertical === 'Merged'),
     };
-    // Compact when Cell Value and Percent Value have *different* verticals
-    // (i.e. the two tokens claim two separate bands of vertical space). When
-    // they share a vertical band — or when only one token is configured —
-    // we have room for the full-size font.
     const layout = (0, exports.resolveBarStyleCellTextLayout)(cellTextProperties);
     const compact = Boolean(layout.CellValue &&
         layout.PercentValue &&

package/src/Utilities/Helpers/StyledColumns/GradientStyleHelper.js

@@ -11,15 +11,7 @@ const tinycolor2_1 = tslib_1.__importDefault(require("tinycolor2"));
 const StyleHelper_1 = require("../StyleHelper");
 const StyleHelper_2 = require("../StyleHelper");
 const NumberExtensions_1 = require("../../Extensions/NumberExtensions");
-/**
- * Default alpha at the **low** end of a Gradient range when `MinAlpha` is unset.
- * Slightly above zero so the bottom-edge value of every range still shows its
- * colour (rather than rendering fully transparent).
- */
 exports.DEFAULT_GRADIENT_MIN_ALPHA = 0.15;
-/**
- * Default alpha at the **high** end of a Gradient range when `MaxAlpha` is unset.
- */
 exports.DEFAULT_GRADIENT_MAX_ALPHA = 1;
 const hasGradientRangesConfigured = (gs) => !!(gs?.ZeroCentred || gs?.CellRanges?.length || gs?.ColumnComparison);
 exports.hasGradientRangesConfigured = hasGradientRangesConfigured;
@@ -30,12 +22,9 @@ const evenlySpacedValues = (min, max, count) => {
     const step = (max - min) / (count - 1);
     return Array.from({ length: count }, (_, i) => min + step * i);
 };
-/** Number of sample cells shown in Gradient / Percent Bar style previews. */
 exports.STYLED_COLUMN_PREVIEW_SAMPLE_COUNT = 6;
-/** Illustrative zero-centred scale endpoints for preview geometry. */
 exports.ZERO_CENTRED_PREVIEW_SCALE_MIN = -20;
 exports.ZERO_CENTRED_PREVIEW_SCALE_MAX = 20;
-/** @deprecated Use {@link getZeroCentredGradientPreviewValues} — kept for callers using endpoints. */
 exports.ZERO_CENTRED_GRADIENT_PREVIEW_VALUES = evenlySpacedValues(exports.ZERO_CENTRED_PREVIEW_SCALE_MIN, exports.ZERO_CENTRED_PREVIEW_SCALE_MAX, exports.STYLED_COLUMN_PREVIEW_SAMPLE_COUNT);
 const getZeroCentredGradientPreviewValues = () => evenlySpacedValues(exports.ZERO_CENTRED_PREVIEW_SCALE_MIN, exports.ZERO_CENTRED_PREVIEW_SCALE_MAX, exports.STYLED_COLUMN_PREVIEW_SAMPLE_COUNT);
 exports.getZeroCentredGradientPreviewValues = getZeroCentredGradientPreviewValues;
@@ -49,9 +38,6 @@ function isDivergingZeroCellRanges(ranges) {
 function isZeroCentredGradientStyle(gs) {
     return !!(gs.ZeroCentred || isDivergingZeroCellRanges(gs.CellRanges));
 }
-/**
- * Sample cell values for gradient preview strips — spread across the effective scale.
- */
 const getGradientPreviewSampleValues = (styledColumn, api) => {
     const gs = styledColumn.GradientStyle;
     if (!gs || !(0, exports.hasGradientRangesConfigured)(gs)) {
@@ -98,11 +84,6 @@ const getZeroCentredPreviewColors = (gradientStyle) => {
     }
     return null;
 };
-/**
- * Zero-centred preview uses a fixed -20…20 illustrative scale so negative
- * colours always appear — independent of whether the column currently has
- * negative values (Col-Min may be above zero today).
- */
 const getZeroCentredGradientPreviewCellStyle = (cellValue, gradientStyle) => {
     const colors = getZeroCentredPreviewColors(gradientStyle);
     if (!colors) {
@@ -159,9 +140,6 @@ const getZeroCentredGradientPreviewCellStyle = (cellValue, gradientStyle) => {
     }
     return style;
 };
-/**
- * Cell CSS for gradient preview — mirrors `AgGridColumnAdapter.getStyledColumnStyle` gradient path.
- */
 const getGradientPreviewCellStyle = (cellValue, styledColumn, api, rowNode) => {
     const gradientStyle = styledColumn.GradientStyle;
     if (!gradientStyle || !(0, exports.hasGradientRangesConfigured)(gradientStyle)) {
@@ -227,10 +205,6 @@ const getGradientPreviewCellStyle = (cellValue, styledColumn, api, rowNode) => {
     return style;
 };
 exports.getGradientPreviewCellStyle = getGradientPreviewCellStyle;
-/**
- * Effective cell ranges for gradient rendering / range lookup.
- * When `ZeroCentred` is set, expands to the canonical Col-Min…0 and 0…Col-Max bands.
- */
 function expandGradientCellRanges(gs) {
     if (gs.ZeroCentred) {
         return [
@@ -240,21 +214,9 @@ function expandGradientCellRanges(gs) {
     }
     return gs.CellRanges;
 }
-/**
- * Value type passed to `findRangeForColumn` / `getCellColorRangeMinValue` /
- * `getCellColorRangeMaxValue` when resolving which gradient band a cell falls in.
- *
- * Zero-centred bands use numeric `0` as an **absolute** zero crossing between
- * `Col-Min…0` and `0…Col-Max`. Under `RangeValueType: 'Percentage'`, numeric
- * endpoints are interpreted as % positions along the column min–max span, so
- * literal `0` wrongly becomes `Col-Min` and the negative band collapses to a
- * single point — boundary cells (e.g. the column minimum) pick up the negative
- * colour incorrectly. Zero-centred lookup must always use Number semantics.
- */
 function gradientStyleRangeLookupValueType(gs) {
     return gs.ZeroCentred ? 'Number' : gs.RangeValueType ?? 'Number';
 }
-/** The negative half of a zero-centred gradient (darker toward Col-Min). */
 function isNegativeZeroCentredBand(range) {
     return range.Min === 'Col-Min' && range.Max === 0;
 }

package/src/Utilities/Helpers/StyledColumns/IconStyleHelper.js

@@ -2,10 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ICON_STYLE_PRESET_DESCRIPTIONS = exports.ICON_STYLE_PRESET_LABELS = exports.getIconStylePresetMappings = exports.ICON_STYLE_PRESETS = void 0;
 exports.isUnsupportedColumnDataTypeForIconStyle = isUnsupportedColumnDataTypeForIconStyle;
-/**
- * Icon Column Style resolves one key → glyph per scalar cell value; AdapTable
- * array column types are not supported (use Badge Style for per-element visuals).
- */
 function isUnsupportedColumnDataTypeForIconStyle(dataType) {
     switch (dataType) {
         case 'textArray':
@@ -17,28 +13,10 @@ function isUnsupportedColumnDataTypeForIconStyle(dataType) {
             return false;
     }
 }
-/**
- * Built-in {@link IconStyle} mapping presets.
- *
- * The wizard's "Apply preset" action seeds `IconStyle.Mappings` from one
- * of these. Once seeded the data is just normal mappings — users can add
- * / edit / remove rows freely.
- *
- * Implementation notes:
- *  - All icons here are emoji strings, so the presets cost nothing at
- *    bundle / asset level. On Windows some flag emojis fall back to
- *    regional-indicator letter pairs (e.g. "🇺🇸" → "US"), which is still
- *    a sensible visual; users who want a consistent SVG flag pack across
- *    platforms can swap individual mappings to URL icons.
- *  - Presets are deliberately *small* — the most common values for each
- *    domain. Users add the long tail by editing the mappings list.
- */
 const flagEmoji = (countryCodeAlpha2) => {
     const cc = countryCodeAlpha2.toUpperCase();
     if (cc.length !== 2)
         return cc;
-    // Convert ISO-3166 alpha-2 → regional indicator pair (each letter +
-    // 0x1F1A5). Browsers render this pair as the country flag.
     const a = 0x1f1a5;
     return String.fromCodePoint(a + cc.charCodeAt(0), a + cc.charCodeAt(1));
 };
@@ -58,8 +36,6 @@ const FLAG_PRESET = [
     { Key: 'NL', Icon: flagEmoji('NL'), Description: 'Netherlands' },
     { Key: 'PT', Icon: flagEmoji('PT'), Description: 'Portugal' },
     { Key: 'US', Icon: flagEmoji('US'), Description: 'United States' },
-    // Friendly synonyms so the preset works against full country names too
-    // (`MatchMode: 'CaseInsensitive'` is a sensible default for this preset).
     { Key: 'Australia', Icon: flagEmoji('AU'), Description: 'Australia' },
     { Key: 'Belgium', Icon: flagEmoji('BE'), Description: 'Belgium' },
     { Key: 'Canada', Icon: flagEmoji('CA'), Description: 'Canada' },
@@ -72,23 +48,11 @@ const FLAG_PRESET = [
     { Key: 'United Kingdom', Icon: flagEmoji('GB'), Description: 'United Kingdom' },
     { Key: 'United States', Icon: flagEmoji('US'), Description: 'United States' },
 ];
-// Currency → issuing country flag. Currency *symbols* (`$`, `¥`) are NOT
-// unique across currencies (USD/AUD/CAD/NZD all share `$`; CNY/JPY both use
-// `¥`), so an icon column built from symbols ends up visually ambiguous —
-// defeating the whole point of an icon column. Flags map 1-to-1 with the
-// issuing economy, give every currency a distinct glyph, and read instantly
-// in financial UIs (which is where this preset is most commonly used).
-//
-// Pair with `Description` ("US Dollar", "Euro") so tooltips / companion
-// text still surface the human-readable name. Anyone who prefers symbols
-// can still build that mapping by hand in the wizard.
 const CURRENCY_PRESET = [
     { Key: 'AUD', Icon: flagEmoji('AU'), Description: 'Australian Dollar' },
     { Key: 'CAD', Icon: flagEmoji('CA'), Description: 'Canadian Dollar' },
     { Key: 'CHF', Icon: flagEmoji('CH'), Description: 'Swiss Franc' },
     { Key: 'CNY', Icon: flagEmoji('CN'), Description: 'Chinese Yuan' },
-    // 'EU' as a regional indicator pair renders as the EU flag in modern
-    // emoji fonts; falls back to the letters "EU" otherwise — still readable.
     { Key: 'EUR', Icon: flagEmoji('EU'), Description: 'Euro' },
     { Key: 'GBP', Icon: flagEmoji('GB'), Description: 'British Pound' },
     { Key: 'HKD', Icon: flagEmoji('HK'), Description: 'Hong Kong Dollar' },
@@ -109,7 +73,6 @@ const TREND_PRESET = [
     { Key: 'Up', Icon: '⬆', Description: 'Up' },
     { Key: 'Down', Icon: '⬇', Description: 'Down' },
     { Key: 'Flat', Icon: '➡', Description: 'Flat' },
-    // Trading-side synonyms so the same preset works against Buy/Sell columns.
     { Key: 'Buy', Icon: '⬆', Description: 'Buy' },
     { Key: 'Sell', Icon: '⬇', Description: 'Sell' },
     { Key: 'Hold', Icon: '➡', Description: 'Hold' },
@@ -120,7 +83,6 @@ const STATUS_PRESET = [
     { Key: 'Pass', Icon: '🟢', Description: 'Pass' },
     { Key: 'Warn', Icon: '🟡', Description: 'Warn' },
     { Key: 'Fail', Icon: '🔴', Description: 'Fail' },
-    // Status-column synonyms (cover the `status` column in the trade demo).
     { Key: 'Completed', Icon: '🟢', Description: 'Completed' },
     { Key: 'In Progress', Icon: '🟡', Description: 'In Progress' },
     { Key: 'Pending', Icon: '🟡', Description: 'Pending' },
@@ -133,7 +95,6 @@ exports.ICON_STYLE_PRESETS = {
     Trend: TREND_PRESET,
     Status: STATUS_PRESET,
 };
-/** Returns a fresh copy of the mappings for a given preset. */
 const getIconStylePresetMappings = (preset) => {
     return exports.ICON_STYLE_PRESETS[preset].map((m) => ({ ...m }));
 };

package/src/Utilities/Helpers/StyledColumns/PercentBarStyleHelper.js

@@ -68,7 +68,6 @@ const getPreviewBarColor = (value, pb, scaleMin, scaleMax) => {
 };
 const hasPercentBarRangesConfigured = (pb) => !!(pb?.CellRanges?.length || pb?.ColumnComparison);
 exports.hasPercentBarRangesConfigured = hasPercentBarRangesConfigured;
-/** Illustrative scale for preview — shows negatives when origin is Auto or Zero. */
 const getPercentBarPreviewScale = (pb) => {
     const origin = pb.Origin ?? 'Auto';
     if (origin === 'Zero' || origin === 'Auto') {

package/src/Utilities/Helpers/StyledColumns/SparklineStyleHelper.js

@@ -22,11 +22,6 @@ const resolveHighlightStyle = (style) => {
     }
     return resolved;
 };
-/**
- * AG Charts sparklines do not resolve Adaptable `var(--ab-color-*)` tokens.
- * Resolve palette / theme CSS variables to concrete colours at render time so
- * wizard previews and grid cells match the chosen Fill / Stroke values.
- */
 function resolveSparklineOptionsForRender(options) {
     const resolved = { ...options };
     if ('fill' in options && options.fill != null) {

package/src/Utilities/Helpers/StyledColumns/StyledColumnHelper.js

@@ -25,11 +25,6 @@ const getStyledColumnRowKindDisabledReason = (styledColumn, kind) => {
     return undefined;
 };
 exports.getStyledColumnRowKindDisabledReason = getStyledColumnRowKindDisabledReason;
-/**
- * Classifies the AG Grid row node into a single {@link StyledColumnRowKind}.
- * Order matches {@link shouldRenderStyledColumnOnRow}: total before summary,
- * summary before group.
- */
 const getStyledColumnRowKindForNode = (node, api) => {
     if (api.gridApi.isGrandTotalRowNode(node)) {
         return 'Total';
@@ -43,11 +38,6 @@ const getStyledColumnRowKindForNode = (node, api) => {
     return 'Data';
 };
 exports.getStyledColumnRowKindForNode = getStyledColumnRowKindForNode;
-/**
- * Forces `Exclude*` to `true` for row kinds that do not apply to the current
- * Styled Column type, so persisted state cannot contradict the capability
- * matrix. Returns `undefined` when no change is needed.
- */
 const sanitizeStyledColumnRowScope = (styledColumn) => {
     const scope = styledColumn.RowScope;
     if (!scope) {
@@ -68,32 +58,6 @@ const sanitizeStyledColumnRowScope = (styledColumn) => {
     return changed ? next : undefined;
 };
 exports.sanitizeStyledColumnRowScope = sanitizeStyledColumnRowScope;
-/**
- * Returns `true` when the Styled Column should render on the given row.
- *
- * Single source of truth consulted by every styled-column cell renderer
- * (Badge / Icon / Rating / Percent Bar / Bullet Chart / Range Bar) and
- * by `getStyledColumnStyle` (Gradient / Percent Bar cellStyle).
- *
- * Semantics:
- *  - **Type capabilities** first: some types never target certain row kinds
- *    (see {@link isStyledColumnRowKindSupported}). Unsupported kinds always
- *    return `false` regardless of `RowScope`.
- *  - When `StyledColumn.RowScope` is **set**, the explicit `Exclude*`
- *    flags win. Unset flags default to "include" — i.e. `RowScope: {}`
- *    means "render on every *supported* row kind".
- *  - When `StyledColumn.RowScope` is **unset**, the historical per-type
- *    default applies:
- *      - Badge: render on every row kind (that the type supports).
- *      - All other types: render on every row kind *except* group rows
- *        (matches the hardcoded "isGroupRowNode → plain text" check that
- *        every other styled-column renderer used to carry), again
- *        intersected with type capabilities (e.g. Sparkline → data only).
- *
- * The per-type default lets us promote `RowScope` cleanly without
- * altering existing user-visible behaviour: state that doesn't mention
- * `RowScope` keeps rendering exactly as before.
- */
 const shouldRenderStyledColumnOnRow = (styledColumn, node, api) => {
     const rowKind = (0, exports.getStyledColumnRowKindForNode)(node, api);
     if (!(0, exports.isStyledColumnRowKindSupported)(styledColumn, rowKind)) {
@@ -115,7 +79,6 @@ const shouldRenderStyledColumnOnRow = (styledColumn, node, api) => {
             return false;
         return true;
     }
-    // No explicit RowScope → fall back to historical per-type behaviour.
     if (styledColumn.BadgeStyle) {
         return true;
     }

package/src/Utilities/Helpers/TimingHelper.js

@@ -1,14 +1,4 @@
 "use strict";
-/**
- * Timing helpers for delaying / rate-limiting function calls.
- *
- * `debounce` and `throttle` are drop-in replacements for the equivalent
- * lodash exports (`debounce` supports `{ leading, trailing, maxWait }`,
- * `throttle` supports `{ leading, trailing }`).
- *
- * Both return an instance of `DebouncedFunction<T>` which exposes
- * `cancel()` and `flush()` for managing pending invocations.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.TimingHelper = void 0;
 exports.debounce = debounce;
@@ -123,9 +113,6 @@ function debounce(func, wait = 0, options) {
     };
     return debounced;
 }
-/**
- * Implemented in terms of `debounce` with `maxWait` (same approach as lodash).
- */
 function throttle(func, wait = 0, options) {
     return debounce(func, wait, {
         leading: options?.leading ?? true,

package/src/Utilities/ObjectFactory.js

@@ -61,7 +61,6 @@ function CreateEmptyCalculatedColumn(defaultSpecialColumnSettings) {
         Query: {
             ScalarExpression: GeneralConstants_1.EMPTY_STRING,
         },
-        // need to create some defaults - which we will change later
         CalculatedColumnSettings: {
             ...defaultSpecialColumnSettings,
             DataType: undefined,
@@ -302,8 +301,8 @@ function CreateDefaultStyledColumnBadge() {
     return {
         Shape: 'Rounded',
         PillStyle: {
-            BackColor: (0, UIHelper_1.getSwatchColorVar)(3), // light gray by default (for styled column badge)
-            ForeColor: (0, UIHelper_1.getSwatchColorVar)(1), // black by default (for styled column badge)
+            BackColor: (0, UIHelper_1.getSwatchColorVar)(3),
+            ForeColor: (0, UIHelper_1.getSwatchColorVar)(1),
         },
     };
 }
@@ -377,7 +376,6 @@ function CreateToastOptions(notificationsOptions, { onClose, containerId }, over
         transition: transition,
         draggablePercent: 75,
         onClose,
-        // progress: undefined,
     };
     return toastProps;
 }

package/src/Utilities/Services/AggregatedScalarLiveValue.js

@@ -32,7 +32,6 @@ class AggregatedScalarLiveValue {
                 .getQueryLanguageService()
                 .evaluateAggregatedScalarExpression(source.aggregatedScalarExpression, requestingModule, getRowNodes).value
             : source.aggregatedScalarExpressionEvaluation;
-        // currently we support only one reducer
         this.aggregationReducerName = Object.keys(this.expressionEvaluation.aggregationParams.reducers)[0];
         this.aggregationResult = this.computeAggregatedValue(this.expressionEvaluation);
     }
@@ -42,7 +41,6 @@ class AggregatedScalarLiveValue {
                 .getQueryLanguageService()
                 .evaluateAggregatedScalarExpression(this.aggregatedScalarExpression, this.requestingModule, this.getRowNodes).value;
         }
-        // currently we support only one reducer
         this.aggregationReducerName = Object.keys(this.expressionEvaluation.aggregationParams.reducers)[0];
         this.aggregationResult = this.computeAggregatedValue(this.expressionEvaluation);
     }
@@ -79,7 +77,6 @@ class AggregatedScalarLiveValue {
     }
     getRowNodeValueForColumnId(rowNode, columnId) {
         const rawValue = this.adaptableApi.gridApi.getRawValueFromRowNode(rowNode, columnId);
-        // we need to convert the Date object to a number, otherwise the grouping will not work (each Date instance is unique)
         return rawValue instanceof Date ? rawValue.getTime() : rawValue;
     }
     computeAggregatedValue(expressionEvaluation) {
@@ -88,14 +85,11 @@ class AggregatedScalarLiveValue {
             : this.adaptableApi.gridApi.getAllRowNodes({
                 filterFn: expressionEvaluation.rowFilterFn,
             });
-        // we iterate over the RowNode list (we need this to handle complex column values (nested values, valueGetters etc)
-        // so we will map the fieldNames to RowNode.data
         const mapReducerValueGetter = (columnId) => {
             return (rowNode) => this.getRowNodeValueForColumnId(rowNode, columnId);
         };
         Object.values(expressionEvaluation.aggregationParams.reducers).forEach((aggregationReducer) => {
             aggregationReducer.getter = mapReducerValueGetter(aggregationReducer.field);
-            // nullify the field to force the fallback on the getter fn
             aggregationReducer.field = null;
         });
         const mapGroupByToKey = (columnId) => {
@@ -107,7 +101,6 @@ class AggregatedScalarLiveValue {
         let aggregatedRowNodes = gridRowNodes;
         if (expressionEvaluation.sortByColumn) {
             const sortByColumn = expressionEvaluation.sortByColumn;
-            // currently, we support only ascending sorting
             aggregatedRowNodes.sort((first, second) => {
                 const firstValue = this.getRowNodeValueForColumnId(first, sortByColumn);
                 const secondValue = this.getRowNodeValueForColumnId(second, sortByColumn);