@iyulab/modern-app 0.30.1 → 0.30.2

package/CHANGELOG.md

@@ -1,5 +1,24 @@
 # Changelog
 
+## [0.30.2] - 2026-09-28
+
+### Fixed
+
+- **An overlay panel that is a component now gets focus inside it.** The shell looked for the
+  panel's `[autofocus]` element and first input control with `querySelector`, which stops at shadow
+  roots — so when the panel was a custom element rendering its controls in its own template, the
+  shell found neither and fell back to the close button. The search now goes through shadow roots
+  and waits for the panel's first render, as route focus does since 0.30.1.
+- **Route and overlay focus skip a `u-select`'s closed popover.** Searching through shadow roots
+  (0.30.1) also reached the popover every `u-select` keeps inside, which carries an `autofocus`
+  attribute of its own — a screen with a `u-select` before its `autofocus` field got the content
+  area instead. Only elements that can actually take focus are candidates now.
+
+### Changed
+
+- **Requires `@iyulab/components` 1.47.1 or later** (peer) — the shell now uses its
+  `querySelectorDeep` / `isFocusCandidate`, the same lookup `u-dialog` and `u-drawer` use.
+
 ## [0.30.1] - 2026-09-28
 
 ### Fixed

package/dist/layouts/SidebarLayout.d.ts

@@ -122,6 +122,7 @@ export declare class SidebarLayout extends StyledElement<SidebarParts> {
      * `[autofocus]` → 첫 입력 컨트롤 → (없으면) 셸의 닫기 버튼.
      * 닫기 버튼으로 떨어지는 것은 의도다 — 포커스가 `<body>` 에 남는 것보다 항상 낫고,
      * 셸이 늘 가진 유일한 컨트롤이다. 소비자가 이미 패널 안으로 옮겨 두었으면 건드리지 않는다.
+     * 탐색은 섀도 경계 안쪽까지 한다 — 패널이 컴포넌트면 컨트롤은 그 섀도 루트에 있다.
      */
     private focusOverlay;
     /**

package/dist/layouts/SidebarLayout.js

@@ -19,32 +19,33 @@ import { unsafeHTML as h } from "lit/directives/unsafe-html.js";
 import { repeat as g } from "lit/directives/repeat.js";
 import { ifDefined as _ } from "lit/directives/if-defined.js";
 import { createDevWarner as v } from "@iyulab/components/dist/utilities/devWarning.js";
-import { UProgressBar as y } from "@iyulab/components/dist/components/progress-bar/UProgressBar.js";
+import { isFocusCandidate as y, querySelectorDeep as b } from "@iyulab/components/dist/utilities/elements.js";
+import { UProgressBar as x } from "@iyulab/components/dist/components/progress-bar/UProgressBar.js";
 //#region src/layouts/SidebarLayout.ts
-var b, x = v("@iyulab/modern-app"), S = 0;
-function C(e) {
+var S, C = v("@iyulab/modern-app"), w = 0;
+function T(e) {
 	let t = e.tagName;
 	if (t === "INPUT" || t === "TEXTAREA" || t === "SELECT" || e.isContentEditable) return !0;
 	let n = e.getAttribute("role");
 	return n === "textbox" || n === "searchbox" || n === "combobox" || n === "spinbutton";
 }
-function w(e) {
+function E(e) {
 	let t = getComputedStyle(e);
 	return (t.overflowY === "auto" || t.overflowY === "scroll") && e.scrollHeight > e.clientHeight;
 }
-function T(e, t) {
+function D(e, t) {
 	for (let n of e) if (n instanceof HTMLElement) {
-		if (w(n)) return n === t;
+		if (E(n)) return n === t;
 		if (n === t) break;
 	}
 	return !0;
 }
-function E() {
+function O() {
 	let e = document.activeElement;
 	for (; e?.shadowRoot?.activeElement;) e = e.shadowRoot.activeElement;
 	return e;
 }
-function D(e, t) {
+function k(e, t) {
 	let n = t;
 	for (; n;) {
 		if (n === e) return !0;
@@ -52,33 +53,12 @@ function D(e, t) {
 	}
 	return !1;
 }
-function O(e, t) {
-	let n = (e) => {
-		let r = e instanceof Element ? [e] : Array.from(e.children);
-		for (let e of r) {
-			let r = document.createTreeWalker(e, NodeFilter.SHOW_ELEMENT);
-			for (let e = r.currentNode; e; e = r.nextNode()) {
-				if (e.matches(t)) return e;
-				if (e.shadowRoot) {
-					let t = n(e.shadowRoot);
-					if (t) return t;
-				}
-			}
-		}
-		return null;
-	};
-	for (let t of e) {
-		let e = n(t);
-		if (e) return e;
-	}
-	return null;
-}
-var k = class extends i {
+var A = class extends i {
 	static {
-		b = this;
+		S = this;
 	}
 	constructor(...t) {
-		super(...t), this.state = "default", this.locale = "", this.hasOverlay = !1, this.hasNotice = !1, this.overlayTrigger = null, this.internals = typeof this.attachInternals == "function" ? this.attachInternals() : void 0, this.context = null, this.unsizedWarnKey = `unsized:${++S}`, this.isMatchedLink = (e) => !this.context || !e ? !1 : (e = typeof e == "string" ? new URLPattern(e, window.location.origin) : e, e.test(this.context.path, window.location.origin)), this.handleToggleButtonClick = () => {
+		super(...t), this.state = "default", this.locale = "", this.hasOverlay = !1, this.hasNotice = !1, this.overlayTrigger = null, this.internals = typeof this.attachInternals == "function" ? this.attachInternals() : void 0, this.context = null, this.unsizedWarnKey = `unsized:${++w}`, this.isMatchedLink = (e) => !this.context || !e ? !1 : (e = typeof e == "string" ? new URLPattern(e, window.location.origin) : e, e.test(this.context.path, window.location.origin)), this.handleToggleButtonClick = () => {
 			let t = this.screenSize ?? e.screen ?? "large";
 			t === "large" ? this.state = this.state === "default" ? "slim" : "default" : t === "medium" ? this.state = this.state === "slim" ? "modal" : "slim" : t === "small" ? this.state = this.state === "mobile" ? "mobile-open" : "mobile" : console.warn("Unknown screen size:", t);
 		}, this.handleBackdropClick = () => {
@@ -100,7 +80,7 @@ var k = class extends i {
 		}, this.handleOverlaySlotChange = (e) => {
 			let t = o(e.target);
 			if (t && !this.hasOverlay) {
-				let e = E();
+				let e = O();
 				this.overlayTrigger = e && e !== document.body ? e : null;
 			}
 			this.hasOverlay = t;
@@ -116,9 +96,9 @@ var k = class extends i {
 			this.fire("overlay-close", { cancelable: !1 });
 		}, this._handleMainKeydown = (e) => {
 			let t = e.composedPath()[0];
-			if (t instanceof HTMLElement && C(t)) return;
+			if (t instanceof HTMLElement && T(t)) return;
 			let n = this.shadowRoot?.querySelector(".main");
-			if (!n || !T(e.composedPath(), n)) return;
+			if (!n || !D(e.composedPath(), n)) return;
 			let r = n.clientHeight;
 			switch (e.key) {
 				case " ":
@@ -173,7 +153,7 @@ var k = class extends i {
 		process.env.NODE_ENV !== "production" && requestAnimationFrame(() => {
 			if (!this.isConnected) return;
 			let e = this.getBoundingClientRect().height;
-			e >= 200 || x(this.unsizedWarnKey, `u-sidebar-layout is only ${Math.round(e)}px tall — its height: 100% found no sized ancestor, so the shell sits at its own chrome height and the route area has almost no room. Give the root element a height (e.g. #app { height: 100vh } — app.load() does this for document.body).`);
+			e >= 200 || C(this.unsizedWarnKey, `u-sidebar-layout is only ${Math.round(e)}px tall — its height: 100% found no sized ancestor, so the shell sits at its own chrome height and the route area has almost no room. Give the root element a height (e.g. #app { height: 100vh } — app.load() does this for document.body).`);
 		});
 	}
 	render() {
@@ -325,7 +305,7 @@ var k = class extends i {
 		};
 	}
 	icon(e) {
-		return this.config?.icons?.[e] ?? b.DEFAULT_ICONS[e];
+		return this.config?.icons?.[e] ?? S.DEFAULT_ICONS[e];
 	}
 	renderLogo() {
 		let e = this.config?.logo, t = this.config?.title ?? "Home";
@@ -362,39 +342,32 @@ var k = class extends i {
 	async focusRoute(e) {
 		let t = this.routeContent();
 		if (await Promise.all(t.map((e) => e.updateComplete)), this.hasOverlay || !e.isConnected) return;
-		let n = E();
-		if (n && t.some((e) => D(e, n))) return;
-		let r = O(t, "[autofocus]");
+		let n = O();
+		if (n && t.some((e) => k(e, n))) return;
+		let r = b(t, "[autofocus]", y);
 		r?.focus();
-		let i = E();
-		r && i && D(r, i) || e.focus({ preventScroll: !0 });
+		let i = O();
+		r && i && k(r, i) || e.focus({ preventScroll: !0 });
 	}
 	overlayPanels() {
 		return (this.shadowRoot?.querySelector("slot[name=\"overlay\"]"))?.assignedElements({ flatten: !0 }) ?? [];
 	}
-	focusOverlay() {
-		let e = this.overlayPanels(), t = E();
-		if (t && e.some((e) => D(e, t))) return;
-		let n = (t) => {
-			for (let n of e) {
-				if (n.matches(t)) return n;
-				let e = n.querySelector(t);
-				if (e) return e;
-			}
-			return null;
-		};
-		(n("[autofocus]") ?? n("input, select, textarea, u-input, u-textarea, u-select, u-checkbox, u-radio, u-switch, u-slider") ?? this.shadowRoot?.querySelector(".overlay-close"))?.focus();
+	async focusOverlay() {
+		let e = this.overlayPanels();
+		if (await Promise.all(e.map((e) => e.updateComplete)), !this.hasOverlay) return;
+		let t = O();
+		t && e.some((e) => k(e, t)) || (b(e, "[autofocus]", y) ?? b(e, "input, select, textarea, u-input, u-textarea, u-select, u-checkbox, u-radio, u-switch, u-slider", y) ?? this.shadowRoot?.querySelector(".overlay-close"))?.focus();
 	}
 	restoreOverlayFocus() {
 		let e = this.overlayTrigger;
 		this.overlayTrigger = null;
-		let t = E();
+		let t = O();
 		t && t !== document.body || (e?.isConnected ? e.focus() : this.mainElement?.focus({ preventScroll: !0 }));
 	}
 };
 r([f({
 	type: String,
 	reflect: !0
-}), n("design:type", Object)], k.prototype, "state", void 0), r([f({ type: Object }), n("design:type", Object)], k.prototype, "config", void 0), r([f({ type: String }), n("design:type", Object)], k.prototype, "locale", void 0), r([p("u-progress-bar"), n("design:type", y === void 0 ? Object : y)], k.prototype, "progressBarEl", void 0), r([m(), n("design:type", Object)], k.prototype, "hasOverlay", void 0), r([m(), n("design:type", Object)], k.prototype, "hasNotice", void 0), r([m(), n("design:type", Object)], k.prototype, "context", void 0), k = b = r([d("u-sidebar-layout")], k);
+}), n("design:type", Object)], A.prototype, "state", void 0), r([f({ type: Object }), n("design:type", Object)], A.prototype, "config", void 0), r([f({ type: String }), n("design:type", Object)], A.prototype, "locale", void 0), r([p("u-progress-bar"), n("design:type", x === void 0 ? Object : x)], A.prototype, "progressBarEl", void 0), r([m(), n("design:type", Object)], A.prototype, "hasOverlay", void 0), r([m(), n("design:type", Object)], A.prototype, "hasNotice", void 0), r([m(), n("design:type", Object)], A.prototype, "context", void 0), A = S = r([d("u-sidebar-layout")], A);
 //#endregion
-export { k as SidebarLayout };
+export { A as SidebarLayout };

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@iyulab/modern-app",
   "description": "web-framework by iyulab based on lit-element",
-  "version": "0.30.1",
+  "version": "0.30.2",
   "keywords": [
     "iyulab",
     "web-framework",
@@ -64,7 +64,7 @@
     "lit": "^3.3.3"
   },
   "peerDependencies": {
-    "@iyulab/components": ">=1.45.2",
+    "@iyulab/components": ">=1.47.1",
     "@lit/react": ">=1.0.8",
     "react": ">=18.0.0"
   },
@@ -89,6 +89,6 @@
     "vite": "^8.1.4",
     "vite-plugin-dts": "^5.0.3",
     "vitest": "^5.0.0",
-    "@iyulab/components": ">=1.45.2"
+    "@iyulab/components": ">=1.47.1"
   }
 }

package/skills/modern-app/references/layout.md

@@ -414,7 +414,7 @@ from either re-implementing what the shell already does, or dropping what it doe
 | Making route content non-interactive | shell | `inert` on `part="main-content"`, which propagates through the slot into your route content |
 | A close affordance | shell | `part="overlay-close"`, firing the non-cancelable `overlay-close` event |
 | Keeping the route mounted underneath | shell | the overlay is independent of routing |
-| Moving focus into the panel when it opens | shell | an `[autofocus]` element in your panel, else its first input control, else `part="overlay-close"`. If you already moved focus into the panel, the shell leaves it there |
+| Moving focus into the panel when it opens | shell | an `[autofocus]` element in your panel, else its first input control, else `part="overlay-close"` — searched through shadow roots, after the panel's first render, so a panel that is itself a component works the same. If you already moved focus into the panel, the shell leaves it there |
 | Restoring focus when it closes | shell | back to the control that held focus when the panel opened — only if focus fell to `<body>`; if you moved it somewhere on purpose, that stands. With no such control (opened from code), focus goes to `part="main"` |
 | Escape to close | shell | Escape inside the panel fires the same `overlay-close` as the button — you still empty the slot. The shell decides **after** every layer inside the panel has: an Escape consumed there (`preventDefault()` — an open list, popover, drawer or dialog, whenever it was opened) closes only that layer. The shell does not mark the Escape consumed itself |
 | **A backdrop / dimmed scrim** | **consumer** | the panel is opaque and full-bleed by design; add a scrim inside your panel if you want one |