@joewinke/jatui 0.2.31 → 0.2.32

package/dist/actions/railNav.js

@@ -84,6 +84,16 @@ export function cycle(list, current, dir) {
 const BUMP_DURATION_MS = 280;
 const RAIL_FOCUS_CLASS = 'rail-focus';
 const RAIL_CYCLED_CLASS = 'rail-cycled';
+/** `<input>` types whose vertical arrows natively step the value (spinner / picker). */
+const STEPPER_INPUT_TYPES = new Set([
+    'number',
+    'range',
+    'date',
+    'time',
+    'month',
+    'week',
+    'datetime-local',
+]);
 function resolveOrder(opts) {
     return typeof opts.order === 'function' ? opts.order() : opts.order;
 }
@@ -213,6 +223,13 @@ function makeController(optsRef) {
         const isTextField = !!el &&
             (tag === 'INPUT' || tag === 'TEXTAREA' || el.isContentEditable === true);
         const isMultiline = !!el && (tag === 'TEXTAREA' || el.isContentEditable === true);
+        const inputType = tag === 'INPUT' ? (el.type || '').toLowerCase() : '';
+        // A <select> cycles its value with arrows on BOTH axes, and a radio moves
+        // within its `name` group on both axes — the rail must yield entirely or it
+        // silently drives an unrelated rung while the user thinks they're picking a value.
+        const ownsBothArrowAxes = tag === 'SELECT' || inputType === 'radio';
+        const blocksVerticalArrows = isMultiline || ownsBothArrowAxes || STEPPER_INPUT_TYPES.has(inputType);
+        const blocksHorizontalArrows = isTextField || ownsBothArrowAxes;
         const hasMod = event.metaKey || event.ctrlKey || event.altKey;
         // Modal trap — stop non-text keystrokes from bubbling to window-level page
         // listeners (e.g., a listNav attached to `window`). Text-field keystrokes
@@ -245,13 +262,13 @@ function makeController(optsRef) {
         const popoverOpen = opts.isPopoverOpen?.() ?? false;
         if (!hasMod && !popoverOpen) {
             // Vertical: ArrowDown / j step down; ArrowUp / k step up.
-            // j / k are blocked in ANY text field; arrows are blocked only inside
-            // multiline fields so the caret can still move line-by-line.
+            // j / k are blocked in ANY text field (they'd type a letter); arrows are
+            // blocked wherever the control owns them natively.
             const goDown = event.key === 'ArrowDown' || event.key === 'j';
             const goUp = event.key === 'ArrowUp' || event.key === 'k';
             if (goDown || goUp) {
                 const isVimKey = event.key === 'j' || event.key === 'k';
-                const blocked = isVimKey ? isTextField : isMultiline;
+                const blocked = isVimKey ? isTextField : blocksVerticalArrows;
                 if (!blocked) {
                     event.preventDefault();
                     moveRung(goDown ? 1 : -1);
@@ -261,7 +278,7 @@ function makeController(optsRef) {
             // Horizontal: ArrowLeft / ArrowRight dispatch to `onAction`.
             // Skipped inside text fields so caret-left / caret-right keep working.
             if ((event.key === 'ArrowLeft' || event.key === 'ArrowRight') &&
-                !isTextField) {
+                !blocksHorizontalArrows) {
                 event.preventDefault();
                 const attr = resolveAttr(opts);
                 // Prefer the element that actually has DOM focus, then fall back to

package/dist/components/VideoPlayer.svelte

@@ -122,6 +122,7 @@
 		captionsSrc = null,
 		captionsLabel = 'Captions',
 		captionsLang = 'en',
+		captionsDefault = true,
 		chapters = [],
 		speakers = [],
 		transcript = [],
@@ -159,6 +160,21 @@
 		captionsSrc?: string | null;
 		captionsLabel?: string;
 		captionsLang?: string;
+		/**
+		 * Should the captions <track> start ENABLED (`default` attribute)?
+		 *
+		 * `true` (default) preserves the historical behaviour: a track with a src
+		 * shows on load. Pass `false` when the video already has captions burned
+		 * into its frames (e.g. an academy episode render) — the track is still
+		 * present and toggleable from the native menu, but not painted on top of
+		 * the burned-in text, which would double every caption.
+		 *
+		 * Long-term direction (see the double-caption bug, jat-abwim): renders
+		 * should STOP burning captions into the frames and rely on this
+		 * toggleable VTT track alone. Until episodes are re-rendered without
+		 * burned-in text, a burned-in episode passes `captionsDefault={false}`.
+		 */
+		captionsDefault?: boolean;
 		chapters?: VideoChapter[];
 		speakers?: VideoSpeakerTurn[];
 		transcript?: VideoTranscriptSegment[];
@@ -1052,8 +1068,12 @@
 				class="vp-video"
 			>
 				<!-- A captions track is always present for a11y; without a src it is
-				     an empty, inert track rather than a missing affordance. -->
-				<track kind="captions" src={captionsSrc ?? undefined} srclang={captionsLang} label={captionsLabel} default={!!captionsSrc} />
+				     an empty, inert track rather than a missing affordance. It starts
+				     ENABLED only when there's a src AND the host opts in via
+				     captionsDefault (true by default). A burned-in-captions video
+				     passes captionsDefault={false} so the track stays available in the
+				     menu but isn't painted on top of the frames' own text (jat-abwim). -->
+				<track kind="captions" src={captionsSrc ?? undefined} srclang={captionsLang} label={captionsLabel} default={!!captionsSrc && captionsDefault} />
 			</video>
 			<div class="vp-osd" class:on={osdOn} aria-hidden="true">{osd}</div>
 		</div>

package/dist/components/VideoPlayer.svelte.d.ts

@@ -68,6 +68,21 @@ type $$ComponentProps = {
     captionsSrc?: string | null;
     captionsLabel?: string;
     captionsLang?: string;
+    /**
+     * Should the captions <track> start ENABLED (`default` attribute)?
+     *
+     * `true` (default) preserves the historical behaviour: a track with a src
+     * shows on load. Pass `false` when the video already has captions burned
+     * into its frames (e.g. an academy episode render) — the track is still
+     * present and toggleable from the native menu, but not painted on top of
+     * the burned-in text, which would double every caption.
+     *
+     * Long-term direction (see the double-caption bug, jat-abwim): renders
+     * should STOP burning captions into the frames and rely on this
+     * toggleable VTT track alone. Until episodes are re-rendered without
+     * burned-in text, a burned-in episode passes `captionsDefault={false}`.
+     */
+    captionsDefault?: boolean;
     chapters?: VideoChapter[];
     speakers?: VideoSpeakerTurn[];
     transcript?: VideoTranscriptSegment[];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@joewinke/jatui",
-  "version": "0.2.31",
+  "version": "0.2.32",
   "private": false,
   "description": "Shared Svelte 5 component library for JAT projects",
   "type": "module",