@trebco/treb 29.5.2 → 29.5.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trebco/treb",
-  "version": "29.5.2",
+  "version": "29.5.4",
   "license": "LGPL-3.0-or-later",
   "homepage": "https://treb.app",
   "repository": {

package/treb-data-model/src/serialize_options.ts

@@ -74,4 +74,9 @@ export interface SerializeOptions {
    */
   export_functions?: boolean;
 
+  /**
+   * @internal
+   */
+  apply_row_pattern?: boolean;
+
 }

package/treb-data-model/src/sheet.ts

@@ -2519,8 +2519,6 @@ export class Sheet {
 
     // same here (note broken naming)
     const sheet_style = JSON.parse(JSON.stringify(this.sheet_style));
-    // const row_style = JSON.parse(JSON.stringify(this.row_styles));
-    // const column_style = JSON.parse(JSON.stringify(this.column_styles));
     const row_pattern = JSON.parse(JSON.stringify(this.row_pattern));
 
     // row and column styles are Record<number, props> and not arrays.
@@ -2545,13 +2543,36 @@ export class Sheet {
       }
     }
 
-    for (const key of Object.keys(this.row_styles)) {
-      const index = Number(key);
-      const style = this.row_styles[index];
-      if (style) {
-        const reference = StyleToRef(style);
+    if (this.row_pattern && this.row_pattern.length && options.apply_row_pattern) {
+
+      let count = this.rows + 1;
+
+      for (const key of Object.keys(this.row_styles)) {
+        const index = Number(key);
+        if (!isNaN(index) && index >= count) { count = index + 1; }
+      }
+
+      for (let i = 0; i< count; i++) {
+        const pattern = this.row_pattern[i % this.row_pattern.length];
+        const style = this.row_styles[i] || {}; 
+        const composite = Style.Composite([pattern, style]);      
+        const reference = StyleToRef(composite);
+
         if (reference) {
-          row_style[index] = reference;
+          row_style[i] = reference;
+        }
+      }
+      
+    }
+    else {
+      for (const key of Object.keys(this.row_styles)) {
+        const index = Number(key);
+        const style = this.row_styles[index];
+        if (style) {
+          const reference = StyleToRef(style);
+          if (reference) {
+            row_style[index] = reference;
+          }
         }
       }
     }

package/treb-embed/src/embedded-spreadsheet.ts

@@ -2822,6 +2822,7 @@ export class EmbeddedSpreadsheet<USER_DATA_TYPE = unknown> {
           tables: true,
           share_resources: false,
           export_functions: true,
+          apply_row_pattern: true, // if there's a row pattern, set it on rows so they export properly
         });
 
         // why do _we_ put this in, instead of the grid method? 

package/treb-export/src/export.ts

@@ -611,9 +611,13 @@ export class Exporter {
 
     const list: CellStyle[] = [sheet.sheet_style];
 
+    /*
+    // should apply to rows, not cells 
+
     if (sheet.row_pattern && sheet.row_pattern.length) {
       list.push(sheet.row_pattern[row % sheet.row_pattern.length]);
     }
+    */
 
     // is this backwards, vis a vis our rendering? I think it might be...
     // YES: should be row pattern -> row -> column -> cell [corrected]