@linzjs/windows 9.1.4 → 9.3.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

@@ -1,4 +1,4 @@
-import { LuiButton, LuiCheckboxInput, LuiMiniSpinner } from '@linzjs/lui';
+import { LuiButton, LuiCheckboxInput, LuiIcon, LuiMiniSpinner } from '@linzjs/lui';
 import { LuiButtonProps } from '@linzjs/lui/dist/components/LuiButton/LuiButton';
 import { IconName } from '@linzjs/lui/dist/components/LuiIcon/LuiIcon';
 import clsx from 'clsx';
@@ -20,6 +20,7 @@ export interface LuiModalAsyncPrefabButton<RT> {
   default?: boolean;
   level?: LuiButtonProps['level'];
   title: string;
+  icon?: IconName;
   value?: RT;
 }
 
@@ -102,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
@@ -148,12 +150,14 @@ export const LuiModalPrefab = (props: PropsWithChildren<LuiModalAsyncPrefabProps
           <LuiButton
             key={i}
             level={pf.level ?? 'secondary'}
+            className={clsx(pf.icon && 'lui-button-icon')}
             onClick={() => {
               setDontShow();
               resolve(pf.value);
             }}
             buttonProps={pf.default ? { 'data-autofocus': true } : undefined}
           >
+            {pf.icon && <LuiIcon name={pf.icon} alt={pf.title} size={'md'} />}
             {pf.title}
           </LuiButton>
         ))}
@@ -185,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/package.json

@@ -13,7 +13,7 @@
     "popout"
   ],
   "main": "./dist/index.ts",
-  "version": "9.1.4",
+  "version": "9.3.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",
@@ -60,7 +59,7 @@
   },
   "devDependencies": {
     "@chromatic-com/storybook": "^4.1.3",
-    "@linzjs/lui": "^24.10.1",
+    "@linzjs/lui": "^24.11.0",
     "@linzjs/step-ag-grid": "^29.14.1",
     "@linzjs/style": "^5.4.0",
     "@rollup/plugin-commonjs": "^28.0.9",
@@ -80,15 +79,15 @@
     "@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",
-    "rollup": "^4.59.0",
+    "rollup": "^4.60.0",
     "rollup-plugin-copy": "^3.5.0",
     "sass": "^1.98.0",
     "sass-loader": "^16.0.7",
@@ -107,8 +106,8 @@
     "vitest": "^4.1.0"
   },
   "optionalDependencies": {
-    "@rollup/rollup-linux-x64-gnu": "^4.59.0",
-    "@swc/core-linux-x64-gnu": "^1.15.18"
+    "@rollup/rollup-linux-x64-gnu": "^4.60.0",
+    "@swc/core-linux-x64-gnu": "^1.15.21"
   },
   "browserslist": {
     "production": [
@@ -124,7 +123,7 @@
   },
   "husky": {
     "hooks": {
-      "pre-commit": "npm run lint"
+      "pre-commit": "npm run lintall"
     }
   }
 }