tosijs-ui 1.16.4 → 1.16.6

package/CHANGELOG.md

@@ -1,5 +1,85 @@
 # Changelog
 
+## 1.16.6
+
+A one-fix patch: a link to the page you are already on no longer adds a history entry that Back
+has to step through (#174). No API change.
+
+### Clicking a link to the page you are on no longer costs an extra Back (#174)
+
+A doc link pointing at the document already showing pushed a duplicate history entry, so the
+next Back press left the page where it was. It now replaces the entry, as a native same-URL
+navigation does. "The same page" means the same document: `/x`, `/x/` and `/x/index.html`, or
+`?x` with extra query parameters, all count. A URL with a hash still pushes, so Back from
+`/x/` returns to `/x/#example`. The `router`'s `navigate()` had the same defect and gets the
+same rule: navigating to the current path (with no hash) replaces.
+
+The double push #174 measured on every link (ensemble on 1.14.1) does not reproduce on 1.16.4
+with the same direct import: one click, one entry, and Back and Forward both work, in Chromium
+and WebKit. The documented consumer entry point is `import 'tosijs-ui/doc-browser'`, which
+registers `<tosi-doc-system>` since 1.14.2. The deep `tosijs-ui/doc-system/doc-system.js`
+import loads the same module.
+
+## 1.16.5
+
+A patch, mostly found by using the doc site on an iPhone: menus that stay on screen as Safari's
+toolbars come and go, a code panel that fits the visible screen, and a REPL that completes as
+you type (touchably, case-insensitively, with `$` and `$$`). Also quiet under tosijs 1.10.7's
+long-list advice, and `pinnedTopRows` now works from the element creator. `dist/iife.js` grows
+by 3.6 kB gzip (478.7 → 482.2 kB), including the move to tosijs 1.10.7, which it bundles.
+
+**One read-side change to check for:** `<tosi-table>`'s `pinnedTopRows` and `pinnedBottomRows`
+now read as `null` rather than `undefined` when unset (typed `any[] | null`). That is what makes
+them work when passed to the element creator. Code that tested `table.pinnedTopRows ===
+undefined` should test `== null` instead; truthiness checks are unaffected.
+
+### `$` and `$$` in the REPL
+
+As in a browser console: `$('.thing')` returns the first match and `$$('.thing')` an array of all
+of them, within the example's own preview. An example's own `$`, or a page's jQuery, takes
+precedence.
+
+### The Console tab's REPL completes as you type
+
+Typing a name, or a dot after a value, opens a list of suggestions above the prompt: the
+value's properties (prototype chain included) or the names in scope (the example's own,
+`preview`, `console`, the page's modules, globals). Tap one, or use ↓/↑ and Enter; Tab takes the
+highlighted or first suggestion, and Escape closes the list. Rows are touch-sized, so it works
+on a phone, where there is no Tab key and no browser console at all. Matching ignores case, and
+the field turns off iOS's auto-capitalisation and autocorrect; the example's own names rank
+above globals.
+
+### Quiet under tosijs 1.10.7's long-list advice; `pinnedTopRows` works from the creator
+
+tosijs 1.10.7 warns once about any list binding of more than 10 items that has neither
+`virtual` nor a `nonVirtualReason`. tosijs-ui builds some lists its users can't reach, so it now
+gives the reason itself:
+
+- `<tosi-table>`'s pinned rows, which are always on screen;
+- its body under `rowHeight: 0`, which is the author choosing to render every row (the table's
+  own warning still fires past 1,000 rows);
+- the doc browser's flat nav in `routing: 'query'`.
+
+While testing this, `tosiTable({ pinnedTopRows: [...] })` (and `pinnedBottomRows`) turned out to
+be silently ignored when passed to the element creator: the getter returned `undefined` when
+unset, so tosijs set an attribute instead. It now returns `null` when unset, and the creator
+works (see the note at the top of these notes). The dev dependency moves to tosijs 1.10.7; the peer range is unchanged (`^1.10.3`).
+
+### Menus stay inside the screen on iPhone as Safari's toolbars come and go
+
+A menu's room was measured once, when it opened. Mobile Safari then shows and hides its toolbars
+(and the keyboard comes and goes) as you touch the page, which changes the visible area without
+a window resize, so a menu measured a moment earlier ran under the toolbar. Open floats (menus,
+tooltips, hint lists) are now re-fitted to their anchor whenever the visible viewport changes.
+Draggable floats are left where they are. Reported by the maintainer on an iPhone (the other
+half of #2460).
+
+### The Console tab's REPL is reachable on iPhone
+
+A full-screen example (the code panel, or maximized) was sized `100vh`, which on iPhone Safari is
+the height without the browser toolbar, so the bottom of the panel sat under it, and that is
+where the Console tab's REPL input lives. It is now sized to the visible viewport (`100dvh`).
+
 ## 1.16.4
 
 A patch: the example console becomes a tab with a REPL, test-only examples render on deployed

package/dist/data-table.d.ts

@@ -243,10 +243,10 @@ export declare class TosiTable extends TosiTable_base {
     private _nonRepeatingGroupedRowCells;
     private _pinnedTopRows?;
     private _pinnedBottomRows?;
-    get pinnedTopRows(): any[] | undefined;
-    set pinnedTopRows(rows: any[] | undefined);
-    get pinnedBottomRows(): any[] | undefined;
-    set pinnedBottomRows(rows: any[] | undefined);
+    get pinnedTopRows(): any[] | null;
+    set pinnedTopRows(rows: any[] | null | undefined);
+    get pinnedBottomRows(): any[] | null;
+    set pinnedBottomRows(rows: any[] | null | undefined);
     get effectivePinnedTopData(): any[];
     get effectivePinnedBottomData(): any[];
     private get effectiveBaseData();

package/dist/data-table.js

@@ -1578,15 +1578,20 @@ export class TosiTable extends withAttributes({
     // count-based slicing.
     _pinnedTopRows;
     _pinnedBottomRows;
+    /*
+    `null`, not `undefined`, when unset: tosijs's element creator assigns a prop as a PROPERTY only
+    when the instance's current value isn't `undefined`, and otherwise as an attribute — so
+    `tosiTable({ pinnedTopRows: [...] })` was silently ignored (tosijs board #2464).
+    */
     get pinnedTopRows() {
-        return this._pinnedTopRows;
+        return this._pinnedTopRows ?? null;
     }
     set pinnedTopRows(rows) {
         this._pinnedTopRows = rows ? tosiValue(rows) : undefined;
         this.queueRender();
     }
     get pinnedBottomRows() {
-        return this._pinnedBottomRows;
+        return this._pinnedBottomRows ?? null;
     }
     set pinnedBottomRows(rows) {
         this._pinnedBottomRows = rows ? tosiValue(rows) : undefined;
@@ -2364,7 +2369,11 @@ export class TosiTable extends withAttributes({
         the visible data, so they have no parity, and blanking their non-repeating cells
         would empty a pinned header/summary row purely because it is not in `firstRows`.
         */
-        (_elements, item) => this.buildRow(item, cols, stickyInfo, rowClass, false), {});
+        (_elements, item) => this.buildRow(item, cols, stickyInfo, rowClass, false), 
+        // tosijs warns about long unvirtualised lists (1.10.7); the author can't reach this one
+        {
+            nonVirtualReason: 'pinned rows are always on screen, so every one is rendered (a handful by design)',
+        });
         return div({
             class: `tbody tbody-${region}`,
             role: 'rowgroup',
@@ -3338,7 +3347,17 @@ export class TosiTable extends withAttributes({
         // The visible-rows listBinding is bound directly to .scroll-area so
         // virtualisation observes the same scroll container that sticky cells
         // stick against.
-        const visibleBinding = this.rowData.visible.listBinding((_elements, item) => this.buildRow(item, cols, stickyInfo), this.rowHeight > 0 ? { virtual: { height: this.rowHeight } } : {});
+        const visibleBinding = this.rowData.visible.listBinding((_elements, item) => this.buildRow(item, cols, stickyInfo), this.rowHeight > 0
+            ? { virtual: { height: this.rowHeight } }
+            : /*
+              `rowHeight: 0` IS the author saying "render every row" (documented for small tables
+              and variable row heights), so say so to tosijs rather than have it warn about a choice
+              already made. A table big enough for that to hurt still gets <tosi-table>'s own
+              warning past NON_VIRTUAL_ROW_ADVICE rows (#84).
+              */
+                {
+                    nonVirtualReason: 'rowHeight: 0 — the author chose to render every row (<tosi-table> warns past 1,000)',
+                });
         this._scrollArea = div({ class: 'scroll-area', part: 'visibleRows' }, ...[
             this._head,
             this._tbodyTop,

package/dist/doc-browser.js

@@ -1027,7 +1027,19 @@ export function createDocBrowser(options) {
         }
         else {
             const href = hrefFor(filename);
-            window.history.pushState({ href }, '', href);
+            /*
+            A link to the document already showing REPLACES the entry rather than pushing one, as a
+            native same-URL navigation does. Pushing left a duplicate the reader had to Back
+            through, and Back then visibly did nothing — the symptom #174 reported.
+      
+            "Same" is the DOCUMENT, the identity the popstate guard uses, not the raw path: `/x`,
+            `/x/` and `/x/index.html`, or `?x` and `?x&_testMode=1`, are one page. A URL carrying a
+            hash still pushes, because leaving `/x/#example` for `/x/` is a navigation the reader
+            may want to come Back from.
+            */
+            const same = filename === String(app.currentDoc.filename) &&
+                document.location.hash === '';
+            window.history[same ? 'replaceState' : 'pushState']({ href }, '', href);
         }
         navigateTo(filename);
     };
@@ -1490,6 +1502,9 @@ export function createDocBrowser(options) {
                 idPath: 'filename',
                 hiddenProp: 'hidden',
                 value: app.docs,
+                // the flat nav (query routing) lists every doc so the search can filter it and a
+                // reader can scan it; the site's author can't reach this binding to say so
+                nonVirtualReason: 'doc navigation: every doc is listed so search can filter it',
             },
         }, template(a({
             class: 'doc-link',

package/dist/float.d.ts

@@ -1,4 +1,5 @@
 import { ElementCreator } from 'tosijs';
+import type { FloatPosition } from './pop-float.js';
 declare const TosiFloat_base: import("tosijs").WithAttributes<{
     drag: boolean;
     remainOnResize: "hide" | "remove" | "remain";
@@ -12,6 +13,11 @@ export declare class TosiFloat extends TosiFloat_base {
      * (the page, or a scroller containing it) triggers `remainOnScroll`. `null`: any scroll does.
      */
     anchor: Element | null;
+    /**
+     * The position it was asked for against its anchor (set by `positionFloat`), so it can be
+     * re-fitted when the visible viewport changes. `null`: never positioned against an anchor.
+     */
+    anchorPosition: FloatPosition | null;
     content: HTMLSlotElement;
     static shadowStyleSpec: {
         ':host': {

package/dist/float.js

@@ -106,6 +106,13 @@ export class TosiFloat extends withAttributes({
      * (the page, or a scroller containing it) triggers `remainOnScroll`. `null`: any scroll does.
      */
     anchor = null;
+    /**
+     * The position it was asked for against its anchor (set by `positionFloat`), so it can be
+     * re-fitted when the visible viewport changes. `null`: never positioned against an anchor.
+     */
+    // typed FloatPosition, not string: as `string` it accepted the CSS position '' / 'fixed'
+    // that a shadowed variable stored there (1.16.5 review B1), and the compiler said nothing
+    anchorPosition = null;
     content = slot();
     static shadowStyleSpec = {
         ':host': {