@fuzdev/fuz_ui 0.202.0 → 0.203.0

package/dist/ContextmenuRootForSafariCompatibility.svelte

@@ -18,20 +18,19 @@
 	 * and relies on the standard `contextmenu` event.
 	 */
 	import {swallow} from '@fuzdev/fuz_util/dom.js';
-	import {DEV} from 'esm-env';
 	import {on} from 'svelte/events';
-	import type {ComponentProps, Snippet} from 'svelte';
 
 	import {
 		contextmenu_context,
-		contextmenu_dimensions_context,
 		ContextmenuState,
 		contextmenu_open,
 		contextmenu_check_global_root,
 	} from './contextmenu_state.svelte.js';
-	import ContextmenuLinkEntry from './ContextmenuLinkEntry.svelte';
-	import ContextmenuTextEntry from './ContextmenuTextEntry.svelte';
-	import ContextmenuSeparator from './ContextmenuSeparator.svelte';
+	import ContextmenuMenu, {
+		link_entry_default,
+		text_entry_default,
+		separator_entry_default,
+	} from './ContextmenuMenu.svelte';
 	import {
 		CONTEXTMENU_DEFAULT_OPEN_OFFSET_X,
 		CONTEXTMENU_DEFAULT_OPEN_OFFSET_Y,
@@ -40,10 +39,10 @@
 		CONTEXTMENU_DEFAULT_LONGPRESS_DURATION,
 		CONTEXTMENU_DEFAULT_LONGPRESS_MOVE_TOLERANCE,
 		contextmenu_is_valid_target,
-		contextmenu_create_keyboard_handlers,
-		contextmenu_create_keydown_handler,
-		contextmenu_calculate_constrained_x,
-		contextmenu_calculate_constrained_y,
+		contextmenu_resolve_contextmenu_event,
+		ContextmenuBypassTracker,
+		ContextmenuOpenGuard,
+		type ContextmenuRootBaseProps,
 	} from './contextmenu_helpers.js';
 
 	const {
@@ -60,8 +59,7 @@
 		text_entry = text_entry_default,
 		separator_entry = separator_entry_default,
 		children,
-	}: {
-		contextmenu?: ContextmenuState;
+	}: ContextmenuRootBaseProps & {
 		/**
 		 * The number of pixels the pointer can be moved without canceling `longpress`.
 		 */
@@ -70,93 +68,25 @@
 		 * The number of milliseconds after a touch starts before opening the Fuz contextmenu.
 		 */
 		longpress_duration?: number;
-		/**
-		 * Whether to detect tap-then-longpress to bypass the Fuz contextmenu.
-		 * This allows access to the system contextmenu by tapping once then long-pressing.
-		 * Setting to `false` disables the gesture.
-		 */
-		bypass_with_tap_then_longpress?: boolean;
-		/**
-		 * The number of milliseconds between taps to detect a gesture that bypasses the Fuz contextmenu.
-		 * Used only when `bypass_with_tap_then_longpress` is true.
-		 * If the duration is too long, it'll detect more false positives and interrupt normal usage,
-		 * but too short and some people will have difficulty performing the gesture.
-		 */
-		bypass_window?: number;
-		/**
-		 * The number of pixels the pointer can be moved between taps to detect a tap-then-longpress.
-		 * Used only when `bypass_with_tap_then_longpress` is true.
-		 */
-		bypass_move_tolerance?: number;
-		/**
-		 * The number of pixels to offset from the pointer X position when opened.
-		 * Useful to ensure the first menu item is immediately under the pointer.
-		 */
-		open_offset_x?: number;
-		/**
-		 * The number of pixels to offset from the pointer Y position when opened.
-		 * Useful to ensure the first menu item is immediately under the pointer.
-		 */
-		open_offset_y?: number;
-		/**
-		 * If `true`, wraps `children` with a div and listens to events on it instead of the window.
-		 */
-		scoped?: boolean;
-		/**
-		 * Snippet for rendering link entries.
-		 * Set to `null` to disable automatic link detection.
-		 * Defaults to `link_entry_default` which renders `ContextmenuLinkEntry`.
-		 */
-		link_entry?: Snippet<[ComponentProps<typeof ContextmenuLinkEntry>]> | null;
-		/**
-		 * Snippet for rendering copy text entries.
-		 * Set to `null` to disable automatic copy text detection.
-		 * Defaults to `text_entry_default` which renders `ContextmenuTextEntry`.
-		 */
-		text_entry?: Snippet<[ComponentProps<typeof ContextmenuTextEntry>]> | null;
-		/**
-		 * Snippet for rendering separator entries.
-		 * Set to `null` to disable automatic separator rendering.
-		 * Defaults to `separator_entry_default` which renders `ContextmenuSeparator`.
-		 */
-		separator_entry?: Snippet<[ComponentProps<typeof ContextmenuSeparator>]> | null;
-		children: Snippet;
 	} = $props();
 
 	contextmenu_context.set(() => contextmenu);
 
-	if (DEV) contextmenu_check_global_root(() => scoped); // TODO @many is this import tree-shaken?
+	contextmenu_check_global_root(() => scoped); // DEV-only internally, eliminated from production bundles
 
+	// The menu element while opened, bound from `ContextmenuMenu.svelte`.
 	let el: HTMLElement | undefined = $state.raw();
 
-	const {layout} = $derived(contextmenu);
-
-	const dimensions = contextmenu_dimensions_context.set();
-
-	const x = $derived(
-		contextmenu_calculate_constrained_x(contextmenu.x, dimensions.width, layout.width),
-	);
-	const y = $derived(
-		contextmenu_calculate_constrained_y(contextmenu.y, dimensions.height, layout.height),
-	);
-
 	// TODO maybe show an indicator fade in at these coordinates
 
-	// State for tap-then-longpress bypass detection.
-	// These values are `undefined` when unused, and `null` after being reset.
-	let touch_x: number | undefined | null = $state.raw();
-	let touch_y: number | undefined | null = $state.raw();
-	let first_tap_time: number | undefined | null = $state.raw();
-	let longpress_timeout: NodeJS.Timeout | undefined | null = $state.raw();
-	let longpress_opened: boolean | undefined = $state.raw();
-	let longpress_bypass: boolean | undefined = $state.raw();
-	let tap_tracking_timeout: NodeJS.Timeout | undefined | null = $state.raw();
+	// Longpress tracking state - plain non-reactive variables, only used inside event handlers.
+	let touch_x = 0;
+	let touch_y = 0;
+	let longpress_timeout: NodeJS.Timeout | null = null;
+	let longpress_opened = false;
 
-	/**
-	 * Blocks the next click event. Set to true when a longpress completes to prevent
-	 * iOS's synthesized click from activating the first menu item.
-	 */
-	let block_next_click = $state.raw(false);
+	const bypass_tracker = new ContextmenuBypassTracker();
+	const open_guard = new ContextmenuOpenGuard();
 
 	/**
 	 * Adds contextmenu-pending class to body during longpress tracking.
@@ -180,7 +110,7 @@
 	 */
 	const reset_longpress_timeout = (): void => {
 		longpress_opened = false;
-		if (longpress_timeout != null) {
+		if (longpress_timeout !== null) {
 			clearTimeout(longpress_timeout);
 			longpress_timeout = null;
 		}
@@ -188,69 +118,45 @@
 		// Don't clear tap tracking state here - we need it for tap-then-longpress detection
 	};
 
-	/**
-	 * Clears tap tracking state and bypass flag used for bypass detection.
-	 */
-	const reset_tap_tracking = (): void => {
-		first_tap_time = null;
-		touch_x = null;
-		touch_y = null;
-		longpress_bypass = false;
-		if (tap_tracking_timeout != null) {
-			clearTimeout(tap_tracking_timeout);
-			tap_tracking_timeout = null;
-		}
-	};
-
 	/**
 	 * Resets all state - both longpress and tap tracking.
 	 */
 	const reset_all = (): void => {
 		reset_longpress_timeout(); // Also removes contextmenu-pending class
-		reset_tap_tracking();
+		bypass_tracker.reset();
 	};
 
+	const open_options = $derived({
+		open_offset_x,
+		open_offset_y,
+		link_enabled: link_entry !== null,
+		text_enabled: text_entry !== null,
+		separator_enabled: separator_entry !== null,
+	});
+
 	const on_window_contextmenu = (e: MouseEvent) => {
-		// handle the tap-then-longpress bypass gesture
-		if (longpress_bypass) {
-			reset_tap_tracking(); // Clear bypass state after using it
-			return;
-		}
-		const {target} = e;
+		// let the system contextmenu through after a tap-then-longpress bypass gesture
+		if (bypass_tracker.consume()) return;
 		// handle touch devices that trigger `'contextmenu'` slower than our longpress
 		if (longpress_opened) {
 			// Don't prevent contextmenu events on elements inside our own contextmenu
 			// This allows the browser's native contextmenu (useful for dev tools, inspecting elements, etc.)
-			if (el?.contains(target as Node)) {
+			if (el?.contains(e.target as Node)) {
 				return; // Let the event pass through
 			}
 			reset_all();
 			swallow(e);
 			return;
 		}
-		if (!contextmenu_is_valid_target(target, e.shiftKey)) {
-			return;
-		}
-		// Don't open contextmenu when clicking on the menu itself
-		if (el?.contains(target)) {
-			return;
-		}
-		if (
-			contextmenu_open(target, e.clientX + open_offset_x, e.clientY + open_offset_y, contextmenu, {
-				link_enabled: link_entry !== null,
-				text_enabled: text_entry !== null,
-				separator_enabled: separator_entry !== null,
-			})
-		) {
-			swallow(e);
+		if (contextmenu_resolve_contextmenu_event(e, contextmenu, el, open_guard, open_options)) {
 			reset_all(); // handle touch devices that trigger `'contextmenu'` faster than our longpress
 		}
 	};
 
-	// Needed for the iOS workaround. Registered with { passive: false } via $effect (window) or attachment (scoped).
+	// Needed for the iOS workaround. Registered non-passively via `touch_event_attachment` (window or scoped element).
 	const touchstart = (e: TouchEvent): void => {
 		longpress_opened = false;
-		block_next_click = false; // Clear any stale click blocking flag
+		open_guard.touchstart(); // begins a gesture, clearing stale flags
 		const {touches, target} = e;
 		if (
 			contextmenu.opened ||
@@ -264,39 +170,13 @@
 
 		const {clientX, clientY} = touches[0]!;
 
-		// Bypass the contextmenu behavior in certain conditions including a tap-and-longpress gesture.
-		// To handle tap-then-longpress we need to see if `first_tap_time`
-		// is less than `bypass_window`, and also allow a small amount
-		// of pointer movement, `bypass_move_tolerance`.
 		// The builtin `'contextmenu'` event will still fire for non-iOS browsers,
-		// so `longpress_bypass` is used to tell the handler `on_window_contextmenu` to exit early.
-		if (bypass_with_tap_then_longpress) {
-			if (
-				first_tap_time != null &&
-				performance.now() - first_tap_time < bypass_window &&
-				Math.hypot(clientX - touch_x!, clientY - touch_y!) < bypass_move_tolerance
-			) {
-				// Tap-then-longpress detected! Set bypass and clear tap tracking state.
-				// Must manually clear state (not call reset_tap_tracking) to preserve bypass flag.
-				longpress_bypass = true;
-				first_tap_time = null;
-				touch_x = null;
-				touch_y = null;
-				if (tap_tracking_timeout != null) {
-					clearTimeout(tap_tracking_timeout); // Prevent timeout from clearing bypass
-					tap_tracking_timeout = null;
-				}
-				return;
-			}
-			// Record this tap for potential future bypass detection
-			first_tap_time = performance.now();
-			// Set timeout to clear stale tap tracking after the detection window expires
-			if (tap_tracking_timeout != null) {
-				clearTimeout(tap_tracking_timeout);
-			}
-			tap_tracking_timeout = setTimeout(() => {
-				reset_tap_tracking();
-			}, bypass_window);
+		// so the tracker's `bypassed` flag tells `on_window_contextmenu` to exit early.
+		if (
+			bypass_with_tap_then_longpress &&
+			bypass_tracker.track(clientX, clientY, bypass_window, bypass_move_tolerance)
+		) {
+			return;
 		}
 
 		touch_x = clientX;
@@ -305,26 +185,33 @@
 		// Add pending class to enable aggressive iOS blocking via CSS during tracking
 		add_contextmenu_pending_class();
 
-		if (longpress_timeout != null) reset_longpress_timeout();
+		if (longpress_timeout !== null) reset_longpress_timeout();
 		longpress_timeout = setTimeout(() => {
 			longpress_opened = true;
 			remove_contextmenu_pending_class(); // Tracking complete, menu opening
-			contextmenu_open(target, touch_x! + open_offset_x, touch_y! + open_offset_y, contextmenu, {
-				link_enabled: link_entry !== null,
-				text_enabled: text_entry !== null,
-				separator_enabled: separator_entry !== null,
-			});
+			if (
+				contextmenu_open(
+					target,
+					touch_x + open_offset_x,
+					touch_y + open_offset_y,
+					contextmenu,
+					open_options,
+				)
+			) {
+				// guard the release of this gesture from interacting with the menu
+				open_guard.opened();
+			}
 		}, longpress_duration);
 	};
 
-	// Needed for the iOS workaround. Registered with { passive: false } via $effect (window) or attachment (scoped).
+	// Needed for the iOS workaround. Registered non-passively via `touch_event_attachment` (window or scoped element).
 	const touchmove = (e: TouchEvent): void => {
 		// Exit early if no pending longpress or menu is already open
-		if (longpress_timeout == null || contextmenu.opened) return;
+		if (longpress_timeout === null || contextmenu.opened) return;
 		const {touches} = e;
 		if (touches.length !== 1) return;
 		const {clientX, clientY} = touches[0]!;
-		const distance = Math.hypot(clientX - touch_x!, clientY - touch_y!);
+		const distance = Math.hypot(clientX - touch_x, clientY - touch_y);
 		if (distance > longpress_move_tolerance) {
 			// User is scrolling - cancel longpress but DON'T preventDefault
 			reset_longpress_timeout();
@@ -334,43 +221,30 @@
 		// CRITICAL: Prevent iOS from showing magnifier, text selection, and link callouts
 		e.preventDefault();
 	};
-	// Needed for the iOS workaround. Registered with { passive: false } via $effect.
+	// Needed for the iOS workaround. Registered non-passively via `touch_event_attachment` (window or scoped element).
 	const touchend = (e: TouchEvent): void => {
+		// Swallow the release of a gesture that opened the menu (custom longpress or
+		// native `contextmenu`), stopping the browser from synthesizing mouse events at
+		// the touch point that would activate the first item, and arming the click
+		// blocker for iOS, which can synthesize the click regardless.
+		open_guard.touchend(e);
 		// Clear longpress timeout if it exists
-		if (longpress_timeout != null) {
-			if (longpress_opened) {
-				swallow(e);
-				// Block the next click to prevent iOS's synthesized click from activating the first menu item
-				block_next_click = true;
-			}
+		if (longpress_timeout !== null) {
 			reset_longpress_timeout();
 		}
-		// Clear bypass flag if it was set but the contextmenu event hasn't fired yet
-		// This handles the case where bypass is detected but user lifts finger before native menu opens
-		if (longpress_bypass) {
-			reset_tap_tracking();
-		}
-		// Note: We don't clear tap tracking state here - we preserve it for tap-then-longpress detection
+		// Clear a pending bypass when the user lifts before the native contextmenu opens.
+		// Tap tracking is otherwise preserved for tap-then-longpress detection.
+		bypass_tracker.consume();
 	};
 
 	/**
 	 * Handle touchcancel - this should reset all state since the gesture was interrupted.
 	 */
 	const touchcancel = (): void => {
+		open_guard.reset();
 		reset_all();
 	};
 
-	// Passive listener that runs during the event's `capture` phase
-	// so that things like the Dialog don't eat the events and prevent the contextmenu from closing.
-	const mousedown = (e: MouseEvent) => {
-		if (el && !el.contains(e.target as any)) {
-			contextmenu.close();
-		}
-	};
-
-	const keyboard_handlers = $derived(contextmenu_create_keyboard_handlers(contextmenu));
-	const keydown = $derived(contextmenu_create_keydown_handler(keyboard_handlers));
-
 	/**
 	 * Creates an attachment that registers touch event listeners with { passive: false }
 	 * to enable preventDefault() on iOS Safari. Works for both window (non-scoped mode) and
@@ -412,15 +286,15 @@
 
 <svelte:window
 	oncontextmenu={scoped ? undefined : on_window_contextmenu}
-	onmousedown={!contextmenu.opened ? undefined : mousedown}
-	onkeydown={!contextmenu.opened ? undefined : keydown}
 	{@attach scoped ? undefined : touch_event_attachment}
 />
 
 {#if scoped}
+	<!-- A transparent (`display: contents`) event-delegation wrapper. `role="group"` rather
+	than a landmark role - it satisfies the a11y lint without requiring an accessible name. -->
 	<div
 		class="contextmenu-root"
-		role="region"
+		role="group"
 		oncontextmenu={on_window_contextmenu}
 		{@attach touch_event_attachment}
 	>
@@ -430,61 +304,7 @@
 	{@render children()}
 {/if}
 
-{#if !contextmenu.has_custom_layout}
-	<div
-		class="contextmenu-layout"
-		bind:clientWidth={layout.width}
-		bind:clientHeight={layout.height}
-		aria-hidden="true"
-	></div>
-{/if}
-
-<!-- TODO Maybe animate a subtle highlight around the contextmenu as it appears? -->
-{#if contextmenu.opened}
-	<ul
-		class="contextmenu unstyled pane"
-		role="menu"
-		aria-label="context menu"
-		tabindex="-1"
-		bind:this={el}
-		bind:offsetWidth={dimensions.width}
-		bind:offsetHeight={dimensions.height}
-		style:transform="translate3d({x}px, {y}px, 0)"
-		onclickcapture={block_next_click
-			? (e) => {
-					// iOS synthesizes a click after touchend which
-					// can unintentionally activate the first menu item. This blocks it.
-					block_next_click = false;
-					swallow(e);
-				}
-			: undefined}
-	>
-		<!-- TODO maybe this should be generic? -->
-		{#each contextmenu.params as p (p)}
-			{#if typeof p === 'function'}
-				{@render p()}
-			{:else if p.snippet === 'link'}
-				{@render link_entry?.(p.props)}
-			{:else if p.snippet === 'text'}
-				{@render text_entry?.(p.props)}
-			{:else if p.snippet === 'separator'}
-				{@render separator_entry?.(p.props)}
-			{/if}
-		{/each}
-	</ul>
-{/if}
-
-{#snippet link_entry_default(props: ComponentProps<typeof ContextmenuLinkEntry>)}
-	<ContextmenuLinkEntry {...props} />
-{/snippet}
-
-{#snippet text_entry_default(props: ComponentProps<typeof ContextmenuTextEntry>)}
-	<ContextmenuTextEntry {...props} />
-{/snippet}
-
-{#snippet separator_entry_default(props: ComponentProps<typeof ContextmenuSeparator>)}
-	<ContextmenuSeparator {...props} />
-{/snippet}
+<ContextmenuMenu bind:el {contextmenu} {open_guard} {link_entry} {text_entry} {separator_entry} />
 
 <style>
 	:global(body.contextmenu-pending) {
@@ -499,35 +319,4 @@
 	.contextmenu-root {
 		display: contents;
 	}
-	.contextmenu {
-		--icon_size: var(--icon_size_xs);
-		/* TODO maybe make this responsive or a max of the page width
-		minus some space to tap items covered by the menu on the side,
-		or consider a totally different design for small screens (more dialog-like)  */
-		--contextmenu_width: 320px;
-		position: fixed;
-		left: 0;
-		top: 0;
-		z-index: var(--contextmenu_z_index, 200);
-		max-width: var(--contextmenu_width);
-		width: 100%;
-		/* Re-enable callouts on the menu itself to allow native contextmenu (for dev tools).
-		   Resets the global body blocking. Prevents the menu from being selected. */
-		-webkit-touch-callout: initial !important;
-	}
-	/* TODO hacky */
-	.contextmenu,
-	.contextmenu :global(menu.pane) {
-		border: var(--contextmenu_border_width, var(--border_width))
-			var(--contextmenu_border_style, var(--border_style))
-			var(--contextmenu_border_color, var(--border_color));
-		border-radius: var(--contextmenu_border_radius, var(--border_radius_xs));
-	}
-
-	/* TODO better way to do this? */
-	.contextmenu-layout {
-		z-index: -200;
-		position: fixed;
-		inset: 0;
-	}
 </style>

package/dist/ContextmenuRootForSafariCompatibility.svelte.d.ts

@@ -1,10 +1,5 @@
-import type { ComponentProps, Snippet } from 'svelte';
-import { ContextmenuState } from './contextmenu_state.svelte.js';
-import ContextmenuLinkEntry from './ContextmenuLinkEntry.svelte';
-import ContextmenuTextEntry from './ContextmenuTextEntry.svelte';
-import ContextmenuSeparator from './ContextmenuSeparator.svelte';
-type $$ComponentProps = {
-    contextmenu?: ContextmenuState;
+import { type ContextmenuRootBaseProps } from './contextmenu_helpers.js';
+type $$ComponentProps = ContextmenuRootBaseProps & {
     /**
      * The number of pixels the pointer can be moved without canceling `longpress`.
      */
@@ -13,57 +8,6 @@ type $$ComponentProps = {
      * The number of milliseconds after a touch starts before opening the Fuz contextmenu.
      */
     longpress_duration?: number;
-    /**
-     * Whether to detect tap-then-longpress to bypass the Fuz contextmenu.
-     * This allows access to the system contextmenu by tapping once then long-pressing.
-     * Setting to `false` disables the gesture.
-     */
-    bypass_with_tap_then_longpress?: boolean;
-    /**
-     * The number of milliseconds between taps to detect a gesture that bypasses the Fuz contextmenu.
-     * Used only when `bypass_with_tap_then_longpress` is true.
-     * If the duration is too long, it'll detect more false positives and interrupt normal usage,
-     * but too short and some people will have difficulty performing the gesture.
-     */
-    bypass_window?: number;
-    /**
-     * The number of pixels the pointer can be moved between taps to detect a tap-then-longpress.
-     * Used only when `bypass_with_tap_then_longpress` is true.
-     */
-    bypass_move_tolerance?: number;
-    /**
-     * The number of pixels to offset from the pointer X position when opened.
-     * Useful to ensure the first menu item is immediately under the pointer.
-     */
-    open_offset_x?: number;
-    /**
-     * The number of pixels to offset from the pointer Y position when opened.
-     * Useful to ensure the first menu item is immediately under the pointer.
-     */
-    open_offset_y?: number;
-    /**
-     * If `true`, wraps `children` with a div and listens to events on it instead of the window.
-     */
-    scoped?: boolean;
-    /**
-     * Snippet for rendering link entries.
-     * Set to `null` to disable automatic link detection.
-     * Defaults to `link_entry_default` which renders `ContextmenuLinkEntry`.
-     */
-    link_entry?: Snippet<[ComponentProps<typeof ContextmenuLinkEntry>]> | null;
-    /**
-     * Snippet for rendering copy text entries.
-     * Set to `null` to disable automatic copy text detection.
-     * Defaults to `text_entry_default` which renders `ContextmenuTextEntry`.
-     */
-    text_entry?: Snippet<[ComponentProps<typeof ContextmenuTextEntry>]> | null;
-    /**
-     * Snippet for rendering separator entries.
-     * Set to `null` to disable automatic separator rendering.
-     * Defaults to `separator_entry_default` which renders `ContextmenuSeparator`.
-     */
-    separator_entry?: Snippet<[ComponentProps<typeof ContextmenuSeparator>]> | null;
-    children: Snippet;
 };
 declare const ContextmenuRootForSafariCompatibility: import("svelte").Component<$$ComponentProps, {}, "">;
 type ContextmenuRootForSafariCompatibility = ReturnType<typeof ContextmenuRootForSafariCompatibility>;

package/dist/ContextmenuRootForSafariCompatibility.svelte.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ContextmenuRootForSafariCompatibility.svelte.d.ts","sourceRoot":"../src/lib/","sources":["../src/lib/ContextmenuRootForSafariCompatibility.svelte"],"names":[],"mappings":"AAwBA,OAAO,KAAK,EAAC,cAAc,EAAE,OAAO,EAAC,MAAM,QAAQ,CAAC;AAEpD,OAAO,EAGL,gBAAgB,EAGhB,MAAM,+BAA+B,CAAC;AACxC,OAAO,oBAAoB,MAAM,+BAA+B,CAAC;AACjE,OAAO,oBAAoB,MAAM,+BAA+B,CAAC;AACjE,OAAO,oBAAoB,MAAM,+BAA+B,CAAC;AAehE,KAAK,gBAAgB,GAAI;IACxB,WAAW,CAAC,EAAE,gBAAgB,CAAC;IAC/B;;OAEG;IACH,wBAAwB,CAAC,EAAE,MAAM,CAAC;IAClC;;OAEG;IACH,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B;;;;OAIG;IACH,8BAA8B,CAAC,EAAE,OAAO,CAAC;IACzC;;;;;OAKG;IACH,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB;;;OAGG;IACH,qBAAqB,CAAC,EAAE,MAAM,CAAC;IAC/B;;;OAGG;IACH,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB;;;OAGG;IACH,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB;;OAEG;IACH,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB;;;;OAIG;IACH,UAAU,CAAC,EAAE,OAAO,CAAC,CAAC,cAAc,CAAC,OAAO,oBAAoB,CAAC,CAAC,CAAC,GAAG,IAAI,CAAC;IAC3E;;;;OAIG;IACH,UAAU,CAAC,EAAE,OAAO,CAAC,CAAC,cAAc,CAAC,OAAO,oBAAoB,CAAC,CAAC,CAAC,GAAG,IAAI,CAAC;IAC3E;;;;OAIG;IACH,eAAe,CAAC,EAAE,OAAO,CAAC,CAAC,cAAc,CAAC,OAAO,oBAAoB,CAAC,CAAC,CAAC,GAAG,IAAI,CAAC;IAChF,QAAQ,EAAE,OAAO,CAAC;CAClB,CAAC;AAoXH,QAAA,MAAM,qCAAqC,sDAAwC,CAAC;AACpF,KAAK,qCAAqC,GAAG,UAAU,CAAC,OAAO,qCAAqC,CAAC,CAAC;AACtG,eAAe,qCAAqC,CAAC"}
+{"version":3,"file":"ContextmenuRootForSafariCompatibility.svelte.d.ts","sourceRoot":"../src/lib/","sources":["../src/lib/ContextmenuRootForSafariCompatibility.svelte"],"names":[],"mappings":"AAmCA,OAAO,EAWL,KAAK,wBAAwB,EAC7B,MAAM,0BAA0B,CAAC;AAElC,KAAK,gBAAgB,GAAI,wBAAwB,GAAG;IACnD;;OAEG;IACH,wBAAwB,CAAC,EAAE,MAAM,CAAC;IAClC;;OAEG;IACH,kBAAkB,CAAC,EAAE,MAAM,CAAC;CAC5B,CAAC;AAiQH,QAAA,MAAM,qCAAqC,sDAAwC,CAAC;AACpF,KAAK,qCAAqC,GAAG,UAAU,CAAC,OAAO,qCAAqC,CAAC,CAAC;AACtG,eAAe,qCAAqC,CAAC"}

package/dist/ContextmenuSeparator.svelte

@@ -4,7 +4,8 @@
 	const {...rest}: SvelteHTMLElements['li'] = $props();
 </script>
 
-<li role="separator" aria-orientation="vertical" {...rest} class:contextmenu-separator={true}></li>
+<!-- `aria-orientation` is omitted - it defaults to horizontal, correct for a vertical menu's rule -->
+<li role="separator" {...rest} class:contextmenu-separator={true}></li>
 
 <style>
 	.contextmenu-separator {

package/dist/ContextmenuSeparator.svelte.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ContextmenuSeparator.svelte.d.ts","sourceRoot":"../src/lib/","sources":["../src/lib/ContextmenuSeparator.svelte"],"names":[],"mappings":"AAkBA,QAAA,MAAM,oBAAoB,gFAAwC,CAAC;AACnE,KAAK,oBAAoB,GAAG,UAAU,CAAC,OAAO,oBAAoB,CAAC,CAAC;AACpE,eAAe,oBAAoB,CAAC"}
+{"version":3,"file":"ContextmenuSeparator.svelte.d.ts","sourceRoot":"../src/lib/","sources":["../src/lib/ContextmenuSeparator.svelte"],"names":[],"mappings":"AAmBA,QAAA,MAAM,oBAAoB,gFAAwC,CAAC;AACnE,KAAK,oBAAoB,GAAG,UAAU,CAAC,OAAO,oBAAoB,CAAC,CAAC;AACpE,eAAe,oBAAoB,CAAC"}