@lavalogic/scoria 0.37.9 → 0.37.11

package/dist/Components/Table/Body/TableBody.svelte

@@ -205,6 +205,7 @@
 .virtual-spacer {
   display: block;
   border: none;
-  background: transparent;
   pointer-events: none;
+  --row-stripe-height: var(--height, 3rem);
+  background-image: repeating-linear-gradient(to bottom, #ffffff 0, #ffffff var(--row-stripe-height), #f4f7f9 var(--row-stripe-height), #f4f7f9 calc(2 * var(--row-stripe-height)));
 }</style>

package/dist/Components/Table/ColumnFactory.svelte.js

@@ -67,14 +67,48 @@ function makeSpec(kind, def, internals = {}) {
     });
     return Object.freeze(spec);
 }
+/** Sensible per-kind default widths so a freshly-built table without explicit
+ *  per-column `width` overrides ends up visually differentiated (rather than
+ *  every column collapsing to the shared `minSize` fallback of 48px and then
+ *  taking equal `1fr` shares of any remaining track space). Numbers chosen
+ *  to roughly match what the legacy `<table>` element produced via the
+ *  browser auto-layout algorithm for typical FloWMS column content. */
+const KIND_DEFAULT_WIDTHS = {
+    text: 200,
+    number: 100,
+    checkbox: 80,
+    date: 140,
+    select: 160,
+    query: 160,
+    display: 150,
+    bubble: 150,
+    progress: 140,
+    validity: 60,
+    actions: 120,
+    rowSelection: 40,
+    expand: 40,
+    filterOnly: 0,
+};
 /** Returns the common Def props for an option object, excluding `id`.
  *  Callers pass `id` directly at the construction site so its strict
- *  literal-key narrowing survives the object spread. */
-function commonProps(opts) {
+ *  literal-key narrowing survives the object spread.
+ *
+ *  `kind` selects the per-kind default width from `KIND_DEFAULT_WIDTHS`. The
+ *  caller's `minWidth` / `width` values still win - the default is only
+ *  applied if the consumer left the field undefined. */
+function commonProps(opts, kind) {
+    const defaultWidth = KIND_DEFAULT_WIDTHS[kind];
     return {
         header: opts.header,
-        minSize: opts.minWidth,
-        width: opts.width,
+        // Set minSize from the per-kind default too so
+        // `TableContext.registerDefs` does NOT run the canvas-based
+        // `measureHeaderWidth` fallback. That fallback floors every
+        // accessor column to ~130px when quick-filtering is enabled
+        // (it returns max(headerWidth, "Lorem Ipsum Dolor" width)),
+        // which equalises every accessor column width and erases the
+        // per-kind differentiation the factory just put in place.
+        minSize: opts.minWidth ?? defaultWidth,
+        width: opts.width ?? defaultWidth,
         allowSorting: opts.allowSorting,
         allowFiltering: opts.allowFiltering,
         defaultHidden: opts.defaultHidden,
@@ -123,7 +157,7 @@ export function createColumnFactory() {
         text(opts) {
             const def = new TextInputDef({
                 id: opts.id,
-                ...commonProps(opts),
+                ...commonProps(opts, 'text'),
                 accessorFn: opts.accessor,
                 getDisplayValue: opts.getDisplayValue,
                 filterValueType: FilterValueType.String,
@@ -136,7 +170,7 @@ export function createColumnFactory() {
         number(opts) {
             const def = new NumberInputDef({
                 id: opts.id,
-                ...commonProps(opts),
+                ...commonProps(opts, 'number'),
                 accessorFn: opts.accessor,
                 getDisplayValue: opts.getDisplayValue,
                 min: opts.min ?? Number.MIN_SAFE_INTEGER,
@@ -151,7 +185,7 @@ export function createColumnFactory() {
         checkbox(opts) {
             const def = new CheckboxDef({
                 id: opts.id,
-                ...commonProps(opts),
+                ...commonProps(opts, 'checkbox'),
                 accessorFn: opts.accessor,
                 getDisplayValue: opts.getDisplayValue,
                 filterValueType: FilterValueType.Bool,
@@ -164,7 +198,7 @@ export function createColumnFactory() {
         date(opts) {
             const def = new DateInputDef({
                 id: opts.id,
-                ...commonProps(opts),
+                ...commonProps(opts, 'date'),
                 accessorFn: opts.accessor,
                 getDisplayValue: opts.getDisplayValue,
                 showTime: opts.withTime ?? false,
@@ -178,7 +212,7 @@ export function createColumnFactory() {
         select(opts) {
             const def = new SelectDef({
                 id: opts.id,
-                ...commonProps(opts),
+                ...commonProps(opts, 'select'),
                 accessorFn: opts.accessor,
                 getOptions: normaliseOptionsLoader(opts.options),
                 getDisplayValue: opts.getDisplayValue ?? ((row) => String(row[opts.id] ?? '')),
@@ -199,7 +233,7 @@ export function createColumnFactory() {
         query(opts) {
             const def = new QueryDef({
                 id: opts.id,
-                ...commonProps(opts),
+                ...commonProps(opts, 'query'),
                 accessorFn: opts.accessor,
                 // Forward the row to the consumer's query callback so it
                 // may scope results by row-level context (e.g. a `pod`
@@ -224,7 +258,7 @@ export function createColumnFactory() {
                 : undefined;
             const def = new DisplayDef({
                 id,
-                ...commonProps(opts),
+                ...commonProps(opts, 'display'),
                 accessorFn: opts.value,
                 modal: opts.modal,
                 // Display columns have no implicit filter type (unlike the
@@ -241,7 +275,7 @@ export function createColumnFactory() {
             const id = mintSyntheticId('bubble', opts.idHint);
             const def = new BubbleDef({
                 id,
-                ...commonProps(opts),
+                ...commonProps(opts, 'bubble'),
                 accessorFn: opts.value,
                 getColour: opts.colour,
                 filterValueType: opts.filterValueType,
@@ -267,7 +301,7 @@ export function createColumnFactory() {
                 : () => 'grey';
             const def = new ProgressBarDef({
                 id,
-                ...commonProps(opts),
+                ...commonProps(opts, 'progress'),
                 accessorFn: tupleAccessor,
                 getColour,
                 filterValueType: opts.filterValueType,
@@ -280,8 +314,8 @@ export function createColumnFactory() {
             const def = new ValidityDef({
                 id,
                 header: opts?.header ?? '',
-                minSize: opts?.minWidth,
-                width: opts?.width,
+                minSize: opts?.minWidth ?? KIND_DEFAULT_WIDTHS.validity,
+                width: opts?.width ?? KIND_DEFAULT_WIDTHS.validity,
                 defaultHidden: opts?.defaultHidden,
                 resizable: opts?.resizable ?? false,
                 debug: opts?.debug,
@@ -295,6 +329,11 @@ export function createColumnFactory() {
             const def = new RowSelectionDef({
                 isSelectable: opts?.isSelectable ?? (() => true),
             });
+            // RowSelection has no public width option - apply the per-kind
+            // default so the column does not collapse to the 48px ColumnDef
+            // fallback and share equal `1fr` track space with wider columns.
+            def.width = KIND_DEFAULT_WIDTHS.rowSelection;
+            def.minSize = KIND_DEFAULT_WIDTHS.rowSelection;
             return makeSpec('rowSelection', def);
         },
         actions(opts) {
@@ -317,6 +356,8 @@ export function createColumnFactory() {
                 actions: wrapped,
                 isEnabled: isEnabledFn ? (item) => isEnabledFn(item) : undefined,
             });
+            // Actions has no public width option - apply the per-kind default.
+            def.width = KIND_DEFAULT_WIDTHS.actions;
             return makeSpec('actions', def);
         },
         expand(opts) {
@@ -359,6 +400,9 @@ export function createColumnFactory() {
                     allowCopy: opts.inner?.allowCopy,
                 },
             });
+            // Expand has no public width option - apply the per-kind default.
+            def.width = KIND_DEFAULT_WIDTHS.expand;
+            def.minSize = KIND_DEFAULT_WIDTHS.expand;
             return makeSpec('expand', def);
         },
         filterOnly(opts) {

package/dist/Components/Table/Table.svelte

@@ -105,8 +105,19 @@
 		untrack(() => {
 			if (tableContext.focusStart && ref) {
 				const { row, column: index } = tableContext.focusStart;
+				// `columnDefs[index]` can be undefined when the focus
+				// coordinate references a memory slot that no longer
+				// exists (e.g. inner expand tables remount with a
+				// different column count, or a preset hydration races
+				// the column-layout init). Guard before destructuring
+				// so an out-of-range index degrades to "no focus
+				// restored" instead of a hard crash.
+				const def = tableContext.columnDefs[index];
+				if (!def) {
+					return;
+				}
 				let element = ref.querySelector(
-					`#${tableContext.tableName}-cell-${row}-${tableContext.columnDefs[index].id}`
+					`#${tableContext.tableName}-cell-${row}-${def.id}`
 				) as HTMLElement | undefined;
 
 				if (element?.tabIndex == -1) {
@@ -136,8 +147,13 @@
 		untrack(() => {
 			if (tableContext.focusEnd && ref) {
 				const { row, column: index } = tableContext.focusEnd;
+				// Same out-of-range guard as the focusStart effect above.
+				const def = tableContext.columnDefs[index];
+				if (!def) {
+					return;
+				}
 				const cell = ref.querySelector(
-					`#${tableContext.tableName}-cell-${row}-${tableContext.columnDefs[index].id}`
+					`#${tableContext.tableName}-cell-${row}-${def.id}`
 				) as HTMLElement | undefined;
 
 				cell?.scrollIntoView({

package/dist/Components/Table/Types/Context/VirtualisationState.svelte.js

@@ -50,7 +50,17 @@ export class VirtualisationState {
     constructor(deps, options = {}) {
         this._deps = deps;
         this._virtualisationThreshold = options.threshold ?? 100;
-        this._overscan = options.overscan ?? 10;
+        // Overscan controls how many rows above/below the viewport are
+        // kept rendered so a fast wheel-scroll does not surface an empty
+        // grey band before the next paint. The previous default of 10
+        // was too tight on tables with hundreds of rows: a single wheel
+        // flick easily moves the viewport past the overscan band before
+        // the `$derived` chain reschedules and re-renders the visible
+        // slice. 30 rows is enough to absorb a typical wheel flick at
+        // 60Hz while still keeping the rendered cell count bounded
+        // (~52 rows for a 600px viewport at the default 28-48px row
+        // height: 12 visible + 30 below + 10 above = 52).
+        this._overscan = options.overscan ?? 30;
     }
     get virtualisationThreshold() {
         return this._virtualisationThreshold;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lavalogic/scoria",
   "description": "Svelte components used for the FloWMS Web Frontend",
-  "version": "0.37.9",
+  "version": "0.37.11",
   "publishConfig": {
     "access": "restricted"
   },