@atlaskit/editor-tables 2.9.42 → 2.9.44

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # @atlaskit/editor-tables
 
+## 2.9.44
+
+### Patch Changes
+
+- Updated dependencies
+
+## 2.9.43
+
+### Patch Changes
+
+- [`1d245f9d74df5`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/1d245f9d74df5) -
+  [ux] [EDITOR-1682] modify the logic in determineTableHeaderStateFromTableNode to read the 1st and
+  n_th cells instead of 1st and 2nd behind `platform_editor_analyse_table_with_merged_cells`
+- Updated dependencies
+
 ## 2.9.42
 
 ### Patch Changes

package/dist/cjs/utils/analyse-table.js

@@ -4,31 +4,33 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.determineTableHeaderStateFromTableNode = determineTableHeaderStateFromTableNode;
+var _expValEquals = require("@atlaskit/tmp-editor-statsig/exp-val-equals");
 function determineTableHeaderStateFromTableNode(table, tableMap, types) {
-  // We only really need to check the 2nd cell in the row/col if it's a header, since we only support a single full row/col header on the top & left
-  // of a table. We can assume that if the 2nd cell is a header then the entire row/col is a header.
-  // Be carefull though! when checking the 1st cell as it shares its header state with both row/cols.
-  // This means we wont be able to reliably identify header state on tables smaller the 2x2, however we can do best guess.
+  // We only really need to check the nth (last) cell in the row/col if it's a header, since we only support a single full row/col header on the top & left
+  // of a table. We can assume that if the nth (last) cell is also a header then the entire row/col is a header.
+  // Be careful though! when checking the 1st cell as it shares its header state with both row/cols.
+  // This means we won't be able to reliably identify header state on tables smaller than 2x2, however we can make a best guess.
 
   // This is a 3 bit mask;
   // bit: 0 = Identifies if the cell at (0, 0) (row, col - 0-based) is a header cell or not
-  // bit: 1 = Identifies if the cell at (0, 1) is a header cell or not
-  // bit: 2 = Identifies if the cell at (1, 0) is a header cell or not
+  // bit: 1 = Identifies if the cell at (0, n) is a header cell or not
+  // bit: 2 = Identifies if the cell at (n, 0) is a header cell or not
   var mask = 0;
+  var isExperimentEnabled = (0, _expValEquals.expValEquals)('platform_editor_analyse_table_with_merged_cells', 'isEnabled', true);
 
   // At minimum we should have 1 cell in the table.
   var topLeftCell = table.nodeAt(tableMap.map[0]);
   // If this cell is a header that could indicate
   mask |= topLeftCell && topLeftCell.type === types.header_cell ? 1 : 0;
   if (tableMap.width > 1) {
-    var cell = table.nodeAt(tableMap.map[1]);
-    // If the cell at (0, 1) is a header then we set the bit flag to indicate row headers are enabled, otherwise if it's
+    var cell = isExperimentEnabled ? table.nodeAt(tableMap.map[tableMap.width - 1]) : table.nodeAt(tableMap.map[1]);
+    // If the cell at (0, n) is a header then we set the bit flag to indicate row headers are enabled, otherwise if it's
     // not then we will set the col headers enabled flag (and vice versa in the branch below) only if the cell at (0,0)
     // was a header cell.
     mask |= cell && cell.type === types.header_cell ? 2 : 4 * (mask & 1);
   }
   if (tableMap.height > 1) {
-    var _cell = table.nodeAt(tableMap.map[tableMap.width]);
+    var _cell = isExperimentEnabled ? table.nodeAt(tableMap.map[tableMap.width * (tableMap.height - 1)]) : table.nodeAt(tableMap.map[tableMap.width]);
     mask |= _cell && _cell.type === types.header_cell ? 4 : 2 * (mask & 1);
   }
   return {

package/dist/cjs/utils/cells.js

@@ -8,6 +8,8 @@ exports.cellNear = cellNear;
 exports.nextCell = nextCell;
 exports.pointsAtCell = pointsAtCell;
 var _tableMap = require("../table-map");
+// eslint-disable-next-line @typescript-eslint/consistent-type-imports
+
 function pointsAtCell($pos) {
   return $pos.parent.type.spec.tableRole === 'row' && $pos.nodeAfter;
 }

package/dist/es2019/utils/analyse-table.js

@@ -1,28 +1,30 @@
+import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 export function determineTableHeaderStateFromTableNode(table, tableMap, types) {
-  // We only really need to check the 2nd cell in the row/col if it's a header, since we only support a single full row/col header on the top & left
-  // of a table. We can assume that if the 2nd cell is a header then the entire row/col is a header.
-  // Be carefull though! when checking the 1st cell as it shares its header state with both row/cols.
-  // This means we wont be able to reliably identify header state on tables smaller the 2x2, however we can do best guess.
+  // We only really need to check the nth (last) cell in the row/col if it's a header, since we only support a single full row/col header on the top & left
+  // of a table. We can assume that if the nth (last) cell is also a header then the entire row/col is a header.
+  // Be careful though! when checking the 1st cell as it shares its header state with both row/cols.
+  // This means we won't be able to reliably identify header state on tables smaller than 2x2, however we can make a best guess.
 
   // This is a 3 bit mask;
   // bit: 0 = Identifies if the cell at (0, 0) (row, col - 0-based) is a header cell or not
-  // bit: 1 = Identifies if the cell at (0, 1) is a header cell or not
-  // bit: 2 = Identifies if the cell at (1, 0) is a header cell or not
+  // bit: 1 = Identifies if the cell at (0, n) is a header cell or not
+  // bit: 2 = Identifies if the cell at (n, 0) is a header cell or not
   let mask = 0;
+  const isExperimentEnabled = expValEquals('platform_editor_analyse_table_with_merged_cells', 'isEnabled', true);
 
   // At minimum we should have 1 cell in the table.
   const topLeftCell = table.nodeAt(tableMap.map[0]);
   // If this cell is a header that could indicate
   mask |= topLeftCell && topLeftCell.type === types.header_cell ? 1 : 0;
   if (tableMap.width > 1) {
-    const cell = table.nodeAt(tableMap.map[1]);
-    // If the cell at (0, 1) is a header then we set the bit flag to indicate row headers are enabled, otherwise if it's
+    const cell = isExperimentEnabled ? table.nodeAt(tableMap.map[tableMap.width - 1]) : table.nodeAt(tableMap.map[1]);
+    // If the cell at (0, n) is a header then we set the bit flag to indicate row headers are enabled, otherwise if it's
     // not then we will set the col headers enabled flag (and vice versa in the branch below) only if the cell at (0,0)
     // was a header cell.
     mask |= cell && cell.type === types.header_cell ? 2 : 4 * (mask & 1);
   }
   if (tableMap.height > 1) {
-    const cell = table.nodeAt(tableMap.map[tableMap.width]);
+    const cell = isExperimentEnabled ? table.nodeAt(tableMap.map[tableMap.width * (tableMap.height - 1)]) : table.nodeAt(tableMap.map[tableMap.width]);
     mask |= cell && cell.type === types.header_cell ? 4 : 2 * (mask & 1);
   }
   return {

package/dist/es2019/utils/cells.js

@@ -1,3 +1,5 @@
+// eslint-disable-next-line @typescript-eslint/consistent-type-imports
+
 import { TableMap } from '../table-map';
 export function pointsAtCell($pos) {
   return $pos.parent.type.spec.tableRole === 'row' && $pos.nodeAfter;

package/dist/esm/utils/analyse-table.js

@@ -1,28 +1,30 @@
+import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 export function determineTableHeaderStateFromTableNode(table, tableMap, types) {
-  // We only really need to check the 2nd cell in the row/col if it's a header, since we only support a single full row/col header on the top & left
-  // of a table. We can assume that if the 2nd cell is a header then the entire row/col is a header.
-  // Be carefull though! when checking the 1st cell as it shares its header state with both row/cols.
-  // This means we wont be able to reliably identify header state on tables smaller the 2x2, however we can do best guess.
+  // We only really need to check the nth (last) cell in the row/col if it's a header, since we only support a single full row/col header on the top & left
+  // of a table. We can assume that if the nth (last) cell is also a header then the entire row/col is a header.
+  // Be careful though! when checking the 1st cell as it shares its header state with both row/cols.
+  // This means we won't be able to reliably identify header state on tables smaller than 2x2, however we can make a best guess.
 
   // This is a 3 bit mask;
   // bit: 0 = Identifies if the cell at (0, 0) (row, col - 0-based) is a header cell or not
-  // bit: 1 = Identifies if the cell at (0, 1) is a header cell or not
-  // bit: 2 = Identifies if the cell at (1, 0) is a header cell or not
+  // bit: 1 = Identifies if the cell at (0, n) is a header cell or not
+  // bit: 2 = Identifies if the cell at (n, 0) is a header cell or not
   var mask = 0;
+  var isExperimentEnabled = expValEquals('platform_editor_analyse_table_with_merged_cells', 'isEnabled', true);
 
   // At minimum we should have 1 cell in the table.
   var topLeftCell = table.nodeAt(tableMap.map[0]);
   // If this cell is a header that could indicate
   mask |= topLeftCell && topLeftCell.type === types.header_cell ? 1 : 0;
   if (tableMap.width > 1) {
-    var cell = table.nodeAt(tableMap.map[1]);
-    // If the cell at (0, 1) is a header then we set the bit flag to indicate row headers are enabled, otherwise if it's
+    var cell = isExperimentEnabled ? table.nodeAt(tableMap.map[tableMap.width - 1]) : table.nodeAt(tableMap.map[1]);
+    // If the cell at (0, n) is a header then we set the bit flag to indicate row headers are enabled, otherwise if it's
     // not then we will set the col headers enabled flag (and vice versa in the branch below) only if the cell at (0,0)
     // was a header cell.
     mask |= cell && cell.type === types.header_cell ? 2 : 4 * (mask & 1);
   }
   if (tableMap.height > 1) {
-    var _cell = table.nodeAt(tableMap.map[tableMap.width]);
+    var _cell = isExperimentEnabled ? table.nodeAt(tableMap.map[tableMap.width * (tableMap.height - 1)]) : table.nodeAt(tableMap.map[tableMap.width]);
     mask |= _cell && _cell.type === types.header_cell ? 4 : 2 * (mask & 1);
   }
   return {

package/dist/esm/utils/cells.js

@@ -1,3 +1,5 @@
+// eslint-disable-next-line @typescript-eslint/consistent-type-imports
+
 import { TableMap } from '../table-map';
 export function pointsAtCell($pos) {
   return $pos.parent.type.spec.tableRole === 'row' && $pos.nodeAfter;

package/dist/types/pm-plugins/plugin-key.d.ts

@@ -1,3 +1,3 @@
 import { PluginKey } from '@atlaskit/editor-prosemirror/state';
-export declare const tableEditingKey: PluginKey<any>;
-export declare const fixTablesKey: PluginKey<any>;
+export declare const tableEditingKey: PluginKey;
+export declare const fixTablesKey: PluginKey;

package/dist/types/utils/cells.d.ts

@@ -1,6 +1,6 @@
-import { type ResolvedPos } from '@atlaskit/editor-prosemirror/model';
+import { Node, type ResolvedPos } from '@atlaskit/editor-prosemirror/model';
 import { type Axis } from '../types';
-export declare function pointsAtCell($pos: ResolvedPos): false | import("prosemirror-model").Node | null;
+export declare function pointsAtCell($pos: ResolvedPos): false | Node | null;
 export declare function cellNear($pos: ResolvedPos): ResolvedPos | null;
 export declare function cellAround($pos: ResolvedPos): ResolvedPos | null;
 export declare function nextCell($pos: ResolvedPos, axis: Axis, dir: number): ResolvedPos | null;

package/dist/types-ts4.5/pm-plugins/plugin-key.d.ts

@@ -1,3 +1,3 @@
 import { PluginKey } from '@atlaskit/editor-prosemirror/state';
-export declare const tableEditingKey: PluginKey<any>;
-export declare const fixTablesKey: PluginKey<any>;
+export declare const tableEditingKey: PluginKey;
+export declare const fixTablesKey: PluginKey;

package/dist/types-ts4.5/utils/cells.d.ts

@@ -1,6 +1,6 @@
-import { type ResolvedPos } from '@atlaskit/editor-prosemirror/model';
+import { Node, type ResolvedPos } from '@atlaskit/editor-prosemirror/model';
 import { type Axis } from '../types';
-export declare function pointsAtCell($pos: ResolvedPos): false | import("prosemirror-model").Node | null;
+export declare function pointsAtCell($pos: ResolvedPos): false | Node | null;
 export declare function cellNear($pos: ResolvedPos): ResolvedPos | null;
 export declare function cellAround($pos: ResolvedPos): ResolvedPos | null;
 export declare function nextCell($pos: ResolvedPos, axis: Axis, dir: number): ResolvedPos | null;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-tables",
-	"version": "2.9.42",
+	"version": "2.9.44",
 	"description": "A package that contains common classes and utility functions for editor tables",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
@@ -29,7 +29,7 @@
 	"dependencies": {
 		"@atlaskit/editor-prosemirror": "^7.3.0",
 		"@atlaskit/platform-feature-flags": "^1.1.0",
-		"@atlaskit/tmp-editor-statsig": "^35.0.0",
+		"@atlaskit/tmp-editor-statsig": "^36.0.0",
 		"@babel/runtime": "^7.0.0"
 	},
 	"devDependencies": {