oolib 2.88.1 → 2.88.2

package/dist/components/SimpleTable/handlers/handleColActions.js

@@ -22,13 +22,16 @@ var __spreadArray = (this && this.__spreadArray) || function (to, from, pack) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.handleClearColContent = exports.handleInsertColBefore = exports.handleInsertColAfter = exports.handleDuplicateCol = exports.handleDeleteCol = exports.handleAddCol = void 0;
 var genHash_1 = require("../utils/genHash");
+var genNewCell = function () { return ({
+    id: "cell_".concat((0, genHash_1.genHash)(8)),
+}); };
 var handleAddColAtIdx = function (_a) {
     var value = _a.value, onChange = _a.onChange, id = _a.id, defaultColWidth = _a.defaultColWidth, colIdx = _a.colIdx;
     var data = value.data, colWidthConfig = value.colWidthConfig;
     var newData = __spreadArray([], data, true);
     newData = newData.map(function (row) {
         var newCellData = __spreadArray([], row.cellData, true);
-        newCellData.splice(colIdx, 0, { id: "cell_".concat((0, genHash_1.genHash)(8)) });
+        newCellData.splice(colIdx, 0, genNewCell());
         return __assign(__assign({}, row), { cellData: newCellData });
     });
     var newColWidthValue = defaultColWidth + "px";
@@ -132,14 +135,20 @@ var handleClearColContent = function (_a) {
     var data = value.data, colWidthConfig = value.colWidthConfig;
     var newData = data.map(function (row) {
         var newCellData = __spreadArray([], row.cellData, true);
-        var cellToClear = newCellData[colIdx];
-        delete cellToClear.value;
-        console.log({ cellToClear: cellToClear });
-        newCellData.splice(colIdx, 1, cellToClear);
-        console.log({ newCellData: newCellData });
+        // let cellToClear = newCellData[colIdx];
+        // delete cellToClear.value;
+        // console.log({cellToClear})
+        /**
+         * this genNewRow does one very important thing apart from the obvious of clearing the value:
+         * it creates a fresh new row id, which changes the 'react key' of the row being rendered, which
+         * ensures the the RTE used within the cell is reinitiated and the previous value in it is cleared.
+         * (since RTEs are not controlled components)
+         */
+        newCellData.splice(colIdx, 1, genNewCell());
+        // console.log({newCellData})
         return __assign(__assign({}, row), { cellData: newCellData });
     });
-    console.log({ newData: newData });
+    // console.log({newData})
     onChange &&
         onChange(id, { data: newData, colWidthConfig: colWidthConfig });
 };

package/dist/components/SimpleTable/handlers/handleRowActions.js

@@ -82,6 +82,12 @@ var handleClearRowContent = function (_a) {
     var value = _a.value, rowIdx = _a.rowIdx, onChange = _a.onChange, id = _a.id;
     var data = value.data, colWidthConfig = value.colWidthConfig;
     var newData = __spreadArray([], data, true);
+    /**
+     * this genNewRow does one very important thing apart from the obvious of clearing the value:
+     * it creates a fresh new row id, which changes the 'react key' of the row being rendered, which
+     * ensures the the RTE used within the cell is reinitiated and the previous value in it is cleared.
+     * (since RTEs are not controlled components)
+     */
     var newRow = genNewRow({ data: newData, rowIdx: rowIdx });
     newData.splice(rowIdx, 1, newRow);
     onChange && onChange(id, { data: newData, colWidthConfig: colWidthConfig });

package/dist/components/SimpleTable/index.js

@@ -104,7 +104,7 @@ function SimpleTable(_a) {
             defaultColWidth: defaultColWidth,
         });
     }, [_value]), value = _e.value, enableColActions = _e.enableColActions, //returns all possible actions config if no config.colHeaderData is defined. else undefined\
-    enableRowActions = _e.enableRowActions, canToggleColHeaderStyle = _e.canToggleColHeaderStyle;
+    enableRowActions = _e.enableRowActions;
     var _f = (0, useResizeTableColumns_1.useResizeTableColumns)({
         setColWidthConfig: function (setterFn) {
             var newColWidthConfig = setterFn(value.colWidthConfig);

package/dist/components/SimpleTable/utils/prepInitValueFromConfigIfNoValue.js

@@ -24,11 +24,12 @@ exports.prepInitValueFromConfigIfNoValue = void 0;
 var makeArrayFromLength_1 = require("../../../utils/makeArrayFromLength");
 var genHash_1 = require("./genHash");
 var prepInitValueFromConfigIfNoValue = function (_a) {
-    var _b, _c;
+    var _b, _c, _d;
     var _value = _a._value, config = _a.config, defaultColWidth = _a.defaultColWidth;
     //if colHeaderData is defined then that decides the noOfCols.
     var noOfCols = ((_b = config.colHeaderData) === null || _b === void 0 ? void 0 : _b.length) || config.noOfCols;
-    var value = ((_c = _value === null || _value === void 0 ? void 0 : _value.data) === null || _c === void 0 ? void 0 : _c.length) > 0 //value exists in the correct shape
+    var noOfRows = ((_c = config.rowHeaderData) === null || _c === void 0 ? void 0 : _c.length) || config.noOfRows;
+    var value = ((_d = _value === null || _value === void 0 ? void 0 : _value.data) === null || _d === void 0 ? void 0 : _d.length) > 0 //value exists in the correct shape
         ? _value
         : Array.isArray(_value) && _value.length > 0 //value exists but in deprecated shape
             ? {
@@ -40,7 +41,7 @@ var prepInitValueFromConfigIfNoValue = function (_a) {
             } //backwards compat
             : {
                 data: __spreadArray([], insertRowAndCellIds({
-                    noOfRows: config.noOfRows,
+                    noOfRows: noOfRows,
                     noOfCols: noOfCols,
                 }), true),
                 colWidthConfig: getInitColWidthConfigs({ noOfCols: noOfCols, defaultColWidth: defaultColWidth }),
@@ -48,7 +49,7 @@ var prepInitValueFromConfigIfNoValue = function (_a) {
     return {
         value: value,
         enableColActions: config.colHeaderData ? false : true,
-        enableRowActions: true,
+        enableRowActions: config.rowHeaderData ? false : true,
         canToggleColHeaderStyle: !!!config.colHeaderData, // meaning its true if no config.colHeaderData is defined. else false
     };
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oolib",
-  "version": "2.88.1",
+  "version": "2.88.2",
   "description": " OKE Component Library",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",