@postxl/ui-components 1.4.0 → 1.4.2

package/dist/index.js

@@ -5927,12 +5927,15 @@ function DataGridViewMenu({ table, savedViews, activeViewId, currentUserId, isAd
 									})
 								]
 							}, view.id);
-							return /* @__PURE__ */ jsxs(Button, {
-								variant: "ghost",
-								size: "max",
-								className: cn("text-start gap-1 px-2 h-6 rounded-sm cursor-pointer group hover:bg-accent", isActive && "bg-accent/50"),
-								onClick: () => onApplyView?.(view.id),
+							return /* @__PURE__ */ jsxs("div", {
+								className: cn("relative flex items-center text-start gap-1 px-2 h-6 rounded-sm cursor-pointer group hover:bg-accent", isActive && "bg-accent/50"),
 								children: [
+									/* @__PURE__ */ jsx("button", {
+										type: "button",
+										className: "absolute inset-0 hover:cursor-pointer",
+										onClick: () => onApplyView?.(view.id),
+										"aria-label": `Apply view "${view.name}"`
+									}),
 									view.isGlobal ? /* @__PURE__ */ jsx(GlobeIcon, { className: cn("size-3.5 shrink-0", isActive ? "text-primary" : "text-muted-foreground") }) : /* @__PURE__ */ jsx(BookmarkIcon, { className: cn("size-3.5 shrink-0", isActive ? "text-primary" : "text-muted-foreground") }),
 									/* @__PURE__ */ jsx("span", {
 										className: "text-sm truncate flex-1",
@@ -5940,7 +5943,7 @@ function DataGridViewMenu({ table, savedViews, activeViewId, currentUserId, isAd
 									}),
 									isActive ? /* @__PURE__ */ jsx(CheckIcon$1, { className: "group-hover:hidden size-3.5 text-primary shrink-0" }) : null,
 									canEdit && /* @__PURE__ */ jsxs("div", {
-										className: "hidden group-hover:flex items-center shrink-0",
+										className: "relative z-10 hidden group-hover:flex items-center shrink-0",
 										children: [
 											onUpdateView && /* @__PURE__ */ jsx(TooltipProvider, {
 												delayDuration: 300,
@@ -5949,8 +5952,7 @@ function DataGridViewMenu({ table, savedViews, activeViewId, currentUserId, isAd
 													children: /* @__PURE__ */ jsx(Button, {
 														variant: "ghost",
 														size: "iconXs",
-														onClick: (e) => {
-															e.stopPropagation();
+														onClick: () => {
 															resetAllStates();
 															setUpdatingViewId(view.id);
 														},
@@ -5968,8 +5970,7 @@ function DataGridViewMenu({ table, savedViews, activeViewId, currentUserId, isAd
 													children: /* @__PURE__ */ jsx(Button, {
 														variant: "ghost",
 														size: "iconXs",
-														onClick: (e) => {
-															e.stopPropagation();
+														onClick: () => {
 															resetAllStates();
 															setRenamingViewId(view.id);
 															setRenameValue(view.name);
@@ -5989,8 +5990,7 @@ function DataGridViewMenu({ table, savedViews, activeViewId, currentUserId, isAd
 														variant: "ghost",
 														size: "iconXs",
 														className: "text-destructive hover:text-destructive",
-														onClick: (e) => {
-															e.stopPropagation();
+														onClick: () => {
 															resetAllStates();
 															setDeletingViewId(view.id);
 														},
@@ -8667,15 +8667,26 @@ const Modal = ({ title = "", subtitle, isOpen, showCloseButton = true, setIsOpen
 			__e2e_test_id__,
 			onInteractOutside: preventClickOutsideToDismiss ? (e) => e.preventDefault() : void 0,
 			onEscapeKeyDown: preventClickOutsideToDismiss ? (e) => e.preventDefault() : void 0,
+			"aria-describedby": subtitle ? void 0 : "modal-content",
 			children: [
 				/* @__PURE__ */ jsxs(DialogHeader, {
 					className: cn("flex flex-row flex-shrink-0 items-center gap-2 p-3 min-h-10 border-b-1 border-(--discreet-border)", showCloseButton ? "pr-12" : "", subtitle ? "pb-0" : ""),
 					children: [/* @__PURE__ */ jsxs("div", {
 						className: "overflow-hidden",
-						children: [title && /* @__PURE__ */ jsx(DialogTitle, {
-							className: "text-xl overflow-hidden text-ellipsis whitespace-nowrap",
-							children: title
-						}), subtitle]
+						children: [
+							title && /* @__PURE__ */ jsx(DialogTitle, {
+								className: "text-xl overflow-hidden text-ellipsis whitespace-nowrap",
+								children: title
+							}),
+							subtitle && /* @__PURE__ */ jsx(DialogDescription, {
+								asChild: true,
+								children: subtitle
+							}),
+							!subtitle && /* @__PURE__ */ jsx(DialogDescription, {
+								className: "sr-only",
+								children: "Modal content"
+							})
+						]
 					}), controls.length > 0 && /* @__PURE__ */ jsx("div", {
 						className: "flex flex-grow justify-end items-baseline gap-2 whitespace-nowrap flex-1",
 						children: controls
@@ -8683,6 +8694,7 @@ const Modal = ({ title = "", subtitle, isOpen, showCloseButton = true, setIsOpen
 				}),
 				/* @__PURE__ */ jsx("div", {
 					className: "max-h-full h-full overflow-auto flex-grow",
+					id: "modal-content",
 					children
 				}),
 				footer && /* @__PURE__ */ jsx(DialogFooter, {