Update os-gui to 0.7.3

This fixes the position of the help window when minimized.

Author: 1j01

CONTRIBUTING.md

@@ -32,8 +32,8 @@ See [**Development Setup**](./README.md#Development-Setup) on the readme.
 - The project uses [jQuery](https://jquery.com/), and a convention of prefixing variables that hold jQuery objects with `$`
 	- There are also some weird pseudo-classes like `$ColorBox` which extend and return jQuery objects. I don't recommend this pattern for new code.
 - Menu code and windowing code lives in the [os-gui](https://github.com/1j01/os-gui) project.
-	- It can be synced (one-way) by running `npm i os-gui@latest && npm run sync-os-gui`
-	- To develop os-gui in the context of jspaint, you can run `npm link` in a clone of os-gui, then run `npm link os-gui` in jspaint and then `npm run sync-os-gui` after making any changes in os-gui.
+	- It can be synced (one-way) by running `npm i os-gui@latest --save-exact && npm run sync-os-gui`
+	- To develop os-gui in the context of jspaint, you can run `npm link` in a clone of os-gui, then run `npm link os-gui && npm run sync-os-gui` in jspaint after making any changes in os-gui.
 	- (Maybe I should version this using git-subrepo?)
 	- Some window behavior specific to jspaint is in `$ToolWindow.js` and `$Component.js`
 - `image-manipulation.js` should contain just rendering related code, and ideally no dialogs except maybe some error messages.

lib/os-gui/$Window.js

@@ -726,9 +726,9 @@ function $Window(options = {}) {
 						$w.css({
 							position: "fixed",
 							top: `calc(100% - ${titlebar_height + 5}px)`,
-							left: `${to_x}`,
-							width: `${to_width}`,
-							height: `${titlebar_height}`,
+							left: `${to_x}px`,
+							width: `${to_width}px`,
+							height: `${titlebar_height}px`,
 						});
 					}
 				};
@@ -1511,7 +1511,13 @@ You can also disable this warning by passing {iframes: {ignoreCrossOrigin: true}
 				resize_pointer_y = e.clientY;
 				resize_pointer_id = (e.pointerId ?? e.originalEvent?.pointerId); // originalEvent doesn't exist for triggerHandler()
 
-				$handle[0].setPointerCapture(resize_pointer_id); // keeps cursor consistent when mouse moves over other elements
+				try {
+					$handle[0].setPointerCapture(resize_pointer_id); // keeps cursor consistent when mouse moves over other elements
+				} catch (error) {
+					// Prevent error from failing test; Cypress sends synthetic events that aren't trusted; I could pass a pointerId but not an active pointer id
+					// NotFoundError: Failed to execute 'setPointerCapture' on 'Element': No active pointer with the given id is found.
+					console.warn("Failed to capture pointer for resize handle drag:", error);
+				}
 
 				// handle_pointermove(e); // was useful for checking that the offset is correct (should not do anything, if it's correct!)
 			});
@@ -1692,6 +1698,8 @@ You can also disable this warning by passing {iframes: {ignoreCrossOrigin: true}
 			$component.detach();
 		}
 		$w.closed = true;
+		minimize_slots[$w._minimize_slot_index] = null;
+
 		$event_target.triggerHandler("closed");
 		$w.trigger("closed");
 		// TODO: change usages of "close" to "closed" where appropriate

lib/os-gui/build/layout.css

@@ -50,6 +50,11 @@ body > .window-titlebar {
 	display: none; /* prevent resizing when window is minimized */
 }
 
+.os-window.minimized-without-taskbar .menus {
+	display: none; /* hide menubar when window is minimized */
+}
+
+
 /* Fix dragging things (like windows) over iframes */
 .dragging iframe {
 	pointer-events: none;

lib/os-gui/build/layout.css.map

@@ -106,7 +106,7 @@
     "globals": "^15.3.0",
     "npm-run-all": "^4.1.5",
     "onchange": "^7.1.0",
-    "os-gui": "0.7.2",
+    "os-gui": "0.7.3",
     "rtlcss": "^4.1.1",
     "start-server-and-test": "^2.0.3",
     "typescript": "^5.4.3"