@torpor/ui 0.2.1 → 0.3.0

package/dist/TabGroup/TabContent.torp

@@ -6,32 +6,35 @@ interface TabContentProps {
 	id?: string;
 	/** Class name(s) for the root element */
 	class?: string,
+	/** The index of this item */
+	index?: number,
+	/** The value for this item */
+	value?: string,
 }
 
 /**
  * Content for a TabGroup, which can be revealed or hidden by an associated TabTrigger.
  */
-export default function TabContent($props: TabContentProps) {
-	$props ??= $watch({});
-	$props.id ||= getId();
+export default function TabContent($props: TabContentProps | undefined) {
+	let defaultId = getId();
 
 	const context = $context[TabGroupContextName] as TabGroupContext;
 	if (!context) {
 		throw new Error("TabContent must be contained within a TabGroup");
 	}
-	let $state = context.registerItem($props.index, $props.value);
+	let $state = context.registerItem($props?.index, $props?.value);
 
 	// Set the content ID so that it can be used with aria-controls in TabTrigger
-	$state.contentId = $props.id;
+	$state.contentId = $props?.id ?? defaultId;
 
 	@render {
 		<div
-			{$props.id}
-			class={`torp-tab-content ${$props.class ?? ""}`.trim()}
+			id={$props?.id ?? defaultId}
+			class={`torp-tab-content ${$props?.class ?? ""}`.trim()}
 			role="tabpanel"
 			aria-labelledby={$state.headerId}
-			data-orientation={$props.orientation}
-			data-state={$state.dataState}
+			data-orientation={context.orientation}
+			data-state={$state.active ? "active" : "inactive"}
 			data-disabled={$state.disabled ? "true" : undefined}
 		>
 			@if ($state.active) {

package/dist/TabGroup/TabGroup.torp

@@ -10,7 +10,7 @@ interface TabGroupProps {
 	/** How tabs are activated; if automatic they are shown on focus, if manual they must be clicked */
 	activation?: "automatic" | "manual",
 	/** The value of the active TabContent */
-	value?: string | string[],
+	value?: string,
 	/** If set to true, the user cannot interact with any items in the TabGroup */
 	disabled?: boolean,
 	/** Raised when the active item has changed */
@@ -36,11 +36,23 @@ interface TabGroupProps {
  * </TabGroup>
  * ```
  */
-export default function TabGroup($props: TabGroupProps) {
-	$props ??= $watch({});
-	$props.orientation ||= "horizontal";
-	$props.activation ||= "automatic";
-	$props.value ||= "";
+export default function TabGroup($props: TabGroupProps | undefined) {
+	let orientation = $props?.orientation ?? "horizontal";
+	let activation = $props?.activation ?? "automatic";
+
+	let $state = $watch({
+		value: ""
+	});
+
+	// Sync props and state
+	$run(() => {
+		$state.value = $props?.value ?? "";
+	});
+	$run(() => {
+		if ($props !== undefined) {
+			$props.value = $state.value;
+		}
+	});
 
 	let itemStates: ItemState[] = [];
 
@@ -55,34 +67,28 @@ export default function TabGroup($props: TabGroupProps) {
 		removeItem,
 		triggerItem,
 		handleHeaderKey,
-		orientation: $props.orientation,
-		activation: $props.activation,
+		orientation,
+		activation,
 	};
 	$context[TabGroupContextName] = context;
 
 	$run(() => {
-		// HACK: This should probably be done with setters?
-		let newValue = $props.value;
-
 		// Ensure the value passed in was a string (in case e.g. the user has set values to ints)
-		newValue = newValue?.toString();
+		$state.value = $state.value.toString();
 		for (let item of itemStates) {
-			item.active = item.value === newValue;
+			item.active = item.value === $state.value;
 		}
-
-		// Set the value
-		$props.value = newValue;
 	});
 
 	$run(() => {
-		itemStates.forEach((s) => s.parentDisabled = $props.disabled === true);
+		itemStates.forEach((s) => s.parentDisabled = $props?.disabled === true);
 	});
 
 	// Dispatch the change event when the value changes
 	let firstChange = true;
 	$run(() => {
-		const ev = $props.onchange;
-		const value = $props.value;
+		const ev = $props?.onchange;
+		const value = $props?.value;
 		if (firstChange) {
 			firstChange = false;
 		} else if (ev) {
@@ -92,8 +98,8 @@ export default function TabGroup($props: TabGroupProps) {
 
 	// On mount, if no value was set, show the first tab content
 	$mount(() => {
-		if ($props.value === "" && itemStates.length) {
-			$props.value = itemStates[0].value;
+		if ($state.value === "" && itemStates.length) {
+			$state.value = itemStates[0].value;
 		}
 	});
 
@@ -113,7 +119,7 @@ export default function TabGroup($props: TabGroupProps) {
 
 	function registerItem(index: number | undefined, value: string | undefined): ItemState {
 		// If an index wasn't set for the item, set it automatically
-		if (!index) {
+		if (index === undefined) {
 			index = nextItemIndex++;
 		}
 
@@ -130,7 +136,7 @@ export default function TabGroup($props: TabGroupProps) {
 		}
 
 		// If a value wasn't set for the item, set it to the index
-		if (!value) {
+		if (value === undefined) {
 			value = index.toString();
 		}
 
@@ -139,8 +145,8 @@ export default function TabGroup($props: TabGroupProps) {
 			index,
 			value,
 			// TODO: this would be better than using $mount, but it doesn't work with nested tabs
-			// active: value === $props.value || ($props.value === "" && index === 0),
-			active: value === $props.value,
+			// active: value === $props?.value || ($props?.value === "" && index === 0),
+			active: value === $state.value,
 			headerId: "",
 			contentId: "",
 			disabled: false,
@@ -149,8 +155,8 @@ export default function TabGroup($props: TabGroupProps) {
 				return this.active ? "active" : "inactive";
 			},
 			setFocused: () => {},
-			orientation: $props.orientation,
-			activation: $props.activation
+			orientation,
+			activation
 		});
 		itemStates.splice(index, 0, newItemState);
 
@@ -171,7 +177,7 @@ export default function TabGroup($props: TabGroupProps) {
 		for (let item of itemStates) {
 			// The item is active if it has the supplied value
 			if (item.value === toggleValue) {
-				$props.value = item.value;
+				$state.value = item.value;
 				break;
 			}
 		}
@@ -181,7 +187,7 @@ export default function TabGroup($props: TabGroupProps) {
 		switch (e.key) {
 			case "Left":
 			case "ArrowLeft": {
-				if ($props.orientation === "horizontal" && headerIndex > 0) {
+				if (orientation === "horizontal" && headerIndex > 0) {
 					e.preventDefault();
 					itemStates[headerIndex - 1].setFocused();
 				}
@@ -189,7 +195,7 @@ export default function TabGroup($props: TabGroupProps) {
 			}
 			case "Up":
 			case "ArrowUp": {
-				if ($props.orientation === "vertical" && headerIndex > 0) {
+				if (orientation === "vertical" && headerIndex > 0) {
 					e.preventDefault();
 					itemStates[headerIndex - 1].setFocused();
 				}
@@ -197,7 +203,7 @@ export default function TabGroup($props: TabGroupProps) {
 			}
 			case "Right":
 			case "ArrowRight": {
-				if ($props.orientation === "horizontal" && headerIndex < itemStates.length - 1) {
+				if (orientation === "horizontal" && headerIndex < itemStates.length - 1) {
 					e.preventDefault();
 					itemStates[headerIndex + 1].setFocused();
 				}
@@ -205,7 +211,7 @@ export default function TabGroup($props: TabGroupProps) {
 			}
 			case "Down":
 			case "ArrowDown": {
-				if ($props.orientation === "vertical" && headerIndex < itemStates.length - 1) {
+				if (orientation === "vertical" && headerIndex < itemStates.length - 1) {
 					e.preventDefault();
 					itemStates[headerIndex + 1].setFocused();
 				}
@@ -230,9 +236,9 @@ export default function TabGroup($props: TabGroupProps) {
 
 	@render {
 		<div
-			{$props.id}
-			class={`torp-tab-group ${$props.class ?? ""}`.trim()}
-			data-orientation={$props.orientation}
+			{$props?.id}
+			class={`torp-tab-group ${$props?.class ?? ""}`.trim()}
+			data-orientation={$props?.orientation}
 		>
 			<slot />
 		</div>

package/dist/TabGroup/TabGroupTypes.ts

@@ -3,9 +3,13 @@ export const TabItemContextName = "TabItem";
 
 export interface TabGroupContext {
 	/** Called from a header when it is added */
-	registerHeader: (index: number, value: string, setFocused: () => void) => ItemState;
+	registerHeader: (
+		index: number | undefined,
+		value: string | undefined,
+		setFocused: () => void,
+	) => ItemState;
 	/** Called from an item when it is added */
-	registerItem: (index: number, value: string) => ItemState;
+	registerItem: (index: number | undefined, value: string | undefined) => ItemState;
 	/** Called from an item when it is removed */
 	removeItem: (index: number) => void;
 	/** Called from an item when it is triggered */
@@ -17,12 +21,14 @@ export interface TabGroupContext {
 }
 
 export interface TabItemContext {
-	/** Called when an item is triggered */
+	/** Raised when an item is triggered */
 	triggerItem: (value: string) => void;
-	/** Called when a TabHeader receives a keyboard event */
+	/** Raised when a TabHeader receives a keyboard event */
 	handleHeaderKey: (index: number, e: KeyboardEvent) => void;
 	/** The reactive state for an item */
 	state: ItemState;
+	orientation: "horizontal" | "vertical";
+	activation: "automatic" | "manual";
 }
 
 export interface ItemState {
@@ -33,7 +39,6 @@ export interface ItemState {
 	contentId: string;
 	disabled: boolean;
 	parentDisabled: boolean;
-	dataState: "active" | "inactive";
 	/** Called in the TabGroup when a header needs to be focused */
 	setFocused: () => void;
 }

package/dist/TabGroup/TabHeader.torp

@@ -7,6 +7,10 @@ interface TabHeaderProps {
 	class?: string,
 	/** The type of HTML header to use */
 	as?: "h1" | "h2" | "h3" | "h4" | "h5" | "h6"
+	/** The index of this header's item */
+	index?: number,
+	/** The value for this header's item */
+	value?: string,
 }
 
 /**
@@ -20,19 +24,21 @@ interface TabHeaderProps {
  * </TabHeader>
  * ```
  */
-export default function TabHeader($props: TabHeaderProps) {
-	$props ??= $watch({});
+export default function TabHeader($props: TabHeaderProps | undefined) {
+	let button: HTMLButtonElement;
 
 	const context = $context[TabGroupContextName] as TabGroupContext;
 	if (!context) {
 		throw new Error("TabHeader must be contained within a TabGroup");
 	}
-	let $state = context.registerHeader($props.index, $props.value);
+	let $state = context.registerHeader($props?.index, $props?.value);
 
 	const itemContext: TabItemContext = {
 		triggerItem: context.triggerItem,
 		handleHeaderKey: context.handleHeaderKey,
 		state: $state,
+		orientation: context.orientation,
+		activation: context.activation
 	};
 	$context[TabItemContextName] = itemContext;
 
@@ -44,16 +50,17 @@ export default function TabHeader($props: TabHeaderProps) {
 	});
 
 	@render {
-		@// TODO: $props.as
+		@// TODO: $props?.as
 		<h4
-			{$props.id}
+			{$props?.id}
 			class={[
 				"torp-tab-header",
-				$props.class,
-				$state.orientation
+				$props?.class,
+				context.orientation
 			]}
-			data-state={$state.dataState}
+			data-state={$state.active ? "active" : "inactive"}
 			data-disabled={$state.disabled ? "true" : undefined}
+			&ref={button}
 		>
 			<slot />
 		</h4>

package/dist/TabGroup/TabList.torp

@@ -22,9 +22,7 @@ interface TabListProps {
  * </TabList>
  * ```
  */
-export default function TabList($props: TabListProps) {
-	$props ??= $watch({});
-
+export default function TabList($props: TabListProps | undefined) {
 	// Get stuff out of the TabGroupContext
 	const context = $context[TabGroupContextName] as TabGroupContext;
 	if (!context) {
@@ -44,10 +42,10 @@ export default function TabList($props: TabListProps) {
 
 	@render {
 		<div
-			{$props.id}
-			class={`torp-tab-list ${$props.class ?? ""}`.trim()}
+			{$props?.id}
+			class={`torp-tab-list ${$props?.class ?? ""}`.trim()}
 			role="tablist"
-			aria-label={$props.ariaLabel}
+			aria-label={$props?.ariaLabel}
 			aria-orientation={orientation}
 			data-orientation={orientation}
 			tabindex="0"

package/dist/TabGroup/TabTrigger.torp

@@ -11,9 +11,8 @@ interface TabTriggerProps {
 /**
  * A header button for a tab item, which can be used to reveal an associated TabContent.
  */
-export default function TabTrigger($props: TabTriggerProps) {
-	$props ??= $watch({});
-	$props.id ||= getId();
+export default function TabTrigger($props: TabTriggerProps | undefined) {
+	let defaultId = getId();
 
 	let button: HTMLButtonElement;
 
@@ -25,7 +24,7 @@ export default function TabTrigger($props: TabTriggerProps) {
 	let $state = context.state;
 
 	// Set the header ID so that it can be used with aria-labelledby in TabContent
-	$state.headerId = $props.id;
+	$state.headerId = $props?.id ?? defaultId;
 
 	// Let the item know how to set the button's focus
 	$state.setFocused = setFocused;
@@ -33,7 +32,7 @@ export default function TabTrigger($props: TabTriggerProps) {
 	// This function is called by the parent TabGroup to focus this item's button
 	function setFocused() {
 		button.focus();
-		if ($state.activation === "automatic") {
+		if (context.activation === "automatic") {
 			context.triggerItem($state.value);
 		}
 	}
@@ -52,17 +51,17 @@ export default function TabTrigger($props: TabTriggerProps) {
 	@render {
 		<button
 			type="button"
-			{$props.id}
-			class={`torp-tab-trigger ${$props.class ?? ""}`.trim()}
+			id={$props?.id ?? defaultId}
+			class={`torp-tab-trigger ${$props?.class ?? ""}`.trim()}
 			role="tab"
 			tabindex="-1"
 			disabled={$state.disabled || $state.parentDisabled}
 			aria-selected={$state.active.toString()}
 			aria-disabled={$state.disabled}
 			aria-controls={$state.contentId}
-			data-state={$state.dataState}
+			data-state={$state.active ? "active" : "inactive"}
 			data-disabled={$state.disabled}
-			data-orientation={$state.orientation}
+			data-orientation={context.orientation}
 			&ref={button}
 			onclick={handleClick}
 			onkeydown={handleKey}

package/dist/TabGroup/index.d.ts

@@ -6,6 +6,10 @@ interface TabContentProps {
 	id?: string;
 	/** Class name(s) for the root element */
 	class?: string,
+	/** The index of this item */
+	index?: number,
+	/** The value for this item */
+	value?: string,
 }
 
 /**
@@ -14,7 +18,7 @@ interface TabContentProps {
 declare function TabContent(
 	$parent: ParentNode,
 	$anchor: Node | null,
-	$props: TabContentProps,
+	$props: TabContentProps | undefined,
 	$context?: Record<PropertyKey, any>,
 	$slots?: Record<string, SlotRender>,
 ): void;
@@ -31,7 +35,7 @@ interface TabGroupProps {
 	/** How tabs are activated; if automatic they are shown on focus, if manual they must be clicked */
 	activation?: "automatic" | "manual",
 	/** The value of the active TabContent */
-	value?: string | string[],
+	value?: string,
 	/** If set to true, the user cannot interact with any items in the TabGroup */
 	disabled?: boolean,
 	/** Raised when the active item has changed */
@@ -60,7 +64,7 @@ interface TabGroupProps {
 declare function TabGroup(
 	$parent: ParentNode,
 	$anchor: Node | null,
-	$props: TabGroupProps,
+	$props: TabGroupProps | undefined,
 	$context?: Record<PropertyKey, any>,
 	$slots?: Record<string, SlotRender>,
 ): void;
@@ -74,6 +78,10 @@ interface TabHeaderProps {
 	class?: string,
 	/** The type of HTML header to use */
 	as?: "h1" | "h2" | "h3" | "h4" | "h5" | "h6"
+	/** The index of this header's item */
+	index?: number,
+	/** The value for this header's item */
+	value?: string,
 }
 
 /**
@@ -90,7 +98,7 @@ interface TabHeaderProps {
 declare function TabHeader(
 	$parent: ParentNode,
 	$anchor: Node | null,
-	$props: TabHeaderProps,
+	$props: TabHeaderProps | undefined,
 	$context?: Record<PropertyKey, any>,
 	$slots?: Record<string, SlotRender>,
 ): void;
@@ -122,7 +130,7 @@ interface TabListProps {
 declare function TabList(
 	$parent: ParentNode,
 	$anchor: Node | null,
-	$props: TabListProps,
+	$props: TabListProps | undefined,
 	$context?: Record<PropertyKey, any>,
 	$slots?: Record<string, SlotRender>,
 ): void;
@@ -142,7 +150,7 @@ interface TabTriggerProps {
 declare function TabTrigger(
 	$parent: ParentNode,
 	$anchor: Node | null,
-	$props: TabTriggerProps,
+	$props: TabTriggerProps | undefined,
 	$context?: Record<PropertyKey, any>,
 	$slots?: Record<string, SlotRender>,
 ): void;

package/dist/icons/ArrowDown.torp

@@ -1,4 +1,6 @@
-export default function ArrowDown($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function ArrowDown($props: IconProps | undefined) {
 	@render {
 		<svg
 			{$props?.id}

package/dist/icons/ArrowLeft.torp

@@ -1,4 +1,6 @@
-export default function ArrowLeft($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function ArrowLeft($props: IconProps | undefined) {
 	@render {
 		<svg
 			{$props?.id}

package/dist/icons/ArrowRight.torp

@@ -1,4 +1,6 @@
-export default function ArrowRight($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function ArrowRight($props: IconProps | any) {
 	@render {
 		<svg
 			{$props?.id}

package/dist/icons/ArrowUp.torp

@@ -1,4 +1,6 @@
-export default function ArrowUp($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function ArrowUp($props: IconProps | undefined) {
 	@render {
 		<svg
 			{$props?.id}

package/dist/icons/Bars.torp

@@ -1,4 +1,6 @@
-export default function Bars($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function Bars($props: IconProps | undefined) {
 	@render {
 		<svg
 			{$props?.id}

package/dist/icons/Check.torp

@@ -1,4 +1,6 @@
-export default function Check($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function Check($props: IconProps | undefined) {
 	@render {
 		<svg
 			{$props?.id}

package/dist/icons/ChevronDown.torp

@@ -1,4 +1,6 @@
-export default function ChevronDown($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function ChevronDown($props: IconProps | undefined) {
 	@render {
 		<svg
 			{$props?.id}

package/dist/icons/ChevronLeft.torp

@@ -1,4 +1,6 @@
-export default function ChevronLeft($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function ChevronLeft($props: IconProps | undefined) {
 	@render {
 		<svg
 			{$props?.id}

package/dist/icons/ChevronRight.torp

@@ -1,4 +1,6 @@
-export default function ChevronRight($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function ChevronRight($props: IconProps | undefined) {
 	@render {
 		<svg
 			{$props?.id}

package/dist/icons/ChevronUp.torp

@@ -1,4 +1,6 @@
-export default function ChevronUp($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function ChevronUp($props: IconProps | undefined) {
 	@render {
 		<svg
 			{$props?.id}

package/dist/icons/Cross.torp

@@ -1,4 +1,6 @@
-export default function Cross($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function Cross($props: IconProps | undefined) {
 	@render {
 		<svg
 			{$props?.id}

package/dist/icons/IconTypes.ts

@@ -0,0 +1,6 @@
+export interface IconProps {
+	/** An ID for the svg element */
+	id?: string;
+	/** Class name(s) for the svg element */
+	class?: string;
+}

package/dist/icons/Loading.torp

@@ -1,4 +1,6 @@
-export default function Loading($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function Loading($props: IconProps | undefined) {
 	@render {
 		<svg
 			{$props?.id}

package/dist/icons/Minus.torp

@@ -1,4 +1,6 @@
-export default function Minus($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function Minus($props: IconProps | undefined) {
 	@render {
 		<svg
 			{$props?.id}

package/dist/icons/Plus.torp

@@ -1,4 +1,6 @@
-export default function Plus($props: any) {
+import { type IconProps } from "./IconTypes";
+
+export default function Plus($props: IconProps | undefined) {
 	@render {
 		<svg
 			{$props?.id}