@linzjs/windows 9.2.0 → 9.4.0

package/README.md

@@ -2,6 +2,7 @@
 
 [![semantic-release: react](https://img.shields.io/badge/semantic--release-react-e10079?logo=semantic-release)](https://github.com/semantic-release/semantic-release)
 
+**Note: Make sure to install oxc plugin in Intellij/VSCode for development.**
 
 > Reusable promise based windowing component for LINZ / Toitū te whenua.
 

package/dist/LuiModalAsync/LuiModalAsync.tsx

@@ -33,7 +33,9 @@ export const LuiModalAsync = ({
   // The only way to create a modal dialog is to call showModal, open attribute will not work
   useEffect(() => {
     // Check if it's open already to support .vite hot deploys
-    !dialogRef.current?.open && dialogRef.current?.showModal?.();
+    if (!dialogRef.current?.open) {
+      dialogRef.current?.showModal?.();
+    }
   }, []);
 
   useEffect(() => {

package/dist/LuiModalAsync/LuiModalAsyncContextProvider.tsx

@@ -85,7 +85,9 @@ export const LuiModalAsyncContextProvider = ({ children }: PropsWithChildren<unk
       const r = oldWindowOpen(...props);
       // We don't want to track non app-windows, e.g. help screens
       if (!props[0]) {
-        r && openWindowsRef.current.push(r as typeof window);
+        if (r) {
+          openWindowsRef.current.push(r as typeof window);
+        }
       }
       return r;
     };
@@ -175,7 +177,9 @@ export const LuiModalAsyncContextProvider = ({ children }: PropsWithChildren<unk
    */
   useInterval(() => {
     const newModals = modals.filter(modalHasView);
-    newModals.length !== modals.length && setModals(newModals);
+    if (newModals.length !== modals.length) {
+      setModals(newModals);
+    }
     // We could do this with window close listeners, but they are unreliable
     openWindowsRef.current = openWindowsRef.current.filter((w) => !w.closed);
 

package/dist/LuiModalAsync/useLuiModalPrefab.tsx

@@ -103,13 +103,14 @@ export const LuiModalPrefab = (props: PropsWithChildren<LuiModalAsyncPrefabProps
 
   // If there are two options, option 1 is secondary, option 2 is primary
   // Otherwise all options are secondary
-  buttons.length === 2 &&
+  if (buttons.length === 2) {
     buttons.forEach((b, i) => {
       if (!b.level) {
         // Last button is primary all others are secondary
         b.level = i === buttons.length - 1 ? 'primary' : 'secondary';
       }
     });
+  }
 
   /**
    * When dialog closes save the "don't show" property
@@ -188,7 +189,7 @@ export const useLuiModalPrefab = () => {
   const { showModal, modalOwnerRef } = useShowAsyncModal();
 
   const showPrefabModal = useCallback(
-    <RT = string,>(props: PropsWithChildren<useLuiModalPrefabProps<RT>>): PromiseWithResolve<RT> => {
+    <RT = string>(props: PropsWithChildren<useLuiModalPrefabProps<RT>>): PromiseWithResolve<RT> => {
       debugLog(`Show '${props.level}' prefab-modal '${props.title}'`, props);
       if (typeof props.children === 'string') {
         // Convert \n to <br/>

package/dist/common/debug.ts

@@ -4,6 +4,6 @@ export const debugLog = (message: string, obj: unknown) => {
       console.debug(`@linzjs/windows: ${message}`, obj);
     }
   } catch (e) {
-    // ignore
+    console.error(e);
   }
 };

package/dist/panel/PanelHeader.tsx

@@ -40,7 +40,9 @@ export const PanelHeader = ({
   const [cursor, setCursor] = useState<'grab' | 'grabbing'>('grab');
 
   const headerMouseDown = () => {
-    !panelPoppedOut && setCursor('grabbing');
+    if (!panelPoppedOut) {
+      setCursor('grabbing');
+    }
   };
 
   return (

package/dist/ribbon/RibbonButton.tsx

@@ -21,6 +21,7 @@ export interface RibbonButtonProps {
   selected?: boolean;
   loading?: boolean;
   processing?: boolean;
+  processingMessage?: string;
   className?: string;
   testId?: string;
   onClick?: () => void;
@@ -39,6 +40,7 @@ export const RibbonButton = forwardRef<HTMLButtonElement, RibbonButtonProps>(fun
     selected,
     loading,
     processing,
+    processingMessage = `Loading ${title}`,
     disabled,
     testId,
     onClick,
@@ -63,7 +65,12 @@ export const RibbonButton = forwardRef<HTMLButtonElement, RibbonButtonProps>(fun
 
   if (processing) {
     return (
-      <div className={'RibbonButtonProcessing'}>
+      <div
+        className={'RibbonButtonProcessing'}
+        role={'status'}
+        title={processingMessage}
+        aria-label={processingMessage}
+      >
         <LuiMiniSpinner size={20} />
       </div>
     );

package/package.json

@@ -13,7 +13,7 @@
     "popout"
   ],
   "main": "./dist/index.ts",
-  "version": "9.2.0",
+  "version": "9.4.0",
   "peerDependencies": {
     "@linzjs/lui": ">=23",
     "lodash-es": ">=4",
@@ -37,11 +37,10 @@
     "bundle": "tsc --noEmit && rollup -c",
     "test": "vitest run",
     "test:watch": "vitest --watch",
-    "lintall": "run-s lint:style lint:eslint lint:circular-deps",
+    "lintall": "run-s lint:style lint:oxlint",
     "lint:style": "stylelint src/**/*.scss src/**/*.css --fix",
-    "lint:eslint": "eslint ./src --ext .js,.ts,.tsx --cache --ignore-path .gitignore",
-    "lint:fix": "eslint ./src --ext .js,.ts,.tsx --fix --cache --ignore-path .gitignore",
-    "lint:circular-deps": "npx madge --circular --extensions ts,tsx ./",
+    "lint:oxlint": "oxlint ./src",
+    "lint:fix": "oxlint --fix ./src",
     "storybook": "storybook dev -p 6006",
     "build-storybook": "storybook build",
     "chromatic": "chromatic --exit-zero-on-changes",
@@ -52,7 +51,7 @@
     "@emotion/react": "^11.14.0",
     "@emotion/styled": "11.14.1",
     "@types/uuid": "^11.0.0",
-    "lodash-es": "^4.17.23",
+    "lodash-es": ">=4",
     "react-loading-skeleton": "^3.5.0",
     "react-rnd": "^10.5.3",
     "usehooks-ts": "^3.1.1",
@@ -80,11 +79,11 @@
     "@vitest/ui": "^4.1.0",
     "ag-grid-community": "34.2.0",
     "ag-grid-react": "34.2.0",
-    "eslint-plugin-react": "^7.37.5",
-    "eslint-plugin-storybook": "^9.1.20",
     "jsdom": "^27.3.0",
     "mkdirp": "^3.0.1",
     "npm-run-all": "^4.1.5",
+    "oxlint": "^1.56.0",
+    "oxfmt": "^0.41.0",
     "react": "^18.3.1",
     "react-app-polyfill": "^3.0.0",
     "react-dom": "18.3.1",
@@ -124,7 +123,7 @@
   },
   "husky": {
     "hooks": {
-      "pre-commit": "npm run lint"
+      "pre-commit": "npm run lintall"
     }
   }
 }