@rokkit/actions 1.0.0-next.92 → 1.0.0-next.94

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rokkit/actions",
-  "version": "1.0.0-next.92",
+  "version": "1.0.0-next.94",
   "description": "Contains generic actions that can be used in various components.",
   "author": "Jerry Thomas <me@jerrythomas.name>",
   "license": "MIT",
@@ -14,22 +14,20 @@
   "devDependencies": {
     "@sveltejs/vite-plugin-svelte": "^3.0.2",
     "@testing-library/svelte": "^4.1.0",
-    "@types/ramda": "^0.29.10",
-    "@vitest/coverage-v8": "^1.3.1",
-    "@vitest/ui": "~1.3.1",
+    "@types/ramda": "^0.29.12",
+    "@vitest/coverage-v8": "^1.4.0",
+    "@vitest/ui": "~1.4.0",
     "jsdom": "^24.0.0",
-    "svelte": "^4.2.11",
-    "typescript": "^5.3.3",
-    "vite": "^5.1.4",
-    "vitest": "~1.3.1",
-    "shared-config": "1.0.0-next.92",
-    "validators": "1.0.0-next.92"
+    "svelte": "^4.2.12",
+    "typescript": "^5.4.4",
+    "vite": "^5.2.8",
+    "vitest": "~1.4.0",
+    "shared-config": "1.0.0-next.94",
+    "validators": "1.0.0-next.94"
   },
   "files": [
     "src/**/*.js",
-    "src/**/*.svelte",
-    "!src/mocks",
-    "!src/**/*.spec.js"
+    "src/**/*.svelte"
   ],
   "exports": {
     "./src": "./src",
@@ -42,13 +40,12 @@
   },
   "dependencies": {
     "ramda": "^0.29.1",
-    "@rokkit/core": "1.0.0-next.92",
-    "@rokkit/stores": "1.0.0-next.92"
+    "@rokkit/core": "1.0.0-next.94",
+    "@rokkit/stores": "1.0.0-next.94"
   },
   "scripts": {
     "format": "prettier --write .",
     "lint": "eslint --fix .",
-    "test:ct": "playwright test -c playwright.config.js",
     "test:ci": "vitest run",
     "test:ui": "vitest --ui",
     "test": "vitest",

package/src/fillable.js

@@ -10,7 +10,7 @@ export function fillable(node, { options, current, check }) {
 	let blanks = node.getElementsByTagName('del')
 
 	function click(event) {
-		if ('?' !== event.target.innerHTML) {
+		if (event.target.innerHTML !== '?') {
 			clear(event, node)
 		}
 	}
@@ -58,7 +58,7 @@ function initialize(blanks, click) {
  */
 function fill(blanks, options, current) {
 	if (current > -1 && current < Object.keys(blanks).length) {
-		let index = options.findIndex(({ actualIndex }) => actualIndex == current)
+		let index = options.findIndex(({ actualIndex }) => actualIndex === current)
 		if (index > -1) {
 			blanks[current].innerHTML = options[index].value
 			blanks[current].classList.remove('empty')
@@ -96,11 +96,11 @@ function clear(event, node) {
  * @param {import('./types').FillOptions} data
  */
 function validate(blanks, data) {
-	Object.keys(blanks).map((ref) => {
-		let index = data.options.findIndex(({ actualIndex }) => actualIndex == ref)
+	Object.keys(blanks).map((_, ref) => {
+		let index = data.options.findIndex(({ actualIndex }) => actualIndex === ref)
 		if (index > -1)
 			blanks[ref].classList.add(
-				data.options[index].expectedIndex == data.options[index].actualIndex ? 'pass' : 'fail'
+				data.options[index].expectedIndex === data.options[index].actualIndex ? 'pass' : 'fail'
 			)
 	})
 }

package/src/hierarchy.js

@@ -80,7 +80,7 @@ export function movePrevious(path) {
 
 	const current = path[path.length - 1]
 
-	if (current.index == 0) {
+	if (current.index === 0) {
 		if (path.length > 1) path.pop()
 		return path
 	}

package/src/lib/viewport.js

@@ -38,13 +38,15 @@ export function virtualListViewport(options) {
 
 		if (items.length !== cache.length) {
 			cache = Array.from({ length: items.length }).fill(null)
-			if (items.length == 0) index = -1
+			if (items.length === 0) index = -1
 		}
 		current = { lower: data.start, upper: data.end }
 
 		cache = updateSizes(cache, data.sizes ?? [], current.lower)
 		averageSize =
-			cache.length == 0 ? minSize : calculateSum(cache, 0, cache.length, averageSize) / cache.length
+			cache.length === 0
+				? minSize
+				: calculateSum(cache, 0, cache.length, averageSize) / cache.length
 
 		let visible = calculateSum(cache, current.lower, current.upper, averageSize, gap)
 

package/src/navigator.js

@@ -100,9 +100,9 @@ export function navigator(element, options) {
 			: target.dataset.path
 					.split(',')
 					.filter((item) => item !== '')
-					.map((item) => +item)
+					.map((item) => Number(item))
 
-		if (indices.length > 0 && event.target.tagName != 'DETAIL') {
+		if (indices.length > 0 && event.target.tagName !== 'DETAIL') {
 			path = pathFromIndices(indices, items, fields)
 			currentNode = getCurrentNode(path)
 			if (hasChildren(currentNode, path[path.length - 1].fields)) {

package/src/switchable.js

@@ -1,3 +1,5 @@
+import { removeListeners, setupListeners } from './lib'
+
 export function switchable(node, data) {
 	let index = 0
 	let { value, options, disabled } = data
@@ -10,14 +12,12 @@ export function switchable(node, data) {
 	}
 
 	const toggle = (increment = 1) => {
-		if (disabled) return
 		index = (index + increment) % options.length
 		value = options[index]
 		node.dispatchEvent(new CustomEvent('change', { detail: value }))
 	}
 
 	const keydown = (e) => {
-		if (disabled) return
 		if ([' ', 'Enter', 'ArrowRight', 'ArrowLeft'].includes(e.key)) {
 			e.preventDefault()
 			e.stopPropagation()
@@ -25,17 +25,16 @@ export function switchable(node, data) {
 			toggle(e.key === 'ArrowLeft' ? options.length - 1 : 1)
 		}
 	}
-	const click = () => toggle(1)
+	const listeners = {
+		click: () => toggle(1),
+		keydown
+	}
 
 	update(data)
-	node.addEventListener('click', click)
-	node.addEventListener('keydown', keydown)
+	setupListeners(node, listeners, { enabled: !disabled })
 
 	return {
 		update,
-		destroy() {
-			node.removeEventListener('click', click)
-			node.removeEventListener('keydown', keydown)
-		}
+		destroy: () => removeListeners(node, listeners)
 	}
 }

package/src/traversable.js

@@ -37,7 +37,7 @@ export function traversable(element, options) {
 	}
 
 	const checkAndEmit = (event) => {
-		if (current && current.item) {
+		if (current?.item) {
 			emit(element, event, pick(['item', 'position'], current))
 		}
 	}
@@ -97,7 +97,7 @@ export function traversable(element, options) {
 // }
 
 // function scrollIntoView(element, position) {
-// 	if (!Array.isArray(position) || position.length == 0) return
+// 	if (!Array.isArray(position) || position.length === 0) return
 // 	const node = element.querySelector(`[data-index="${position.join(',')}"]`)
 // 	if (node) node.scrollIntoView()
 // }