@adaptabletools/adaptable 23.0.4 → 23.0.5-canary.0

package/src/components/WindowModal/useStacking.js

@@ -2,7 +2,6 @@ import * as React from 'react';
 import { createUuid } from '../utils/uuid';
 const setZIndexMap = {};
 const zIndexMap = [];
-// starts with 3000, to be above the default zIndex of the Modal component
 const BASE_Z_INDEX = 3000;
 const Z_INDEX_GAP = 5;
 let GLOBAL_COUNTER = 0;
@@ -13,18 +12,12 @@ const getNewOffset = () => {
 const getNewZIndex = () => {
     return BASE_Z_INDEX + getNewOffset();
 };
-/**
- * Used to facilitate the stacking between multiple opened windows.
- * When calling 'bringToFront' brings caller to top.
- * This is done by changing the zIndex of all registered window-modals.
- */
 export const useStacking = () => {
     const modalZIndexId = React.useMemo(() => createUuid(), []);
     const [zIndex, setZIndex] = React.useState(getNewZIndex);
     React.useEffect(() => {
         setZIndexMap[modalZIndexId] = setZIndex;
         return () => {
-            // clean-up
             delete setZIndexMap[modalZIndexId];
             const indexInMap = zIndexMap.indexOf(modalZIndexId);
             zIndexMap.splice(indexInMap, 1);

package/src/components/icons/bullet-chart.js

@@ -1,7 +1,3 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import DefaultIcon from './DefaultIcon';
-// Bullet Chart icon - a horizontal layout consisting of (back to front):
-//  - three qualitative bands (light/medium/dark) along the full width
-//  - a foreground "actual value" bar over the lower portion of the bands
-//  - a vertical target marker line that crosses the bar
 export default (props) => (_jsxs(DefaultIcon, { ...props, children: [_jsx("rect", { x: "2", y: "9", width: "6", height: "6", fill: "currentColor", opacity: "0.25" }), _jsx("rect", { x: "8", y: "9", width: "6", height: "6", fill: "currentColor", opacity: "0.45" }), _jsx("rect", { x: "14", y: "9", width: "8", height: "6", fill: "currentColor", opacity: "0.65" }), _jsx("rect", { x: "2", y: "11", width: "13", height: "2", fill: "currentColor" }), _jsx("rect", { x: "17", y: "8", width: "1.5", height: "8", fill: "currentColor" })] }));

package/src/components/icons/icon-style.js

@@ -1,7 +1,3 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import DefaultIcon from './DefaultIcon';
-// Icon Style icon - represents the "key → icon" mapping idea:
-//  - left: a stylised value (rectangle suggesting a cell)
-//  - middle: a small arrow / connector
-//  - right: a glyph (filled diamond) standing in for the matched icon
 export default (props) => (_jsxs(DefaultIcon, { ...props, children: [_jsx("rect", { x: "2", y: "8", width: "6", height: "8", rx: "1.2", ry: "1.2", fill: "none", stroke: "currentColor", strokeWidth: "1.5" }), _jsx("path", { d: "M9.5 12 L13.5 12", stroke: "currentColor", strokeWidth: "1.5", strokeLinecap: "round" }), _jsx("polygon", { points: "13.5,10 13.5,14 15,12", fill: "currentColor" }), _jsx("path", { d: "M17 6 v12", stroke: "currentColor", strokeWidth: "1.5", strokeLinecap: "round" }), _jsx("path", { d: "M17 7 h4 l-1 2 l1 2 h-4 z", fill: "currentColor" })] }));

package/src/components/icons/index.js

@@ -177,17 +177,17 @@ import StyledGrid from './styled-grid';
 import { isAdaptableCustomIcon, isAdaptableSystemIcon } from '../Icon';
 import { AdaptableLogger } from '../../agGrid/AdaptableLogger';
 export const allIcons = {
-    search: Query, // original icon is search
-    grid: Layout, // original icon is apps,
-    cells: CellSummary, // original icon is view-module
+    search: Query,
+    grid: Layout,
+    cells: CellSummary,
     columns: ColumnChooser,
     copy: Copy,
-    'calculated-column': CalculatedColumn, // original icon is phone-setup
-    laptop: Application, // original icon is   laptop,
+    'calculated-column': CalculatedColumn,
+    laptop: Application,
     alert: Alert,
     building: Analysis,
-    'edit-table': BulkUpdate, // original icon is update,
-    chart: Chart, // original icon is bar-chart
+    'edit-table': BulkUpdate,
+    chart: Chart,
     'column-add': ColumnAdd,
     person: Person,
     quote: Quote,
@@ -199,36 +199,35 @@ export const allIcons = {
     json: DataObject,
     export: Export,
     broadcast: Campaign,
-    theme: Theme, // original icon is color-fill
+    theme: Theme,
     brain: Instrument,
     calendar: Calendar,
     chat: Chat,
-    'filter-list': ColumnFilter, // original icon is filter-list
+    'filter-list': ColumnFilter,
     'column-outline': ColumnOutline,
     'grid-filter': GridFilter,
     'grid-info': GridInfo,
     filter: Filter,
     'filter-off': FilterOff,
-    'search-table': QuickSearch, // original icon is page-view
-    lightning: FlashingCell, // flashOn,
-    'color-palette': FormatColumn, /// color-lens
-    'export-data': ExportData, /// color-lens
-    'horizontal-lines': FreetextColumn, //  short-text
-    'percent-tag': PercentBar, // linear-scale
-    'spark-line': Sparkline, //    show-chart
-    resume: Resume, //    unsuspend
+    'search-table': QuickSearch,
+    lightning: FlashingCell,
+    'color-palette': FormatColumn,
+    'export-data': ExportData,
+    'horizontal-lines': FreetextColumn,
+    'percent-tag': PercentBar,
+    'spark-line': Sparkline,
+    resume: Resume,
     csv: CSV,
     'pie-chart': PieChart,
-    'add-circle': PlusMinus, //add-circle
-    reminder: Reminder, // alarm_on
-    schedule: Schedule, // schedule
-    link: Shortcut, // link
-    assignment: StateManagement, //   assignment,
+    'add-circle': PlusMinus,
+    reminder: Reminder,
+    schedule: Schedule,
+    link: Shortcut,
+    assignment: StateManagement,
     clipboard: ToolPanel,
-    'traffic-lights': SystemStatus, // was traffic
+    'traffic-lights': SystemStatus,
     add: AddRow,
-    downloaded: UpdatedRow, //   system-update
-    // others
+    downloaded: UpdatedRow,
     edit: edit,
     undo: Undo,
     clone: Clone,
@@ -278,7 +277,7 @@ export const allIcons = {
     'folder-open': FolderOpen,
     'folder-shared': FolderShared,
     folder: Folder,
-    newpage: NewPage, //note_add
+    newpage: NewPage,
     drag: Drag,
     'date-range': DateRange,
     settings: Settings,
@@ -335,7 +334,7 @@ export const allIcons = {
     'case-lower': Lowercase,
     'case-upper': Uppercase,
     'case-sentence': SentenceCase,
-    flag: CellValidation, // original icon is flag,
+    flag: CellValidation,
     fdc3: FDC3,
     mail: Mail,
     dollar: Dollar,
@@ -368,8 +367,6 @@ export const Icon = ({ name, style, className, tabIndex, ...props }) => {
 export const NaturallySizedIcon = (props) => {
     return _jsx(Icon, { ...props, size: '1.5em', style: { marginTop: '-0.125em', ...props.style } });
 };
-// SVG attributes that React accepts in camelCase but emits as-is in markup
-// (everything else gets converted to kebab-case at serialise time).
 const SVG_KEEP_AS_IS = new Set([
     'viewBox',
     'preserveAspectRatio',
@@ -389,14 +386,12 @@ const SVG_KEEP_AS_IS = new Set([
     'baseProfile',
     'repeatCount',
 ]);
-// React-prop -> HTML/SVG attribute name mapping
 const REACT_TO_HTML_ATTR = {
     className: 'class',
     htmlFor: 'for',
     tabIndex: 'tabindex',
     acceptCharset: 'accept-charset',
 };
-// CSS properties whose numeric values do NOT need a "px" suffix
 const UNITLESS_CSS_PROPS = new Set([
     'animationIterationCount',
     'borderImageOutset',
@@ -432,7 +427,6 @@ const UNITLESS_CSS_PROPS = new Set([
     'widows',
     'zIndex',
     'zoom',
-    // SVG
     'fillOpacity',
     'floodOpacity',
     'stopOpacity',
@@ -486,7 +480,6 @@ const propToAttrName = (key) => {
         return key;
     if (SVG_KEEP_AS_IS.has(key))
         return key;
-    // Camel-case SVG attributes (fillRule, clipPath, strokeWidth, ...) are emitted kebab-case.
     return /[A-Z]/.test(key) ? camelToKebab(key) : key;
 };
 const reactElementToString = (node) => {
@@ -504,9 +497,6 @@ const reactElementToString = (node) => {
     if (type === React.Fragment) {
         return reactElementToString(props.children);
     }
-    // Function component: invoke it and recurse on the returned element.
-    // (All AdapTable system-icon components and the <Icon> wrapper are pure
-    // and use no hooks, so this is safe.)
     if (typeof type === 'function') {
         return reactElementToString(type(props));
     }

package/src/components/icons/range-bar.js

@@ -1,8 +1,3 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import DefaultIcon from './DefaultIcon';
-// Range Bar icon - a horizontal layout consisting of:
-//  - end-caps at the min and max bounds
-//  - a thin track running between them representing the [Min, Max] range
-//  - a diamond-shaped value marker positioned along the track
-//  - a smaller secondary mark hinting at an optional reference value
 export default (props) => (_jsxs(DefaultIcon, { ...props, children: [_jsx("rect", { x: "2", y: "8", width: "1.5", height: "8", fill: "currentColor" }), _jsx("rect", { x: "20.5", y: "8", width: "1.5", height: "8", fill: "currentColor" }), _jsx("rect", { x: "3.5", y: "11", width: "17", height: "2", fill: "currentColor", opacity: "0.35" }), _jsx("rect", { x: "8", y: "9", width: "1", height: "6", fill: "currentColor", opacity: "0.6" }), _jsx("polygon", { points: "14,7.5 17,12 14,16.5 11,12", fill: "currentColor" })] }));

package/src/components/icons/star.js

@@ -1,4 +1,3 @@
 import { jsx as _jsx } from "react/jsx-runtime";
 import DefaultIcon from './DefaultIcon';
-// 5-pointed star, the canonical Rating column glyph.
 export default (props) => (_jsx(DefaultIcon, { ...props, children: _jsx("path", { d: "M12 2 L14.9 8.6 L22 9.3 L16.7 14.1 L18.2 21 L12 17.4 L5.8 21 L7.3 14.1 L2 9.3 L9.1 8.6 Z" }) }));

package/src/components/twUtils.js

@@ -1,19 +1,10 @@
 export const targetOwn = {
-    focusOutline: 
-    // 'twa:outline-0 twa:focus:outline-2 twa:focus:outline-offset-1 twa:focus:outline-accent',
-    // 'twa:outline-0 twa:focus:shadow-(--ab-focus__box-shadow)',
-    'twa:outline-none twa:focus-visible:border-ring twa:focus-visible:ring-3 twa:focus-visible:ring-ring/50',
-    focusWithinOutline: 
-    // 'twa:outline-0 twa:focus-within:outline-2 twa:focus-within:outline-offset-1 twa:focus-within:outline-accent',
-    // 'twa:outline-0 twa:focus-within:shadow-(--ab-focus__box-shadow)',
-    'twa:outline-none twa:focus-within:border-ring twa:focus-within:ring-3 twa:focus-within:ring-ring/50',
+    focusOutline: 'twa:outline-none twa:focus-visible:border-ring twa:focus-visible:ring-3 twa:focus-visible:ring-ring/50',
+    focusWithinOutline: 'twa:outline-none twa:focus-within:border-ring twa:focus-within:ring-3 twa:focus-within:ring-ring/50',
 };
 export const targetPeer = {
     focusVisibleOutline: 'twa:peer-focus-visible:border-ring twa:peer-focus-visible:ring-3 twa:peer-focus-visible:ring-ring/50',
 };
 export const targetChildren = {
-    focusWithinOutline: 
-    // 'twa:*:outline-0 twa:*:focus-within:outline-2 twa:*:focus-within:outline-offset-1 twa:*:focus-within:outline-accent',
-    // 'twa:*:outline-0 twa:*:focus-within:shadow-(--ab-focus__box-shadow)',
-    'twa:*:outline-none twa:*:focus-within:border-ring twa:*:focus-within:ring-3 twa:*:focus-within:ring-ring/50',
+    focusWithinOutline: 'twa:*:outline-none twa:*:focus-within:border-ring twa:*:focus-within:ring-3 twa:*:focus-within:ring-ring/50',
 };

package/src/components/ui/calendar.js

@@ -41,11 +41,7 @@ function Calendar({ className, classNames, showOutsideDays = true, captionLayout
             week: cn('twa:mt-2 twa:flex twa:w-full', defaultClassNames.week),
             week_number_header: cn('twa:w-(--cell-size) twa:select-none', defaultClassNames.week_number_header),
             week_number: cn('twa:text-[0.8rem] twa:text-muted-foreground twa:select-none', defaultClassNames.week_number),
-            day: cn('twa:group/day twa:relative twa:aspect-square twa:h-full twa:w-full twa:rounded-standard twa:p-0 twa:text-center twa:select-none twa:[&:last-child[data-selected=true]_button]:rounded-r-standard', 
-            // props.showWeekNumber
-            //   ? "twa:[&:nth-child(2)[data-selected=true]_button]:rounded-l-standard"
-            //   : "twa:[&:first-child[data-selected=true]_button]:rounded-l-standard",
-            defaultClassNames.day),
+            day: cn('twa:group/day twa:relative twa:aspect-square twa:h-full twa:w-full twa:rounded-standard twa:p-0 twa:text-center twa:select-none twa:[&:last-child[data-selected=true]_button]:rounded-r-standard', defaultClassNames.day),
             range_start: cn('twa:relative twa:isolate twa:z-0 twa:rounded-l-standard twa:bg-muted twa:after:absolute twa:after:inset-y-0 twa:after:right-0 twa:after:w-4 twa:after:bg-muted', defaultClassNames.range_start),
             range_middle: cn('twa:rounded-none', defaultClassNames.range_middle),
             range_end: cn('twa:relative twa:isolate twa:z-0 twa:rounded-r-standard twa:bg-muted twa:after:absolute twa:after:inset-y-0 twa:after:left-0 twa:after:w-4 twa:after:bg-muted', defaultClassNames.range_end),
@@ -73,8 +69,6 @@ function CalendarDayButton({ className, day, modifiers, ...props }) {
     React.useEffect(() => {
         if (modifiers.focused) {
             const active = document.activeElement;
-            // Only grab focus when no other interactive element (nav button, dropdown)
-            // is focused, or when keyboard-navigating between day cells.
             const shouldFocus = !active ||
                 active === document.body ||
                 active === document.documentElement ||
@@ -88,8 +82,6 @@ function CalendarDayButton({ className, day, modifiers, ...props }) {
     React.useEffect(() => {
         if (selected) {
             const active = document.activeElement;
-            // Only grab focus when no other interactive element (nav button, dropdown)
-            // is focused, or when keyboard-navigating between day cells.
             const shouldFocus = !active ||
                 active === document.body ||
                 active === document.documentElement ||

package/src/components/ui/combobox.js

@@ -80,11 +80,7 @@ const ComboboxChips = React.forwardRef(function ComboboxChips({ className, ...pr
 const ComboboxChip = React.forwardRef(function ComboboxChip({ className, children, showRemove = true, onRemove, ...props }, ref) {
     return (_jsxs(ComboboxPrimitive.Chip, { ref: ref, "data-slot": "combobox-chip", className: cn('twa:flex twa:h-[calc(--spacing(5.5))] twa:w-fit twa:items-center twa:justify-center twa:gap-1 twa:rounded-input twa:bg-muted twa:px-1.5 twa:text-xs twa:font-medium twa:whitespace-nowrap twa:text-input-foreground twa:has-disabled:pointer-events-none twa:has-disabled:cursor-not-allowed twa:has-disabled:opacity-50 twa:has-data-[slot=combobox-chip-remove]:pr-0', className), ...props, children: [children, showRemove &&
                 (onRemove ? (_jsx(Button, { type: "button", variant: "ghost", size: "icon-xs", tabIndex: -1, "data-slot": "combobox-chip-remove", className: "twa:-ml-1 twa:opacity-50 twa:hover:opacity-100", onMouseDown: (e) => {
-                        // Don't let base-ui's Chip onMouseDown steal focus back to the
-                        // input before the click fires.
                         e.preventDefault();
-                        // and also stop the propagation, so the mousedown wont
-                        // trigger the combobox to open
                         e.stopPropagation();
                     }, onClick: (e) => {
                         e.stopPropagation();

package/src/components/utils/captureTabNavigation/getFocusableChildren.js

@@ -9,6 +9,5 @@ const selector = [
 ].join(', ');
 export default (node) => {
     const children = [...node.querySelectorAll(selector)];
-    // ensure they are all in the dom
     return children.filter((child) => child.offsetParent);
 };

package/src/components/utils/useDraggable.js

@@ -19,10 +19,6 @@ export default function useDraggable({ onMove, onDrop, getBoundingRect = () => d
     }, []);
     useEffect(() => {
         if (handleSelector) {
-            /**
-             * Element may not be yet attached.
-             * This gives time react to render the node handle.
-             */
             setTimeout(() => {
                 const node = targetRef?.current?.querySelector(handleSelector);
                 node && handleRefCallback(node);

package/src/components/utils/useProperty.js

@@ -2,24 +2,6 @@ import { useState, useRef } from 'react';
 const toUpperFirst = (str) => `${str.charAt(0).toUpperCase()}${str.substring(1)}`;
 const isControlled = (value) => value !== undefined;
 const emptyFn = () => { };
-/**
- *
- * useProperty can be used to easily manage a given prop - especially when you want to support both the controlled and the uncontrolled behviour.
- *
- * @param props - the props of the component
- * @param propName - the name of the property to manage
- * @param defaultValue? - a default value for the prop - especially useful when the prop you're targeting via `propName` is uncontrolled
- * @param config.onChange? - defaults to `props.on${PropName}Change` - useful when you want to be notified of when the property is changed - via the returned setter.
- * config.onChange is called for both controlled and uncontrolled prop.
- *
- * If the prop is uncontrolled, when the returned propSetter is called, the useProperty hook will update the state internally - you can be notified of this via config.onChange.
- *
- * If you don't have config.onChange specified, the useProperty hook will try to call `props.on${PropName}Change` if it's defined.
- *
- * So even for a controlled prop, if you respect this convention and have this pair `${propName}` + `on${PropName}Change` - useProperty will do all the work and call the "event" handler prop for you.
- *
- * Returns [propValue, propSetterFunction] - the value of the prop and a setter function
- */
 const useProperty = (props, propName, defaultValue, config) => {
     const PropName = toUpperFirst(propName);
     const defaultValueFromProps = props[`default${PropName}`];

package/src/devTools/index.js

@@ -10,7 +10,6 @@ const AVAILABLE_COLORS = [
     'tertiary',
     'tertiary-light',
     'tertiary-dark',
-    // 'error',
 ];
 const colorMap = new Map();
 function getColor(identifier) {

package/src/env.js

@@ -1,5 +1,5 @@
 export default {
     NEXT_PUBLIC_INFINITE_TABLE_LICENSE_KEY: "StartDate=2021-06-29|EndDate=2030-01-01|Owner=Adaptable|Type=distribution|TS=1624971462479|C=137829811,1004007071,2756196225,1839832928,3994409405,636616862" || '',
-    PUBLISH_TIMESTAMP: 1782415229232 || Date.now(),
-    VERSION: "23.0.4" || '--current-version--',
+    PUBLISH_TIMESTAMP: 1782469286273 || Date.now(),
+    VERSION: "23.0.5-canary.0" || '--current-version--',
 };

package/src/layout-manager/src/LMEmitter.js

@@ -7,10 +7,6 @@ export class LMEmitter extends LMLogger {
     constructor(options) {
         super(options);
     }
-    /**
-     * For each supported event, we have a Set that holds all the callbacks that should be executed
-     * when the event is triggered.
-     */
     eventSets = {
         gridLayoutChanged: new Set(),
         columnDefsChanged: new Set(),
@@ -40,7 +36,6 @@ export class LMEmitter extends LMLogger {
             if (this.destroyed) {
                 return;
             }
-            // @ts-ignore
             callback(...args);
         });
     }

package/src/layout-manager/src/destructurePivotColumnId.js

@@ -1,22 +1,14 @@
-// supports only pivotTotalColumns (aggregation total) and pivotGroupTotal
-// AG Grid builds the ID as: `pivot_${pivotCols.join('-')}_${pivotKeys.join('-')}_${measureColumnId}`
-// see https://github.com/ag-grid/ag-grid/blob/e0cfe533b55b75cdc148cdfb1a4e977731dc0712/packages/ag-grid-enterprise/src/pivot/pivotColDefService.ts#L454C16-L454C88
 export function destructurePivotColumnId(colDef, currentModel, logWarning) {
     const { colId } = colDef;
-    // Basic validation
     if (!colId?.startsWith('pivot_')) {
         logWarning(`Column id must start with 'pivot_': ${colId}`);
         return '!unknown!';
     }
     const hasSpecialChars = (arr) => arr.some((str) => str.includes('-') || str.includes('_'));
-    // AG Grid uses _ and - as delimiters
-    // if we have these special characters in the column IDs, we need to parse them differently
     const colIdsOrKeysContainSpecialChars = hasSpecialChars(currentModel.pivotColIds) ||
         hasSpecialChars(currentModel.aggColIds) ||
         hasSpecialChars(colDef.pivotKeys || []);
-    // Fast path - no special characters
     if (!colIdsOrKeysContainSpecialChars) {
-        // Split by underscore to get 4 parts
         const parts = colId.split('_');
         if (parts.length !== 4) {
             logWarning(`Unsupported format of pivot total column id: ${colId}`);
@@ -37,17 +29,11 @@ export function destructurePivotColumnId(colDef, currentModel, logWarning) {
             aggregationColumnId: aggregationColumnId !== '' ? aggregationColumnId : undefined,
         };
     }
-    // the complex path, where we have to handle the special characters
-    // Remove 'pivot_' prefix
     const withoutPrefix = colId.slice(6);
-    // Check if it's a pivot group total
     const isPivotGroupTotal = withoutPrefix.endsWith('_');
-    // Get pivot keys from colDef (more reliable than string parsing)
     const pivotKeys = colDef.pivotKeys || [];
     if (isPivotGroupTotal) {
-        // Remove trailing underscore for pivot group totals
         const content = withoutPrefix.slice(0, -1);
-        // The remaining content should be the pivot columns
         const pivotColumnIds = currentModel.pivotColIds.filter((id) => content.includes(id));
         if (!pivotColumnIds.length) {
             logWarning(`Could not identify pivot columns in: ${content}`);
@@ -59,10 +45,6 @@ export function destructurePivotColumnId(colDef, currentModel, logWarning) {
             pivotColumnId: pivotColumnIds[pivotKeys.length - 1],
         };
     }
-    // For regular pivot columns, work backwards to find aggregation column.
-    // Check if the full string ends with _${aggColId} rather than splitting by _,
-    // since agg column IDs may themselves contain underscores (e.g. github_stars).
-    // Sort by length descending so the longest match wins.
     const aggregationColumnId = [...currentModel.aggColIds]
         .sort((a, b) => b.length - a.length)
         .find((aggId) => withoutPrefix.endsWith('_' + aggId));
@@ -70,7 +52,6 @@ export function destructurePivotColumnId(colDef, currentModel, logWarning) {
         logWarning(`Could not identify aggregation column in: ${withoutPrefix}`);
         return '!unknown!';
     }
-    // Remove aggregation part and get pivot columns
     const withoutAgg = withoutPrefix.slice(0, -(aggregationColumnId.length + 1));
     const pivotColumnIds = currentModel.pivotColIds.filter((id) => withoutAgg.includes(id));
     if (!pivotColumnIds.length) {