@linzjs/step-ag-grid 14.5.0 → 14.6.0

package/package.json

@@ -2,7 +2,7 @@
   "name": "@linzjs/step-ag-grid",
   "repository": "github:linz/step-ag-grid.git",
   "license": "MIT",
-  "version": "14.5.0",
+  "version": "14.6.0",
   "keywords": [
     "aggrid",
     "ag-grid",

package/src/components/Grid.tsx

@@ -97,8 +97,8 @@ export const Grid = ({
     setOnCellEditingComplete,
     getColDef,
   } = useContext(GridContext);
-  const { updatedDep, updatingCols } = useContext(GridUpdatingContext);
-  const gridContext = useContext(GridContext);
+  const { checkUpdating, updatedDep, updatingCols } = useContext(GridUpdatingContext);
+  const { prePopupOps } = useContext(GridContext);
 
   const gridDivRef = useRef<HTMLDivElement>(null);
 
@@ -387,13 +387,23 @@ export const Grid = ({
    */
   const startCellEditing = useCallback(
     (event: CellEvent) => {
-      event.data.id &&
-        gridContext.startCellEditing({
-          rowId: event.data.id,
-          colId: event.column.getColId(),
+      prePopupOps();
+      if (!event.node.isSelected()) {
+        event.node.setSelected(true, true);
+      }
+      // Cell already being edited, so don't re-edit until finished
+      if (checkUpdating([event.colDef.field ?? ""], event.data.id)) {
+        return;
+      }
+
+      if (event.rowIndex !== null) {
+        event.api.startEditingCell({
+          rowIndex: event.rowIndex,
+          colKey: event.column.getColId(),
         });
+      }
     },
-    [gridContext],
+    [checkUpdating, prePopupOps],
   );
 
   /**

package/src/stories/grid/gridFormInteraction/GridFormEditBearingCorrectionInteraction.stories.tsx

@@ -54,7 +54,7 @@ GridFormEditBearingCorrectionInteractions_.play = async ({ canvasElement }) => {
   // Test formatting a bearing
   expect(inputField).toBeInTheDocument();
   userEvent.type(inputField, "1.2345");
-  expect(await canvas.findByText("+1° 23' 45.0\"")).toBeInTheDocument();
+  expect(await canvas.findByText("+1° 23' 45\"")).toBeInTheDocument();
 
   // Test enter to save
   updateValue.mockClear();

package/src/utils/bearing.test.ts

@@ -7,30 +7,35 @@ import {
 
 describe("bearing", () => {
   test("convertDDToDMS converts decimal-ish degrees to DMS", () => {
-    expect(convertDDToDMS(-0.001, false, false)).toBe("-0° 00' 10\"");
-    expect(convertDDToDMS(-10.001, false, false)).toBe("-10° 00' 10\"");
-    expect(convertDDToDMS(-370.001, false, false)).toBe("-10° 00' 10\"");
-    expect(convertDDToDMS(359.595999, false, false)).toBe("0° 00'");
-    expect(convertDDToDMS(369.696999, false, false)).toBe("10° 10' 10\"");
-    expect(convertDDToDMS(221.555999, false, false)).toBe("221° 56'");
-    expect(convertDDToDMS(221.555999, false, true)).toBe("221° 56' 00.0\"");
-    expect(convertDDToDMS(5)).toBe("+5° 00' 00.0\"");
-    expect(convertDDToDMS(5.0)).toBe("+5° 00' 00.0\"");
-    expect(convertDDToDMS(5.00001)).toBe("+5° 00' 00.1\"");
-    expect(convertDDToDMS(5.1)).toBe("+5° 10' 00.0\"");
+    expect(convertDDToDMS(-0.001, false)).toBe("-0° 00' 10\"");
+    expect(convertDDToDMS(-10.001, false)).toBe("-10° 00' 10\"");
+    expect(convertDDToDMS(-370.001, false)).toBe("-10° 00' 10\"");
+    expect(convertDDToDMS(359.595999, false, 2)).toBe("0° 00'");
+    expect(convertDDToDMS(359.595999, false, 4)).toBe("0° 00' 00\"");
+    expect(convertDDToDMS(359.595999, false, 5)).toBe("0° 00' 00.0\"");
+    expect(convertDDToDMS(369.696999, false)).toBe("10° 10' 10\"");
+    expect(convertDDToDMS(369.696999, false, 4)).toBe("10° 10' 10\"");
+    expect(convertDDToDMS(221.555999, false, 2)).toBe("221° 56'");
+    expect(convertDDToDMS(221.555999, false, 5)).toBe("221° 56' 00.0\"");
+    expect(convertDDToDMS(5, true, 5)).toBe("+5° 00' 00.0\"");
+    expect(convertDDToDMS(5.0, true, 5)).toBe("+5° 00' 00.0\"");
+    expect(convertDDToDMS(5.00001, true, 5)).toBe("+5° 00' 00.1\"");
+    expect(convertDDToDMS(5.1, true, 5)).toBe("+5° 10' 00.0\"");
     expect(convertDDToDMS(5.12345)).toBe("+5° 12' 34.5\"");
     expect(convertDDToDMS(5.12345, false)).toBe("5° 12' 34.5\"");
-
-    expect(convertDDToDMS(300)).toBe("+300° 00' 00.0\"");
-    expect(convertDDToDMS(300.0)).toBe("+300° 00' 00.0\"");
+    expect(convertDDToDMS(5.12345, false, 2)).toBe("5° 12' 34.5\"");
+    expect(convertDDToDMS(300, true, 5)).toBe("+300° 00' 00.0\"");
+    expect(convertDDToDMS(300.0, true, 5)).toBe("+300° 00' 00.0\"");
     expect(convertDDToDMS(300.00001)).toBe("+300° 00' 00.1\"");
-    expect(convertDDToDMS(300.1)).toBe("+300° 10' 00.0\"");
+    expect(convertDDToDMS(300.1, true, 5)).toBe("+300° 10' 00.0\"");
     expect(convertDDToDMS(300.12345)).toBe("+300° 12' 34.5\"");
     expect(convertDDToDMS(300.12345, false)).toBe("300° 12' 34.5\"");
-
-    expect(convertDDToDMS(300, false, false)).toBe("300° 00'");
-    expect(convertDDToDMS(300.1, false, false)).toBe("300° 10'");
-    expect(convertDDToDMS(0, false)).toBe("0° 00'");
+    expect(convertDDToDMS(300, false, 2)).toBe("300° 00'");
+    expect(convertDDToDMS(300.1, false, 2)).toBe("300° 10'");
+    expect(convertDDToDMS(0, false, 2)).toBe("0° 00'");
+    expect(convertDDToDMS(0.0, false)).toBe("0° 00'");
+    expect(convertDDToDMS(0.0, false, 4)).toBe("0° 00' 00\"");
+    expect(convertDDToDMS(0.0, false, 5)).toBe("0° 00' 00.0\"");
   });
 
   test("bearingStringValidator", () => {

package/src/utils/bearing.ts

@@ -3,7 +3,7 @@ export const bearingValueFormatter = (value: any): string => {
   if (safeValue == null) {
     return "–";
   }
-  return convertDDToDMS(safeValue, false, false);
+  return convertDDToDMS(safeValue, false);
 };
 
 export const bearingCorrectionValueFormatter = (value: any): string => {
@@ -12,9 +12,9 @@ export const bearingCorrectionValueFormatter = (value: any): string => {
     return "–";
   }
   if (typeof safeValue === "string") {
-    return convertDDToDMS(bearingNumberParser(safeValue), true, true);
+    return convertDDToDMS(bearingNumberParser(safeValue), true, 4);
   }
-  return convertDDToDMS(safeValue, true, true);
+  return convertDDToDMS(safeValue, true, 4);
 };
 
 export const bearingNumberParser = (value: string): number | null => {
@@ -40,12 +40,21 @@ export const bearingStringValidator = (
   return customInvalid ? customInvalid(bearing) : null;
 };
 
-// Decimal-ish degrees to Degrees Minutes Seconds converter
-export const convertDDToDMS = (dd: number | null, showPositiveSymbol = true, addTrailingZeros = true): string => {
+/**
+ *Decimal-ish degrees to Degrees Minutes Seconds converter
+ *
+ * @param dd Decimal-ish degrees
+ * @param showPositiveSymbol whether the + sign appears before the number
+ * @param trailingZeroDp 2 | 4 | 5
+ * Example:
+ * 2 = 300° 00'
+ * 4 = 300° 00' 00"
+ * 5 = 300° 00' 00.0"
+ * @returns Degrees Minutes Seconds
+ */
+export const convertDDToDMS = (dd: number | null, showPositiveSymbol = true, trailingZeroDp: 2 | 4 | 5 = 2): string => {
   if (dd == null) return "–";
 
-  if (dd === 0) addTrailingZeros = false;
-
   // toFixed rounds parts up greater than 60, which has to be corrected below
   const [bearingWholeString, bearingDecimalString] = dd.toFixed(5).split(".");
 
@@ -71,14 +80,13 @@ export const convertDDToDMS = (dd: number | null, showPositiveSymbol = true, add
   const deciSecString = bearingDecimalString?.substring(4, 5);
 
   let dmsString = `${showPositiveSymbol && dd > 0 ? "+" : ""}${dd < 0 ? "-" : ""}${bearingWhole}°`;
-  if (addTrailingZeros || deciSecString != "0") {
+  if (trailingZeroDp === 5 || deciSecString != "0") {
     dmsString += `\xa0${minString}'\xa0${secString}.${deciSecString}"`; // "\xa0" is here for non-breaking space
-  } else if (secNumeric != 0) {
+  } else if (trailingZeroDp === 4 || secNumeric != 0) {
     dmsString += `\xa0${minString}'\xa0${secString}"`;
   } else {
     dmsString += `\xa0${minString}'`;
   }
-
   return dmsString;
 };