@linzjs/step-ag-grid 7.11.2 → 7.11.4

package/package.json

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

package/src/components/gridForm/GridFormMultiSelect.tsx

@@ -354,7 +354,7 @@ const MenuRadioItem = (props: {
     <MenuItem
       onClick={(e: ClickEvent) => {
         // Global react-menu MenuItem handler handles tabs
-        if (e.key !== "Tab") {
+        if (e.key !== "Tab" && e.key !== "Enter") {
           e.keepOpen = true;
           toggleValue(item);
         }

package/src/react-menu3/utils/utils.ts

@@ -88,16 +88,17 @@ export function commonProps(isDisabled?: boolean, isHovering?: boolean) {
 export const indexOfNode = (nodeList: NodeListOf<Node>, node: Node) => findIndex(nodeList, node);
 
 export const focusFirstInput = (container: any) => {
-  if (!(container instanceof Element)) return false;
+  // We can't use instanceof Element in portals, so I use querySelectorAll as a proxy here
+  if (!container || !("querySelectorAll" in container)) return false;
   const inputs = container.querySelectorAll("input[type='text'],textarea");
   const input = inputs[0];
-  if (input instanceof HTMLElement) {
-    input.focus();
-    // Text areas should start at end
-    if (input instanceof HTMLTextAreaElement) {
-      input.selectionStart = input.value.length;
-    }
-    return true;
+  // Using focus as proxy for HTMLElement
+  if (!input || !("focus" in input)) return false;
+  input.focus();
+  // Text areas should start at end
+  // this is a proxy for instanceof HTMLTextAreaElement
+  if (input.type === "textarea") {
+    input.selectionStart = input.value.length;
   }
-  return false;
+  return true;
 };

package/src/styles/GridTheme.scss

@@ -74,7 +74,7 @@
     align-items: center;
   }
 
-  .ag-cell.ag-cell-inline-editing {
+  .ag-cell.ag-cell-inline-editing:not(.ag-cell-auto-height) {
     bottom: 0;
   }