@openmrs/esm-styleguide 8.0.1-pre.3648 → 8.0.1-pre.3662

package/.turbo/turbo-build.log

@@ -13,6 +13,6 @@
 [0]   │ You can limit the size of your bundles by using import() to lazy load some parts of your application.
 [0]   │ For more info visit https://www.rspack.dev/guide/optimization/code-splitting
 [0] 
-[0] Rspack compiled with 3 warnings in 10.31 s
+[0] Rspack compiled with 3 warnings in 9.49 s
 [0] rspack --mode=production exited with code 0
 [1] tsc --project tsconfig.build.json exited with code 0

package/dist/workspaces2/workspace2.d.ts

@@ -75,6 +75,10 @@ type PromptReason = {
     explicit: boolean;
     windowName: string;
     workspaceName: string;
+} | {
+    reason: 'CLOSE_OTHER_WINDOWS';
+    explicit: false;
+    windowNameToSpare: string;
 };
 /**
  * A user can perform actions that explicitly result in closing workspaces

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openmrs/esm-styleguide",
-  "version": "8.0.1-pre.3648",
+  "version": "8.0.1-pre.3662",
   "license": "MPL-2.0",
   "description": "The styleguide for OpenMRS SPA",
   "main": "dist/openmrs-esm-styleguide.js",
@@ -98,17 +98,17 @@
     "swr": "2.x"
   },
   "devDependencies": {
-    "@openmrs/esm-api": "8.0.1-pre.3648",
-    "@openmrs/esm-config": "8.0.1-pre.3648",
-    "@openmrs/esm-emr-api": "8.0.1-pre.3648",
-    "@openmrs/esm-error-handling": "8.0.1-pre.3648",
-    "@openmrs/esm-extensions": "8.0.1-pre.3648",
-    "@openmrs/esm-globals": "8.0.1-pre.3648",
-    "@openmrs/esm-navigation": "8.0.1-pre.3648",
-    "@openmrs/esm-react-utils": "8.0.1-pre.3648",
-    "@openmrs/esm-state": "8.0.1-pre.3648",
-    "@openmrs/esm-translations": "8.0.1-pre.3648",
-    "@openmrs/esm-utils": "8.0.1-pre.3648",
+    "@openmrs/esm-api": "8.0.1-pre.3662",
+    "@openmrs/esm-config": "8.0.1-pre.3662",
+    "@openmrs/esm-emr-api": "8.0.1-pre.3662",
+    "@openmrs/esm-error-handling": "8.0.1-pre.3662",
+    "@openmrs/esm-extensions": "8.0.1-pre.3662",
+    "@openmrs/esm-globals": "8.0.1-pre.3662",
+    "@openmrs/esm-navigation": "8.0.1-pre.3662",
+    "@openmrs/esm-react-utils": "8.0.1-pre.3662",
+    "@openmrs/esm-state": "8.0.1-pre.3662",
+    "@openmrs/esm-translations": "8.0.1-pre.3662",
+    "@openmrs/esm-utils": "8.0.1-pre.3662",
     "@rspack/cli": "^1.3.11",
     "@rspack/core": "^1.3.11",
     "@types/geopattern": "^1.2.9",

package/src/workspaces2/workspace2.component.tsx

@@ -115,7 +115,8 @@ export const Workspace2: React.FC<Workspace2Props> = ({ title, children, hasUnsa
   }
 
   const { icon, canMaximize } = windowDef;
-  const canHide = !!icon;
+  const canCloseGroup = group.persistence === 'closable';
+  const canHide = !!icon && !canCloseGroup;
   const { maximized } = openedWindow;
   const width = windowDef?.width ?? 'narrow';
 
@@ -173,12 +174,14 @@ export const Workspace2: React.FC<Workspace2Props> = ({ title, children, hasUnsa
                         <ArrowRightIcon />
                       </HeaderGlobalAction>
                     ) : (
-                      <HeaderGlobalAction
-                        aria-label={getCoreTranslation('close')}
-                        onClick={() => closeWorkspace({ closeWindow: true })}
-                      >
-                        <CloseIcon />
-                      </HeaderGlobalAction>
+                      !canCloseGroup && (
+                        <HeaderGlobalAction
+                          aria-label={getCoreTranslation('close')}
+                          onClick={() => closeWorkspace({ closeWindow: true })}
+                        >
+                          <CloseIcon />
+                        </HeaderGlobalAction>
+                      )
                     )}
                   </>
                 ) : (

package/src/workspaces2/workspace2.ts

@@ -272,6 +272,33 @@ export async function launchWorkspace2<
         return false;
       }
     }
+  } else if (groupDef.persistence == 'closable') {
+    const okToCloseWorkspaces = await promptForClosingWorkspaces({
+      reason: 'CLOSE_OTHER_WINDOWS',
+      explicit: false,
+      windowNameToSpare: windowDef.name,
+    });
+    if (okToCloseWorkspaces) {
+      workspace2Store.setState({
+        ...storeState,
+        openedGroup: {
+          groupName: groupDef.name,
+          props: groupProps ?? storeState?.openedGroup?.props ?? null,
+        },
+        openedWindows: [
+          {
+            windowName: windowName,
+            openedWorkspaces: [newOpenedWorkspace(workspaceName, workspaceProps)], // root workspace at index 0
+            props: windowProps,
+            maximized: false,
+          },
+        ],
+        isMostRecentlyOpenedWindowHidden: false,
+      });
+      return true;
+    } else {
+      return false;
+    }
   } else {
     workspace2Store.setState({
       ...storeState,
@@ -316,7 +343,8 @@ function arePropsCompatible(a: Record<string, any> | null, b: Record<string, any
 type PromptReason =
   | { reason: 'CLOSE_WORKSPACE_GROUP'; explicit: boolean }
   | { reason: 'CLOSE_WINDOW'; explicit: boolean; windowName: string }
-  | { reason: 'CLOSE_WORKSPACE'; explicit: boolean; windowName: string; workspaceName: string };
+  | { reason: 'CLOSE_WORKSPACE'; explicit: boolean; windowName: string; workspaceName: string }
+  | { reason: 'CLOSE_OTHER_WINDOWS'; explicit: false; windowNameToSpare: string };
 
 /**
  * A user can perform actions that explicitly result in closing workspaces
@@ -340,7 +368,7 @@ export function promptForClosingWorkspaces(promptReason: PromptReason): Promise<
     for (let i = openedWindow.openedWorkspaces.length - 1; i >= 0; i--) {
       const openedWorkspace = openedWindow.openedWorkspaces[i];
 
-      if (openedWorkspace.hasUnsavedChanges || !promptReason.explicit) {
+      if (openedWorkspace.hasUnsavedChanges) {
         ret.push(openedWorkspace);
       }
       if (onlyUpToThisWorkspace && openedWorkspace.workspaceName === onlyUpToThisWorkspace) {
@@ -371,6 +399,12 @@ export function promptForClosingWorkspaces(promptReason: PromptReason): Promise<
         throw new Error(`Window ${promptReason.windowName} not found in opened windows.`);
       }
       affectedWorkspaces = getAffectedWorkspacesInWindow(openedWindow, promptReason.workspaceName);
+      break;
+    }
+    case 'CLOSE_OTHER_WINDOWS': {
+      const windowsToClose = openedWindows.filter((window) => window.windowName !== promptReason.windowNameToSpare);
+      affectedWorkspaces = windowsToClose.flatMap((w) => getAffectedWorkspacesInWindow(w));
+      break;
     }
   }