acttrader-charts 1.4.0-beta.12 → 1.4.0-beta.13

package/dist/{MACD-B8LTAGUU.d.cts → MACD-ico-au5d.d.cts}

@@ -323,10 +323,6 @@ interface ChartSettingsTemplate {
 }
 /** Mints a stable, collision-free id for a settings-template name. */
 declare function makeSettingsTemplateId(name: string, existingIds?: readonly string[]): string;
-/**
- * Deep-copies a settings snapshot one level past `canvasColors`, so a saved
- * template can never be mutated by later edits to the live chart (or vice versa).
- */
 declare function cloneSettings(s: ChartSettingsSnapshot): ChartSettingsSnapshot;
 /** Builds a template from a settings snapshot. */
 declare function captureSettingsTemplate(name: string, settings: ChartSettingsSnapshot, opts?: {

package/dist/{MACD-B8LTAGUU.d.ts → MACD-ico-au5d.d.ts}

@@ -323,10 +323,6 @@ interface ChartSettingsTemplate {
 }
 /** Mints a stable, collision-free id for a settings-template name. */
 declare function makeSettingsTemplateId(name: string, existingIds?: readonly string[]): string;
-/**
- * Deep-copies a settings snapshot one level past `canvasColors`, so a saved
- * template can never be mutated by later edits to the live chart (or vice versa).
- */
 declare function cloneSettings(s: ChartSettingsSnapshot): ChartSettingsSnapshot;
 /** Builds a template from a settings snapshot. */
 declare function captureSettingsTemplate(name: string, settings: ChartSettingsSnapshot, opts?: {

package/dist/index.cjs

@@ -27089,6 +27089,123 @@ var IndicatorSettingsDialog = class {
   }
 };
 
+// src/core/templateStore.ts
+function slugify(name) {
+  return name.trim().toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "");
+}
+function mintId(name, existingIds = [], fallback = "template") {
+  const base = slugify(name) || fallback;
+  if (!existingIds.includes(base)) return base;
+  let n = 2;
+  while (existingIds.includes(`${base}-${n}`)) n++;
+  return `${base}-${n}`;
+}
+function upsertById(list, entry) {
+  const idx = list.findIndex((e) => e.id === entry.id);
+  if (idx === -1) return [...list, entry];
+  const next = [...list];
+  next[idx] = entry;
+  return next;
+}
+function removeById(list, id) {
+  return list.filter((e) => e.id !== id);
+}
+function findByIdOrName(list, idOrName) {
+  const byId = list.find((e) => e.id === idOrName);
+  if (byId) return byId;
+  const slug = slugify(idOrName);
+  return list.find((e) => slugify(e.name) === slug);
+}
+
+// src/core/settingsTemplates.ts
+function makeSettingsTemplateId(name, existingIds = []) {
+  return mintId(name, existingIds, "settings");
+}
+function cloneCanvasColors(c) {
+  if (!c) return c;
+  return {
+    ...c,
+    ...c.dark ? { dark: { ...c.dark } } : {},
+    ...c.light ? { light: { ...c.light } } : {}
+  };
+}
+function cloneSettings(s) {
+  return {
+    tradeDisplayFilter: s.tradeDisplayFilter,
+    positionRenderStyle: s.positionRenderStyle,
+    theme: s.theme,
+    showTradeLevelsAlways: s.showTradeLevelsAlways,
+    showPriceAxisCountdown: s.showPriceAxisCountdown,
+    ...s.canvasColors ? {
+      canvasColors: {
+        ...s.canvasColors.dark ? { dark: { ...s.canvasColors.dark } } : {},
+        ...s.canvasColors.light ? { light: { ...s.canvasColors.light } } : {}
+      }
+    } : {},
+    // Each group is one level deep, so a spread copy is a full copy.
+    ...s.statusLine ? { statusLine: { ...s.statusLine } } : {},
+    ...s.scales ? { scales: { ...s.scales } } : {},
+    ...s.canvas ? { canvas: { ...s.canvas } } : {},
+    ...s.barColorSource ? { barColorSource: s.barColorSource } : {}
+  };
+}
+function captureSettingsTemplate(name, settings, opts = {}) {
+  return {
+    id: opts.id ?? makeSettingsTemplateId(name, opts.existingIds ?? []),
+    name: name.trim(),
+    settings: cloneSettings(settings),
+    ...opts.savedAt !== void 0 ? { savedAt: opts.savedAt } : {}
+  };
+}
+function upsertSettingsTemplate(list, template) {
+  return upsertById(list, template);
+}
+function removeSettingsTemplate(list, id) {
+  return removeById(list, id);
+}
+function findSettingsTemplate(list, idOrName) {
+  return findByIdOrName(list, idOrName);
+}
+function isGroup(v) {
+  return typeof v === "object" && v !== null && !Array.isArray(v);
+}
+function sanitizeSettingsTemplates(input) {
+  if (!Array.isArray(input)) return [];
+  const out = [];
+  for (const raw of input) {
+    if (typeof raw !== "object" || raw === null) continue;
+    const t = raw;
+    if (typeof t.name !== "string" || t.name.trim() === "") continue;
+    if (typeof t.settings !== "object" || t.settings === null) continue;
+    const s = t.settings;
+    out.push({
+      id: typeof t.id === "string" && t.id !== "" ? t.id : makeSettingsTemplateId(
+        t.name,
+        out.map((x) => x.id)
+      ),
+      name: t.name.trim(),
+      settings: {
+        tradeDisplayFilter: s.tradeDisplayFilter ?? "all",
+        positionRenderStyle: s.positionRenderStyle ?? "line",
+        theme: s.theme === "light" ? "light" : "dark",
+        showTradeLevelsAlways: s.showTradeLevelsAlways === true,
+        showPriceAxisCountdown: s.showPriceAxisCountdown === true,
+        ...typeof s.canvasColors === "object" && s.canvasColors !== null ? { canvasColors: s.canvasColors } : {},
+        // Groups are carried through when present and dropped when not.
+        // Absent is meaningful: it means "leave the chart as it is",
+        // which is what an old template saved before these existed has
+        // to mean if applying it is not to rearrange the chart.
+        ...isGroup(s.statusLine) ? { statusLine: s.statusLine } : {},
+        ...isGroup(s.scales) ? { scales: s.scales } : {},
+        ...isGroup(s.canvas) ? { canvas: s.canvas } : {},
+        ...s.barColorSource === "previousClose" || s.barColorSource === "open" ? { barColorSource: s.barColorSource } : {}
+      },
+      ...typeof t.savedAt === "number" ? { savedAt: t.savedAt } : {}
+    });
+  }
+  return out;
+}
+
 // src/ui/ChartSettingsDialog.ts
 var TAB_LABELS = {
   appearance: "Appearance",
@@ -27198,7 +27315,7 @@ var ChartSettingsDialog = class {
   }
   show(current, onChange, onReset) {
     const wasVisible = this.isVisible;
-    this.current = { ...current };
+    this.current = { ...current, canvasColors: cloneCanvasColors(current.canvasColors) };
     this.onChange = onChange;
     this.onReset = onReset ?? null;
     this.renderContent();
@@ -27258,7 +27375,7 @@ var ChartSettingsDialog = class {
   }
   // ── Private – emit ────────────────────────────────────────────────────────
   emit() {
-    this.onChange?.({ ...this.current });
+    this.onChange?.({ ...this.current, canvasColors: cloneCanvasColors(this.current.canvasColors) });
   }
   // ── Private – panel theme ─────────────────────────────────────────────────
   applyPanelTheme() {
@@ -28038,9 +28155,11 @@ var ChartSettingsDialog = class {
       }
     ];
     const makeSwatch = (def) => this.makeColorSwatch(current[def.field] ?? def.fallback, (hex) => {
-      if (!this.current.canvasColors) this.current.canvasColors = {};
-      if (!this.current.canvasColors[themeKey]) this.current.canvasColors[themeKey] = {};
-      this.current.canvasColors[themeKey][def.field] = hex;
+      const cc = this.current.canvasColors ?? {};
+      this.current.canvasColors = {
+        ...cc,
+        [themeKey]: { ...cc[themeKey] ?? {}, [def.field]: hex }
+      };
       this.emit();
     });
     for (const { label, s1, s2 } of rows) {
@@ -32039,34 +32158,6 @@ function formatMeasureDate(time, timeframe, timezone) {
   return `${date} ${pick(cp, "hour")}:${pick(cp, "minute")}`;
 }
 
-// src/core/templateStore.ts
-function slugify(name) {
-  return name.trim().toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "");
-}
-function mintId(name, existingIds = [], fallback = "template") {
-  const base = slugify(name) || fallback;
-  if (!existingIds.includes(base)) return base;
-  let n = 2;
-  while (existingIds.includes(`${base}-${n}`)) n++;
-  return `${base}-${n}`;
-}
-function upsertById(list, entry) {
-  const idx = list.findIndex((e) => e.id === entry.id);
-  if (idx === -1) return [...list, entry];
-  const next = [...list];
-  next[idx] = entry;
-  return next;
-}
-function removeById(list, id) {
-  return list.filter((e) => e.id !== id);
-}
-function findByIdOrName(list, idOrName) {
-  const byId = list.find((e) => e.id === idOrName);
-  if (byId) return byId;
-  const slug = slugify(idOrName);
-  return list.find((e) => slugify(e.name) === slug);
-}
-
 // src/core/savedLayouts.ts
 function makeLayoutId(name, existingIds = []) {
   return mintId(name, existingIds, "layout");
@@ -32121,87 +32212,6 @@ function sanitizeLayouts(input) {
   return out;
 }
 
-// src/core/settingsTemplates.ts
-function makeSettingsTemplateId(name, existingIds = []) {
-  return mintId(name, existingIds, "settings");
-}
-function cloneSettings(s) {
-  return {
-    tradeDisplayFilter: s.tradeDisplayFilter,
-    positionRenderStyle: s.positionRenderStyle,
-    theme: s.theme,
-    showTradeLevelsAlways: s.showTradeLevelsAlways,
-    showPriceAxisCountdown: s.showPriceAxisCountdown,
-    ...s.canvasColors ? {
-      canvasColors: {
-        ...s.canvasColors.dark ? { dark: { ...s.canvasColors.dark } } : {},
-        ...s.canvasColors.light ? { light: { ...s.canvasColors.light } } : {}
-      }
-    } : {},
-    // Each group is one level deep, so a spread copy is a full copy.
-    ...s.statusLine ? { statusLine: { ...s.statusLine } } : {},
-    ...s.scales ? { scales: { ...s.scales } } : {},
-    ...s.canvas ? { canvas: { ...s.canvas } } : {},
-    ...s.barColorSource ? { barColorSource: s.barColorSource } : {}
-  };
-}
-function captureSettingsTemplate(name, settings, opts = {}) {
-  return {
-    id: opts.id ?? makeSettingsTemplateId(name, opts.existingIds ?? []),
-    name: name.trim(),
-    settings: cloneSettings(settings),
-    ...opts.savedAt !== void 0 ? { savedAt: opts.savedAt } : {}
-  };
-}
-function upsertSettingsTemplate(list, template) {
-  return upsertById(list, template);
-}
-function removeSettingsTemplate(list, id) {
-  return removeById(list, id);
-}
-function findSettingsTemplate(list, idOrName) {
-  return findByIdOrName(list, idOrName);
-}
-function isGroup(v) {
-  return typeof v === "object" && v !== null && !Array.isArray(v);
-}
-function sanitizeSettingsTemplates(input) {
-  if (!Array.isArray(input)) return [];
-  const out = [];
-  for (const raw of input) {
-    if (typeof raw !== "object" || raw === null) continue;
-    const t = raw;
-    if (typeof t.name !== "string" || t.name.trim() === "") continue;
-    if (typeof t.settings !== "object" || t.settings === null) continue;
-    const s = t.settings;
-    out.push({
-      id: typeof t.id === "string" && t.id !== "" ? t.id : makeSettingsTemplateId(
-        t.name,
-        out.map((x) => x.id)
-      ),
-      name: t.name.trim(),
-      settings: {
-        tradeDisplayFilter: s.tradeDisplayFilter ?? "all",
-        positionRenderStyle: s.positionRenderStyle ?? "line",
-        theme: s.theme === "light" ? "light" : "dark",
-        showTradeLevelsAlways: s.showTradeLevelsAlways === true,
-        showPriceAxisCountdown: s.showPriceAxisCountdown === true,
-        ...typeof s.canvasColors === "object" && s.canvasColors !== null ? { canvasColors: s.canvasColors } : {},
-        // Groups are carried through when present and dropped when not.
-        // Absent is meaningful: it means "leave the chart as it is",
-        // which is what an old template saved before these existed has
-        // to mean if applying it is not to rearrange the chart.
-        ...isGroup(s.statusLine) ? { statusLine: s.statusLine } : {},
-        ...isGroup(s.scales) ? { scales: s.scales } : {},
-        ...isGroup(s.canvas) ? { canvas: s.canvas } : {},
-        ...s.barColorSource === "previousClose" || s.barColorSource === "open" ? { barColorSource: s.barColorSource } : {}
-      },
-      ...typeof t.savedAt === "number" ? { savedAt: t.savedAt } : {}
-    });
-  }
-  return out;
-}
-
 // src/core/indicatorTemplates.ts
 function makeTemplateId(name, existingIds = []) {
   return mintId(name, existingIds);
@@ -39352,7 +39362,7 @@ var _ChartEngine = class _ChartEngine {
       this.tradeDisplayFilter = config.tradeDisplayFilter;
     }
     if (config.canvasColors) {
-      this.settingsCanvasColors = config.canvasColors;
+      this.settingsCanvasColors = cloneCanvasColors(config.canvasColors);
     }
     this.recomputeRenderStyle();
     this.showBidAskLines = config.showBidAskLines ?? false;
@@ -41877,7 +41887,7 @@ var _ChartEngine = class _ChartEngine {
    * like the dialog's own picks, and `stateChange` fires.
    */
   setCanvasColors(colors) {
-    this.settingsCanvasColors = colors ?? {};
+    this.settingsCanvasColors = cloneCanvasColors(colors ?? {});
     this.setTheme(this.themeName);
     return this;
   }
@@ -42292,7 +42302,7 @@ var _ChartEngine = class _ChartEngine {
       })),
       positionRenderStyle: this.baseRenderStyle(),
       tradeDisplayFilter: this.tradeDisplayFilter,
-      canvasColors: Object.keys(this.settingsCanvasColors).length > 0 ? { ...this.settingsCanvasColors } : void 0,
+      canvasColors: Object.keys(this.settingsCanvasColors).length > 0 ? cloneCanvasColors(this.settingsCanvasColors) : void 0,
       tfcEnabled: this.tfcActive,
       viewport: this.getViewportTimestamps(),
       priceScale: this.priceScaleFactor !== 1 || this.pricePanOffset !== 0 ? { factor: this.priceScaleFactor, panOffset: this.pricePanOffset } : void 0,
@@ -42395,7 +42405,7 @@ var _ChartEngine = class _ChartEngine {
         this.tradeDisplayFilter = state.tradeDisplayFilter;
       }
       if (state.canvasColors) {
-        this.settingsCanvasColors = state.canvasColors;
+        this.settingsCanvasColors = cloneCanvasColors(state.canvasColors);
       }
       if (state.tfcEnabled !== void 0 && this.tfcEnabled) {
         this.tfcActive = state.tfcEnabled;
@@ -44559,7 +44569,7 @@ var _ChartEngine = class _ChartEngine {
       theme: this.themeName,
       showTradeLevelsAlways: this.showTradeLevelsAlways,
       showPriceAxisCountdown: this.showPriceAxisCountdown,
-      canvasColors: { ...this.settingsCanvasColors },
+      canvasColors: cloneCanvasColors(this.settingsCanvasColors),
       // The resolved sets, not the sparse ones. A template is a thing
       // the user means to reproduce exactly; saving the gaps would make
       // it reproduce whatever the *next* chart's defaults happen to be.
@@ -44583,7 +44593,7 @@ var _ChartEngine = class _ChartEngine {
     }
     this.positionRenderStyle = settings.positionRenderStyle;
     if (settings.canvasColors) {
-      this.settingsCanvasColors = settings.canvasColors;
+      this.settingsCanvasColors = cloneCanvasColors(settings.canvasColors);
     }
     if (this.showTradeLevelsAlways !== settings.showTradeLevelsAlways) {
       this.showTradeLevelsAlways = settings.showTradeLevelsAlways;