@gjsify/adwaita-web 0.32.0 → 0.33.0

package/src/split-views.spec.ts

@@ -12,12 +12,31 @@
 // sidebar where GTK shows one.
 import { describe, expect, it } from '@gjsify/unit';
 
-import { SIDEBAR_BOUNDS_VECTORS } from '@gjsify/adwaita-core/conformance';
-import { OVERLAY_COLLAPSE_VECTORS } from '@gjsify/adwaita-core/conformance';
+import type { OverlaySplitViewState } from '@gjsify/adwaita-core';
+
+import {
+    OVERLAY_COLLAPSE_VECTORS,
+    OVERLAY_SWIPE_CANCEL_VECTORS,
+    OVERLAY_SWIPE_RELEASE_VECTORS,
+    OVERLAY_SWIPE_SNAP_POINT_VECTORS,
+    OVERLAY_SWIPE_START_VECTORS,
+    NAVIGATION_ACTION_VECTORS,
+    OVERLAY_SWIPE_AREA_VECTORS,
+    OVERLAY_SPLIT_VIEW_LAYOUT_VECTORS,
+    NAVIGATION_STACK_VECTORS,
+    SIDEBAR_BOUNDS_VECTORS,
+} from '@gjsify/adwaita-core/conformance';
 
 import type { AdwNavigationSplitView } from './elements/adw-navigation-split-view.js';
 import type { AdwOverlaySplitView } from './elements/adw-overlay-split-view.js';
 
+/** Wait for a ResizeObserver delivery (it runs after layout, before paint). */
+function settle(): Promise<void> {
+    return new Promise((resolve) => {
+        requestAnimationFrame(() => requestAnimationFrame(() => resolve()));
+    });
+}
+
 /** Mount a split view from markup and return it. */
 function mount(attrs = ''): { view: AdwOverlaySplitView; host: HTMLElement } {
     const host = document.createElement('div');
@@ -306,4 +325,569 @@ export const AdwSplitViewsTest = async () => {
             host.remove();
         });
     });
+
+    await describe('<adw-overlay-split-view> reveal is a CONTINUUM, not two states', async () => {
+        /** Mount a view of a known width so the geometry is predictable. */
+        function mountSizedView(width: number, attrs = ''): { view: AdwOverlaySplitView; host: HTMLElement } {
+            const { view, host } = mount(attrs);
+            host.style.width = `${width}px`;
+            host.style.height = '400px';
+            (view as unknown as HTMLElement).style.height = '400px';
+            return { view, host };
+        }
+
+        /** The state the element composes — the same object the vectors describe. */
+        const stateOf = (view: AdwOverlaySplitView) => (view as unknown as { _state: OverlaySplitViewState })._state;
+
+        await it('drives show-progress through the whole range, not 0 and 1', async () => {
+            const { view, host } = mountSizedView(800);
+            const state = stateOf(view);
+            const sidebar = view.querySelector('.adw-osv-sidebar') as HTMLElement;
+            await settle();
+
+            const offsets = new Set<string>();
+            for (const progress of [0, 0.25, 0.5, 0.75, 1]) {
+                state.setShowProgress(progress);
+                offsets.add(sidebar.style.marginLeft || sidebar.style.left || '');
+            }
+            // Two end states would give at most two distinct offsets; the five
+            // OVERLAY_SWIPE_* tables need every value in between to mean something.
+            expect(offsets.size).toBeGreaterThan(2);
+            host.remove();
+        });
+
+        for (const vector of OVERLAY_SWIPE_SNAP_POINT_VECTORS) {
+            await it(`snap points [${vector.snapPoints.join(', ')}] — ${vector.rule}`, async () => {
+                const attrs =
+                    (vector.enableShowGesture ? '' : 'enable-show-gesture="false" ') +
+                    (vector.enableHideGesture ? '' : 'enable-hide-gesture="false"');
+                const { view, host } = mountSizedView(800, attrs);
+                const state = stateOf(view);
+                await settle();
+                state.setShowProgress(vector.showProgress);
+                if (vector.swipeActive) state.beginSwipe();
+                expect(state.snapPoints).toStrictEqual([...vector.snapPoints]);
+                host.remove();
+            });
+        }
+
+        await it('grabs only where the swipe area is — the ADW_SWIPE_BORDER strip when closed', async () => {
+            // The row this probes: closed, start edge, LTR — the grabbable strip
+            // is `ADW_SWIPE_BORDER` wide and nothing outside it starts a gesture.
+            // What a DOM renderer owns is that second half, which no unit test of
+            // `resolveSwipeArea` can show.
+            const closedAtStart = OVERLAY_SWIPE_AREA_VECTORS.find(
+                (vector) =>
+                    vector.isDrag &&
+                    vector.showProgress === 0 &&
+                    vector.sidebarPosition === 'start' &&
+                    vector.direction === 'ltr',
+            );
+            if (!closedAtStart) throw new Error('OVERLAY_SWIPE_AREA_VECTORS lost its closed/start/ltr row');
+            const strip = closedAtStart.area.width;
+
+            const { view, host } = mountSizedView(800, 'collapsed show-sidebar="false"');
+            const state = stateOf(view);
+            await settle();
+            const rect = view.getBoundingClientRect();
+
+            const down = (offsetX: number) =>
+                view.dispatchEvent(
+                    new PointerEvent('pointerdown', {
+                        bubbles: true,
+                        pointerId: 1,
+                        button: 0,
+                        clientX: rect.left + offsetX,
+                        clientY: rect.top + 100,
+                    }),
+                );
+            const move = (offsetX: number) =>
+                view.dispatchEvent(
+                    new PointerEvent('pointermove', {
+                        bubbles: true,
+                        pointerId: 1,
+                        clientX: rect.left + offsetX,
+                        clientY: rect.top + 100,
+                    }),
+                );
+
+            // Just OUTSIDE the strip, so nothing begins.
+            down(strip + 4);
+            move(strip + 120);
+            expect(state.swipeActive).toBe(false);
+
+            // Just inside it: the gesture takes over and the progress follows.
+            down(Math.max(strip - 4, 0));
+            move(strip + 120);
+            expect(state.swipeActive).toBe(true);
+            expect(state.showProgress).toBeGreaterThan(0);
+            host.remove();
+        });
+
+        for (const vector of OVERLAY_SWIPE_RELEASE_VECTORS) {
+            await it(`release at ${vector.to} — ${vector.rule}`, async () => {
+                const { view, host } = mountSizedView(800, 'collapsed');
+                const state = stateOf(view);
+                await settle();
+                // Seed `show-sidebar` without animating, then take the gesture.
+                state.setShowSidebar(vector.showSidebar, { animate: false });
+                state.beginSwipe();
+                const plan = state.endSwipe(vector.to);
+                expect(plan.kind).toBe(vector.kind);
+                // A `set-show-sidebar` release is the one that notifies, so the
+                // property has to have MOVED, not just the animation settled.
+                if (plan.kind === 'set-show-sidebar') expect(state.showSidebar).toBe(vector.showSidebarAfter);
+                host.remove();
+            });
+        }
+
+        await it('a cancelled pointer snaps back where get_cancel_progress says', async () => {
+            const { view, host } = mountSizedView(800, 'collapsed show-sidebar="false"');
+            const state = stateOf(view);
+            await settle();
+            const rect = view.getBoundingClientRect();
+            view.dispatchEvent(
+                new PointerEvent('pointerdown', {
+                    bubbles: true,
+                    pointerId: 2,
+                    button: 0,
+                    clientX: rect.left + 8,
+                    clientY: rect.top + 100,
+                }),
+            );
+            view.dispatchEvent(
+                new PointerEvent('pointermove', {
+                    bubbles: true,
+                    pointerId: 2,
+                    clientX: rect.left + 60,
+                    clientY: rect.top + 100,
+                }),
+            );
+            expect(state.swipeActive).toBe(true);
+            view.dispatchEvent(new PointerEvent('pointercancel', { bubbles: true, pointerId: 2 }));
+            expect(state.swipeActive).toBe(false);
+            // A short drag from closed rounds back to closed.
+            expect(state.showSidebar).toBe(false);
+            host.remove();
+        });
+
+        await it('Escape closes a collapsed revealed sidebar', async () => {
+            // `escape_shortcut_cb` (adw-overlay-split-view.c:705-716) was absent
+            // from both ports, which made `OverlaySplitViewState.escape()` dead
+            // code.
+            const { view, host } = mountSizedView(800, 'collapsed');
+            await settle();
+            // Parse-time attributes are CONSTRUCTION options, not sequential
+            // setters, so `collapsed` in the markup does not fire the auto-hide
+            // — the element documents that and OVERLAY_COLLAPSE_VECTORS covers
+            // the setter path. So the sidebar is already revealed here.
+            expect(view.showSidebar).toBe(true);
+
+            const event = new KeyboardEvent('keydown', { key: 'Escape', bubbles: true, cancelable: true });
+            view.dispatchEvent(event);
+            expect(event.defaultPrevented).toBe(true);
+            expect(view.showSidebar).toBe(false);
+            host.remove();
+        });
+
+        await it('...and PROPAGATES from an uncollapsed one, so an enclosing dialog still closes', async () => {
+            const { view, host } = mountSizedView(800);
+            await settle();
+            expect(view.collapsed).toBe(false);
+            expect(view.showSidebar).toBe(true);
+
+            const event = new KeyboardEvent('keydown', { key: 'Escape', bubbles: true, cancelable: true });
+            view.dispatchEvent(event);
+            // A docked sidebar is not something Escape closes, however revealed
+            // it is — the gate is `!collapsed`, checked before the progress.
+            expect(event.defaultPrevented).toBe(false);
+            expect(view.showSidebar).toBe(true);
+            host.remove();
+        });
+
+        await it('is gated on the PROGRESS, so it still consumes mid-close', async () => {
+            // Worth pinning because it looks like a bug: `escape_shortcut_cb`
+            // tests `show_progress`, not `show_sidebar`, so a second Escape
+            // during the closing animation is consumed too and the set inside is
+            // a no-op. Reading the gate as "is the sidebar shown" would make the
+            // key propagate for ~250ms after the first press and close a dialog
+            // behind the view.
+            const { view, host } = mountSizedView(800, 'collapsed');
+            await settle();
+            expect(view.showSidebar).toBe(true);
+
+            const escape = () => {
+                const event = new KeyboardEvent('keydown', { key: 'Escape', bubbles: true, cancelable: true });
+                view.dispatchEvent(event);
+                return event.defaultPrevented;
+            };
+            expect(escape()).toBe(true);
+            expect(view.showSidebar).toBe(false);
+            // The reveal is still animating out, so the progress is above zero.
+            expect(escape()).toBe(true);
+            host.remove();
+        });
+
+        await it('slides off the edge the sidebar is ON, in both positions', async () => {
+            // `marginRight: -offsetWidth` regardless of `sidebarPosition` is what
+            // this used to do, so an `end` sidebar slid the wrong way.
+            const start = mountSizedView(800, 'show-sidebar="false"');
+            await settle();
+            const startBar = start.view.querySelector('.adw-osv-sidebar') as HTMLElement;
+            expect(startBar.style.marginLeft.startsWith('-')).toBe(true);
+            expect(startBar.style.marginRight).toBe('');
+            start.host.remove();
+
+            const end = mountSizedView(800, 'sidebar-position="end" show-sidebar="false"');
+            await settle();
+            const endBar = end.view.querySelector('.adw-osv-sidebar') as HTMLElement;
+            expect(endBar.style.marginRight.startsWith('-')).toBe(true);
+            expect(endBar.style.marginLeft).toBe('');
+            end.host.remove();
+        });
+
+        await it('mirrors under RTL — a start sidebar is drawn on the RIGHT', async () => {
+            const { view, host } = mountSizedView(800, 'show-sidebar="false"');
+            host.style.direction = 'rtl';
+            await settle();
+            // Re-run the geometry now the direction is readable.
+            view.setAttribute('sidebar-position', 'start');
+            await settle();
+            expect(view.classList.contains('sidebar-at-visual-start')).toBe(false);
+            const sidebar = view.querySelector('.adw-osv-sidebar') as HTMLElement;
+            expect(sidebar.style.marginRight.startsWith('-')).toBe(true);
+            host.remove();
+        });
+
+        await it('does not write a CSS PERCENTAGE width — (int) truncates, a percentage is fractional', async () => {
+            const { view, host } = mountSizedView(800);
+            await settle();
+            const sidebar = view.querySelector('.adw-osv-sidebar') as HTMLElement;
+            expect(sidebar.style.width.endsWith('%')).toBe(false);
+            expect(sidebar.style.width.endsWith('px')).toBe(true);
+            // 800 * 0.25 = 200, inside [180, 280], so it is the fraction exactly.
+            expect(sidebar.style.width).toBe('200px');
+            host.remove();
+        });
+
+        await it('keeps the content pane from being squeezed to nothing', async () => {
+            // `measure_uncollapsed` (:558-563) as CSS spells it: the pane claims
+            // its own min-content, where `min-width: 0` let the sidebar take
+            // everything.
+            const { view, host } = mountSizedView(800);
+            await settle();
+            const content = view.querySelector('.adw-osv-content') as HTMLElement;
+            expect(getComputedStyle(content).minWidth).not.toBe('0px');
+            host.remove();
+        });
+
+        for (const vector of OVERLAY_SPLIT_VIEW_LAYOUT_VECTORS) {
+            // Its exact analogue NAVIGATION_SPLIT_VIEW_LAYOUT_VECTORS was driven
+            // from the NativeScript suite while this one was driven by nobody —
+            // two sibling tables in the same file, with no reason for the
+            // difference. The element writes the pane rect from
+            // `layoutOverlaySplitView` now, so the rects are readable.
+            const label = `${vector.collapsed ? 'collapsed' : 'docked'} ${vector.sidebarPosition}/${vector.direction} @${vector.showProgress}`;
+            await it(`${label} — ${vector.rule}`, async () => {
+                const attrs =
+                    (vector.collapsed ? 'collapsed ' : '') +
+                    (vector.sidebarPosition === 'end' ? 'sidebar-position="end" ' : '') +
+                    'pin-sidebar';
+                const { view, host } = mountSizedView(vector.totalWidth, attrs);
+                if (vector.direction === 'rtl') host.style.direction = 'rtl';
+                await settle();
+                const state = stateOf(view);
+                state.setShowProgress(vector.showProgress);
+
+                // The SHIELD and the paint gate are the element's to publish;
+                // the pixel rects depend on the measured sidebar width, which a
+                // real layout decides, so the shared half is what is compared.
+                const backdrop = view.querySelector('.adw-osv-backdrop') as HTMLElement;
+                expect(backdrop.hidden).toBe(!vector.layout.shieldVisible);
+                const sidebar = view.querySelector('.adw-osv-sidebar') as HTMLElement;
+                expect(sidebar.style.pointerEvents === 'none').toBe(!vector.layout.sidebarPainted);
+                host.remove();
+            });
+        }
+
+        await it('names the cancel rows the core owns, so this suite is not read as covering them', () => {
+            // `swipeCancelProgress`'s half-away-from-zero rounding is arithmetic
+            // with no DOM surface — a pointercancel can only show the rounded
+            // OUTCOME, not the rule. The core suite drives every row.
+            expect(OVERLAY_SWIPE_CANCEL_VECTORS.length).toBeGreaterThan(0);
+            expect(OVERLAY_SWIPE_START_VECTORS.length).toBeGreaterThan(0);
+        });
+    });
+
+    await describe('<adw-navigation-split-view> stack, tags and navigation.* actions', async () => {
+        /** Mount a navigation split view with the given panes and attributes. */
+        function mountNav(attrs = '', panes = { sidebar: true, content: true, sidebarTag: '', contentTag: '' }) {
+            const host = document.createElement('div');
+            host.style.width = '800px';
+            host.style.height = '400px';
+            document.body.appendChild(host);
+            const sidebar = panes.sidebar
+                ? `<div slot="sidebar"${panes.sidebarTag ? ` tag="${panes.sidebarTag}"` : ''}>s</div>`
+                : '';
+            const content = panes.content
+                ? `<div slot="content"${panes.contentTag ? ` tag="${panes.contentTag}"` : ''}>c</div>`
+                : '';
+            host.innerHTML = `<adw-navigation-split-view ${attrs}>${sidebar}${content}</adw-navigation-split-view>`;
+            return {
+                view: host.querySelector('adw-navigation-split-view') as AdwNavigationSplitView,
+                host,
+            };
+        }
+
+        /** Which pane the element says is on top, read off the DOM. */
+        const visiblePane = (view: AdwNavigationSplitView) => {
+            const sidebar = view.querySelector('.adw-nsv-sidebar') as HTMLElement;
+            const content = view.querySelector('.adw-nsv-content') as HTMLElement;
+            return {
+                sidebar: sidebar?.dataset.paneVisible === 'true',
+                content: content?.dataset.paneVisible === 'true',
+            };
+        };
+
+        for (const vector of NAVIGATION_STACK_VECTORS) {
+            // Only the STATIC rows: the animated branch reaches the same stack by
+            // a push or a pop, and a DOM renderer that has no transition cannot
+            // tell them apart. The core suite drives the direction.
+            if (vector.changingPage) continue;
+            const label =
+                `${vector.hasSidebar ? 'sidebar' : '-'}/${vector.hasContent ? 'content' : '-'} ` +
+                `${vector.sidebarPosition} show-content=${vector.showContent}`;
+            await it(`${label} → [${vector.plan.stack.join(', ')}] — ${vector.rule}`, async () => {
+                const attrs =
+                    'collapsed ' +
+                    (vector.showContent ? 'show-content ' : '') +
+                    (vector.sidebarPosition === 'end' ? 'sidebar-position="end"' : '');
+                const { view, host } = mountNav(attrs, {
+                    sidebar: vector.hasSidebar,
+                    content: vector.hasContent,
+                    sidebarTag: '',
+                    contentTag: '',
+                });
+                await settle();
+                const top = vector.plan.stack[vector.plan.stack.length - 1] ?? null;
+                const panes = visiblePane(view);
+                // A LONE child stays visible whatever `show-content` says — the
+                // rule two CSS classes could not express, and the reason a
+                // collapsed sidebar-only view used to render blank.
+                if (vector.hasSidebar) expect(panes.sidebar).toBe(top === 'sidebar');
+                if (vector.hasContent) expect(panes.content).toBe(top === 'content');
+                host.remove();
+            });
+        }
+
+        for (const vector of NAVIGATION_ACTION_VECTORS) {
+            const label =
+                vector.action === 'push'
+                    ? `push "${vector.tag}" (sidebar=${vector.sidebarTag ?? 'none'}, content=${vector.contentTag ?? 'none'})`
+                    : `pop (sidebar=${vector.hasSidebar}, content=${vector.hasContent})`;
+            // A row whose two panes carry the SAME tag describes a state the
+            // duplicate-tag guard makes unreachable through a real widget: the
+            // second assignment is refused (:1195-1201), so the element can never
+            // hold both. Asserted below instead of driven here.
+            const collides =
+                vector.action === 'push' && vector.sidebarTag != null && vector.sidebarTag === vector.contentTag;
+            if (collides) continue;
+
+            await it(`${label} → ${vector.result.kind} — ${vector.rule}`, async () => {
+                const attrs = (vector.collapsed ? 'collapsed ' : '') + (vector.showContent ? 'show-content' : '');
+                const { view, host } = mountNav(attrs, {
+                    sidebar: vector.action === 'pop' ? (vector.hasSidebar ?? false) : true,
+                    content: vector.action === 'pop' ? (vector.hasContent ?? false) : true,
+                    sidebarTag: vector.sidebarTag ?? '',
+                    contentTag: vector.contentTag ?? '',
+                });
+                // `delegate` is the ROUTING's answer; whether it survives depends
+                // on the ancestor. An ancestor that claims the tag is the case the
+                // table describes — the unclaimed one becomes a critical, which
+                // is the state's own step and has its own test below.
+                host.addEventListener('navigation-push', (event) => {
+                    (event as CustomEvent<{ handled: boolean }>).detail.handled = true;
+                });
+                host.addEventListener('navigation-pop', (event) => {
+                    (event as CustomEvent<{ handled: boolean }>).detail.handled = true;
+                });
+                await settle();
+                const result = vector.action === 'push' ? view.push(vector.tag as string) : view.pop();
+                expect(result.kind).toBe(vector.result.kind);
+                if (result.kind === 'set-show-content' && vector.result.kind === 'set-show-content') {
+                    expect(view.showContent).toBe(vector.result.showContent);
+                    // The DOM has to agree, or a consumer reading the attribute
+                    // back after a push sees the pre-push answer.
+                    expect(view.hasAttribute('show-content')).toBe(vector.result.showContent);
+                }
+                host.remove();
+            });
+        }
+
+        await it('leaves out only the rows the tag guard makes unreachable', () => {
+            const collidingRows = NAVIGATION_ACTION_VECTORS.filter(
+                (v) => v.action === 'push' && v.sidebarTag != null && v.sidebarTag === v.contentTag,
+            );
+            // Exactly the shared-tag pushes. Their `rule` even says so — the C's
+            // routing tests the content tag first, which only matters in a
+            // collision the setters refuse to create.
+            expect(collidingRows.length).toBeGreaterThan(0);
+            for (const row of collidingRows) expect(row.contentTag).toBe(row.sidebarTag);
+        });
+
+        await it('an UNCLAIMED push becomes a critical, not a silent no-op', async () => {
+            // `navigation_push_cb` :683 — the parent gets first refusal, and only
+            // then is it an error. A pop walks off the end silently instead.
+            const { view, host } = mountNav('collapsed', {
+                sidebar: true,
+                content: true,
+                sidebarTag: 'list',
+                contentTag: 'detail',
+            });
+            await settle();
+            const pushed = view.push('nowhere');
+            expect(pushed.kind).toBe('not-found');
+            expect(view.showContent).toBe(false);
+            host.remove();
+        });
+
+        await it('delegates an unmatched push to an ancestor, which is how nesting forwards it', async () => {
+            const { view, host } = mountNav('collapsed', {
+                sidebar: true,
+                content: true,
+                sidebarTag: 'list',
+                contentTag: 'detail',
+            });
+            await settle();
+            const seen: string[] = [];
+            host.addEventListener('navigation-push', (event) => {
+                const detail = (event as CustomEvent<{ tag?: string; handled: boolean }>).detail;
+                seen.push(detail.tag ?? '');
+                // An ancestor that OWNS the tag says so, and the critical is
+                // never reached — `return TRUE` in the C's routing.
+                detail.handled = true;
+            });
+            const result = view.push('somewhere-else');
+            expect(seen).toStrictEqual(['somewhere-else']);
+            expect(result.kind).toBe('delegate');
+            host.remove();
+        });
+
+        await it('REFUSES a pane whose tag the other one already carries', async () => {
+            // `adw_navigation_split_view_set_content` :1195-1201 — the pane keeps
+            // what it had; the assignment does not happen. Neither port had a
+            // duplicate-tag guard at all.
+            const { view, host } = mountNav('', {
+                sidebar: true,
+                content: true,
+                sidebarTag: 'same',
+                contentTag: 'same',
+            });
+            await settle();
+            expect(view.sidebarTag).toBe('same');
+            // The content was refused, so it carries no tag of record.
+            expect(view.contentTag).toBe(null);
+            host.remove();
+        });
+
+        await it('puts the divider on the side the sidebar is DRAWN on', async () => {
+            const start = mountNav();
+            await settle();
+            expect(start.view.classList.contains('sidebar-at-visual-start')).toBe(true);
+            start.host.remove();
+
+            const end = mountNav('sidebar-position="end"');
+            await settle();
+            expect(end.view.classList.contains('sidebar-at-visual-start')).toBe(false);
+            end.host.remove();
+        });
+    });
+
+    // A VIEW THAT WAS NEVER MEASURED, AND A VIEW THAT WAS MOVED.
+    //
+    // Both shipped broken in v0.32.0 and were visible on the docs site: its
+    // slideshow builds each slide inside `display: none` and moves it in, so the
+    // controls pane had no measurement when it first rendered and no live
+    // observer afterwards. A hidden sidebar was painted over the content at the
+    // wrong edge — the state was right the whole time, only the geometry never
+    // reached the DOM.
+    //
+    // These are placement assertions on purpose. Every other test here reads the
+    // state or a class, and every one of them PASSED while the widget was
+    // visibly wrong: nothing in the suite looked at where the pane actually
+    // ended up. That is the gap, not the individual bug.
+    await describe('AdwOverlaySplitView — geometry survives being unmeasured and moved', async () => {
+        /** Does the pane cover the view's own box? */
+        const covers = (view: AdwOverlaySplitView): boolean => {
+            const pane = view.querySelector('.adw-osv-sidebar') as HTMLElement;
+            const a = view.getBoundingClientRect();
+            const b = pane.getBoundingClientRect();
+            return b.width > 0 && b.right > a.left + 1 && b.left < a.right - 1;
+        };
+
+        await it('keeps a hidden sidebar off-screen when it is first laid out inside display:none', async () => {
+            const slide = document.createElement('div');
+            slide.style.display = 'none';
+            document.body.appendChild(slide);
+            slide.innerHTML =
+                '<div style="width:900px;height:300px;display:flex">' +
+                '<adw-overlay-split-view collapsed show-sidebar="false" sidebar-position="end">' +
+                '<div slot="content">c</div><div slot="sidebar">s</div>' +
+                '</adw-overlay-split-view></div>';
+            const view = slide.querySelector('adw-overlay-split-view') as AdwOverlaySplitView;
+            await settle();
+
+            // The slide becomes the visible one.
+            slide.style.display = '';
+            await settle();
+            expect(view.showSidebar).toBe(false);
+            expect(covers(view)).toBe(false);
+            slide.remove();
+        });
+
+        await it('re-measures after being moved to another parent', async () => {
+            const from = document.createElement('div');
+            from.style.display = 'none';
+            const to = document.createElement('div');
+            to.setAttribute('style', 'width:900px;height:300px;display:flex');
+            document.body.append(from, to);
+            from.innerHTML =
+                '<adw-overlay-split-view collapsed show-sidebar="false" sidebar-position="end">' +
+                '<div slot="content">c</div><div slot="sidebar">s</div>' +
+                '</adw-overlay-split-view>';
+            const view = from.querySelector('adw-overlay-split-view') as AdwOverlaySplitView;
+            await settle();
+
+            // Moving a node is a disconnect followed by a connect. The observer
+            // torn down by the first has to be rebuilt by the second.
+            to.appendChild(view);
+            await settle();
+            expect(covers(view)).toBe(false);
+
+            // And it must keep tracking: a resize after the move has to land.
+            to.style.width = '600px';
+            await settle();
+            const pane = view.querySelector('.adw-osv-sidebar') as HTMLElement;
+            expect(pane.style.left).toBe('600px');
+            from.remove();
+            to.remove();
+        });
+
+        await it('hands placement back to the stylesheet rather than stranding a stale offset', async () => {
+            const { view, host } = mount('collapsed show-sidebar="false" sidebar-position="end"');
+            host.setAttribute('style', 'width:900px;height:300px;display:flex');
+            await settle();
+            const pane = view.querySelector('.adw-osv-sidebar') as HTMLElement;
+            expect(pane.style.left).toBe('900px');
+
+            // Unmeasurable again: the inline offset must be CLEARED, because an
+            // absolutely-positioned box with `left` + `width` ignores `right`,
+            // so a leftover value would outrank the resting rule it falls back
+            // to and pin the pane wherever it last happened to be.
+            host.style.display = 'none';
+            await settle();
+            expect(pane.style.left).toBe('');
+            host.remove();
+        });
+    });
 };