horizon-layout 1.0.3 → 1.0.4

package/dist/HorizonLayout.svelte

@@ -22,7 +22,7 @@
 		validateConfig
 	} from './utils.js';
 	import { DEFAULT_MIN_WIDTH_RATIO, DEFAULT_MIN_HEIGHT_RATIO } from './internal-utils.js';
-	import { dropTargetType } from './internal-utils.js';
+	import { dropTargetType, roundSplitPoint } from './internal-utils.js';
 	import type { DropTarget } from './internal-types.js';
 
 	interface Popout {
@@ -364,7 +364,7 @@
 		const minRatio = split.direction === 'horizontal' ? minWidthRatio : minHeightRatio;
 		const min = (split.splitPoints[index - 1] ?? 0) + minRatio;
 		const max = (split.splitPoints[index + 1] ?? 1) - minRatio;
-		const clamped = Number(Math.min(Math.max(ratio, min), max).toFixed(4));
+		const clamped = roundSplitPoint(Math.min(Math.max(ratio, min), max));
 		if (clamped !== split.splitPoints[index]) split.splitPoints[index] = clamped;
 	}
 

package/dist/internal-utils.d.ts

@@ -2,5 +2,6 @@ import type { DropTarget } from './internal-types.ts';
 import type { Modifier } from './types.ts';
 export declare const DEFAULT_MIN_WIDTH_RATIO = 0.1;
 export declare const DEFAULT_MIN_HEIGHT_RATIO = 0.2;
+export declare function roundSplitPoint(ratio: number): number;
 export declare function getModifier(event: KeyboardEvent): Modifier | null;
 export declare function dropTargetType(target: DropTarget): 'side' | 'tab';

package/dist/internal-utils.js

@@ -1,5 +1,10 @@
 export const DEFAULT_MIN_WIDTH_RATIO = 0.1;
 export const DEFAULT_MIN_HEIGHT_RATIO = 0.2;
+/// rounds a split point to the precision it is stored with, so that values
+/// computed with floating-point arithmetic can be compared reliably
+export function roundSplitPoint(ratio) {
+    return Number(ratio.toFixed(4));
+}
 /// returns the modifier (ctrl, alt, shift, or a combination of them)
 /// associated with the keyboard event, or null if no modifier keys were pressed
 export function getModifier(event) {

package/dist/utils.js

@@ -1,5 +1,5 @@
 import { SvelteMap } from 'svelte/reactivity';
-import { DEFAULT_MIN_WIDTH_RATIO, DEFAULT_MIN_HEIGHT_RATIO } from "./internal-utils.js";
+import { DEFAULT_MIN_WIDTH_RATIO, DEFAULT_MIN_HEIGHT_RATIO, roundSplitPoint } from "./internal-utils.js";
 /// Remove a tab group that has become empty, then collapse any resulting
 /// single-child splits up the tree.
 export function simplifyTabGroup(tabGroup, nodeParentMap, config) {
@@ -115,11 +115,11 @@ export function areConfigsEquivalent(config1, config2) {
 /// split-point constraints are checked with the same thresholds.
 export function validateConfig(config, views, options = {}) {
     const { minWidthRatio = DEFAULT_MIN_WIDTH_RATIO, minHeightRatio = DEFAULT_MIN_HEIGHT_RATIO } = options;
-    if (config.maximizedView && !views.get(config.maximizedView)) {
+    if (config.maximizedView && !views.has(config.maximizedView)) {
         throw new Error(`unknown maximizedView id "${config.maximizedView}"`);
     }
     for (const id of config.popouts ?? []) {
-        if (!views.get(id))
+        if (!views.has(id))
             throw new Error(`unknown popout id "${id}"`);
     }
     if (!config.root)
@@ -130,7 +130,7 @@ export function validateConfig(config, views, options = {}) {
             throw new Error('activeTabIndex out of range');
         }
         for (const id of tabGroup.tabs) {
-            if (!views.get(id))
+            if (!views.has(id))
                 throw new Error(`unknown tab id "${id}"`);
             if (seenIds.has(id))
                 throw new Error(`duplicate tab id "${id}"`);
@@ -143,9 +143,9 @@ export function validateConfig(config, views, options = {}) {
         }
         const minRatio = split.direction === 'horizontal' ? minWidthRatio : minHeightRatio;
         for (let i = 0; i < split.splitPoints.length; i++) {
-            const point = split.splitPoints[i];
+            const point = roundSplitPoint(split.splitPoints[i]);
             const prev = split.splitPoints[i - 1] ?? 0;
-            if (point < prev + minRatio || point > 1 - minRatio) {
+            if (point < roundSplitPoint(prev + minRatio) || point > roundSplitPoint(1 - minRatio)) {
                 throw new Error('splitPoints violate minRatio constraints');
             }
         }
@@ -159,12 +159,10 @@ export function validateConfig(config, views, options = {}) {
     validateNode(config.root);
 }
 function parseNodeConfig(object, path) {
-    try {
-        return parseSplitConfig(object, path);
-    }
-    catch {
-        return parseTabGroupConfig(object, path);
-    }
+    if (typeof object !== 'object' || object === null)
+        throw new Error(`${path}: expected an object`);
+    const isSplit = 'direction' in object || 'views' in object || 'splitPoints' in object;
+    return isSplit ? parseSplitConfig(object, path) : parseTabGroupConfig(object, path);
 }
 function parseSplitConfig(object, path) {
     if (typeof object !== 'object' || object === null)

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "horizon-layout",
-	"version": "1.0.3",
+	"version": "1.0.4",
 	"keywords": [
 		"svelte"
 	],
@@ -40,6 +40,7 @@
 		"check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json",
 		"check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --watch",
 		"lint": "oxlint . && oxfmt --check .",
+		"test": "vitest run",
 		"format": "oxfmt ."
 	},
 	"devDependencies": {
@@ -54,7 +55,8 @@
 		"svelte": "^5.57.1",
 		"svelte-check": "^4.7.6",
 		"typescript": "^6.0.3",
-		"vite": "^8.3.2"
+		"vite": "^8.3.2",
+		"vitest": "^5.0.3"
 	},
 	"peerDependencies": {
 		"svelte": "^5.0.0"