@phuong-tran-redoc/document-engine-core 0.1.4 → 0.1.6

package/index.esm.js

@@ -1273,6 +1273,30 @@ function calculateColumnWidthsForNewColumn(oldColWidths) {
     const reduction = MIN_NEW_COL_WIDTH / oldColWidths.length;
     return oldColWidths.map((w) => w - reduction);
 }
+/**
+ * Read the `border` attrs of the cell the command should merge against.
+ *
+ * `$from.node(-1)` alone is only the cell for a collapsed cursor: with a `CellSelection`
+ * prosemirror-tables resolves `$from` *before* the head cell, so that lookup lands on the
+ * row and reports an empty border — which, now that an omitted field means "leave this
+ * one alone", would clear every field the caller did not pass. `getSelectedCells` covers
+ * the cell-selection and cursor cases; the depth walk covers a text selection inside one
+ * cell, which `getSelectedCells` returns nothing for.
+ */
+function getCurrentCellBorder(state) {
+    var _a, _b, _c, _d;
+    const [firstSelected] = getSelectedCells(state);
+    if (firstSelected)
+        return (_b = (_a = firstSelected.node.attrs) === null || _a === void 0 ? void 0 : _a['border']) !== null && _b !== void 0 ? _b : {};
+    const { $from } = state.selection;
+    for (let depth = $from.depth; depth > 0; depth--) {
+        const node = $from.node(depth);
+        if (node.type.name === 'tableCell' || node.type.name === 'tableHeader') {
+            return (_d = (_c = node.attrs) === null || _c === void 0 ? void 0 : _c['border']) !== null && _d !== void 0 ? _d : {};
+        }
+    }
+    return {};
+}
 const TableDefaultAttributes = {
     border: {
         style: 'double',
@@ -1598,7 +1622,6 @@ const StyledTable = Table.extend({
             }, 
             // Cell
             setCellBorder: (border) => ({ chain, state }) => {
-                var _a;
                 if (!border)
                     return chain()
                         .focus()
@@ -1606,14 +1629,15 @@ const StyledTable = Table.extend({
                         .updateAttributes('tableHeader', { border: null })
                         .run();
                 // Get current border and merge with new values
-                const { selection } = state;
-                const { $from } = selection;
-                const cellPos = $from.node(-1);
-                const currentBorder = ((_a = cellPos === null || cellPos === void 0 ? void 0 : cellPos.attrs) === null || _a === void 0 ? void 0 : _a['border']) || {};
+                const currentBorder = getCurrentCellBorder(state);
+                // `undefined` means "leave this field alone"; an explicit `null` clears it.
+                // A truthiness check conflates the two, so clearing a cell's border colour
+                // fell straight back to the colour being cleared and the UI could never
+                // remove one. Matches `setTableBorder` above.
                 const mergedBorder = {
-                    style: border.style ? border.style : currentBorder.style || null,
-                    color: border.color ? border.color : currentBorder.color || null,
-                    width: border.width ? border.width : currentBorder.width || null,
+                    style: border.style !== undefined ? border.style : currentBorder.style || null,
+                    color: border.color !== undefined ? border.color : currentBorder.color || null,
+                    width: border.width !== undefined ? border.width : currentBorder.width || null,
                 };
                 return chain()
                     .focus()

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@phuong-tran-redoc/document-engine-core",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "type": "module",
   "license": "MIT",
   "author": "Realestatedoc (Redoc)",