@mhmo91/schmancy 0.10.37 → 0.10.38

package/src/splash-screen/splash-screen.ts

@@ -77,13 +77,13 @@ export default class SchmancySplashScreen extends SchmancyElement {
 	 */
 	@property({ type: Boolean, attribute: 'initially-hidden' }) initiallyHidden = false
 
-	@state() private _visible = true
+	@state() private visible = true
 
 	connectedCallback() {
 		super.connectedCallback()
 
 		if (this.initiallyHidden) {
-			this._visible = false
+			this.visible = false
 			return
 		}
 
@@ -93,7 +93,7 @@ export default class SchmancySplashScreen extends SchmancyElement {
 		zip(readySignal$, minTimer$)
 			.pipe(
 				take(1),
-				tap(() => this._dismiss()),
+				tap(() => this.dismiss()),
 				takeUntil(this.disconnecting),
 			)
 			.subscribe()
@@ -111,11 +111,11 @@ export default class SchmancySplashScreen extends SchmancyElement {
 	 * Force the splash to show again (e.g. between route transitions).
 	 */
 	public show(): void {
-		this._visible = true
+		this.visible = true
 	}
 
-	private _dismiss(): void {
-		this._visible = false
+	private dismiss(): void {
+		this.visible = false
 		this.dispatchEvent(
 			new CustomEvent<void>('schmancy-splash-done', {
 				bubbles: true,
@@ -128,10 +128,10 @@ export default class SchmancySplashScreen extends SchmancyElement {
 		return html`
 			<div
 				class="splash-layer"
-				aria-hidden=${!this._visible}
+				aria-hidden=${!this.visible}
 				style=${styleMap({
-					opacity: this._visible ? '1' : '0',
-					pointerEvents: this._visible ? 'auto' : 'none',
+					opacity: this.visible ? '1' : '0',
+					pointerEvents: this.visible ? 'auto' : 'none',
 				})}
 			>
 				<slot name="splash"></slot>
@@ -139,8 +139,8 @@ export default class SchmancySplashScreen extends SchmancyElement {
 			<div
 				class="content-layer"
 				style=${styleMap({
-					opacity: this._visible ? '0' : '1',
-					pointerEvents: this._visible ? 'none' : 'auto',
+					opacity: this.visible ? '0' : '1',
+					pointerEvents: this.visible ? 'none' : 'auto',
 				})}
 			>
 				<slot></slot>

package/src/state/index.ts

@@ -155,7 +155,7 @@ export interface BaseAPI<NS extends string, T, S extends StorageBackend> {
 	 *  current value. Exposed on the type so a state can be passed to
 	 *  `<schmancy-context provides={[…]}>` without casts. Not for end-user
 	 *  code — the leading underscore signals "internal API". */
-	_isolatedInstance(): { destroy(): void } & Record<string | symbol, unknown>
+	isolatedInstance(): { destroy(): void } & Record<string | symbol, unknown>
 }
 
 export interface SyncState<NS extends string, T, S extends SyncStorage>
@@ -219,12 +219,12 @@ function detectKind(value: unknown): RuntimeKind {
 // who arrived first.
 const CLAIMED_KEY = Symbol.for('schmancy.state.claimed')
 const INSTANCES_KEY = Symbol.for('schmancy.state.instances')
-const __claimedSlot = (globalThis as { [CLAIMED_KEY]?: Set<string> })
-__claimedSlot[CLAIMED_KEY] ??= new Set<string>()
-const claimed = __claimedSlot[CLAIMED_KEY]!
-const __instancesSlot = (globalThis as { [INSTANCES_KEY]?: Map<string, unknown> })
-__instancesSlot[INSTANCES_KEY] ??= new Map<string, unknown>()
-const instances = __instancesSlot[INSTANCES_KEY]!
+const claimedSlot = (globalThis as { [CLAIMED_KEY]?: Set<string> })
+claimedSlot[CLAIMED_KEY] ??= new Set<string>()
+const claimed = claimedSlot[CLAIMED_KEY]!
+const instancesSlot = (globalThis as { [INSTANCES_KEY]?: Map<string, unknown> })
+instancesSlot[INSTANCES_KEY] ??= new Map<string, unknown>()
+const instances = instancesSlot[INSTANCES_KEY]!
 
 // ---------------------------------------------------------------------------
 // Context resolution.
@@ -245,11 +245,11 @@ const instances = __instancesSlot[INSTANCES_KEY]!
 // cache keys are DOM elements (process-global), but the cache itself must
 // also be shared.
 const CACHE_KEY = Symbol.for('schmancy.state.hostResolverCache')
-const __cacheSlot = (globalThis as {
+const cacheSlot = (globalThis as {
 	[CACHE_KEY]?: WeakMap<HTMLElement, Map<string, unknown>>
 })
-__cacheSlot[CACHE_KEY] ??= new WeakMap<HTMLElement, Map<string, unknown>>()
-const hostResolverCache = __cacheSlot[CACHE_KEY]!
+cacheSlot[CACHE_KEY] ??= new WeakMap<HTMLElement, Map<string, unknown>>()
+const hostResolverCache = cacheSlot[CACHE_KEY]!
 
 function resolveContextual(namespace: string, fallback: unknown): unknown {
 	const host = resolveActiveHost()
@@ -436,7 +436,7 @@ interface CreateInstanceOptions {
 	 *  it does NOT claim the namespace in `claimed`, it does NOT route reads
 	 *  or writes through `resolveContextual` (so it can serve as the
 	 *  resolution target without recursion), and it does NOT expose
-	 *  `_isolatedInstance` (you only isolate the global). */
+	 *  `isolatedInstance` (you only isolate the global). */
 	isolated?: boolean
 }
 
@@ -578,7 +578,7 @@ function createInstance(args: CreateInstanceArgs, options: CreateInstanceOptions
 
 	// Hook used by `<schmancy-context>` to mint a per-context copy seeded
 	// with the current value. Internal: not part of the public State<> type.
-	instance._isolatedInstance = (): unknown =>
+	instance.isolatedInstance = (): unknown =>
 		createInstance(
 			{ namespace, initial: signal.get(), storage: 'memory' },
 			{ isolated: true },

package/src/state/schmancy-context.ts

@@ -51,7 +51,7 @@ export interface StateTemplate {
    *  per-context copy seeded with the current value, sharing nothing with the
    *  global instance. The returned object exposes `destroy()` so the context
    *  element can flush + close on disconnect. */
-  _isolatedInstance(): { destroy(): void } & Record<string | symbol, unknown>;
+  isolatedInstance(): { destroy(): void } & Record<string | symbol, unknown>;
 }
 
 export class SchmancyContext extends SchmancyElement {
@@ -63,12 +63,12 @@ export class SchmancyContext extends SchmancyElement {
 
   // Per-mount records. Keep both the provider (so we can release it on
   // disconnect) and the isolated instance (so we can destroy it).
-  private _scoped: Array<{
+  private scoped: Array<{
     isolated: { destroy(): void };
     provider: ContextProvider<Context<unknown, unknown>>;
   }> = [];
 
-  private _publishEventTargetAsHost = (e: Event): void => {
+  private publishEventTargetAsHost = (e: Event): void => {
     // Publish the deepest event target (across shadow boundaries) rather
     // than `this`. The downstream `ContextRequestEvent` will be dispatched
     // from this host and must bubble up through `<schmancy-context>` to
@@ -87,28 +87,28 @@ export class SchmancyContext extends SchmancyElement {
   override connectedCallback(): void {
     super.connectedCallback();
     for (const tmpl of this.provides) {
-      const isolated = tmpl._isolatedInstance();
+      const isolated = tmpl.isolatedInstance();
       const ctx = createContext<unknown>(stateContextKey(tmpl.namespace));
       // ContextProvider auto-wires via the host's ReactiveControllerHost
       // interface (LitElement satisfies it). It listens for
       // `context-request` events and responds with `isolated` for any
       // requester whose `event.context === ctx`.
       const provider = new ContextProvider(this, { context: ctx, initialValue: isolated });
-      this._scoped.push({ isolated, provider });
+      this.scoped.push({ isolated, provider });
     }
     for (const type of EVENT_TYPES) {
-      this.addEventListener(type, this._publishEventTargetAsHost, { capture: true });
+      this.addEventListener(type, this.publishEventTargetAsHost, { capture: true });
     }
   }
 
   override disconnectedCallback(): void {
     for (const type of EVENT_TYPES) {
-      this.removeEventListener(type, this._publishEventTargetAsHost, { capture: true });
+      this.removeEventListener(type, this.publishEventTargetAsHost, { capture: true });
     }
-    for (const entry of this._scoped) {
+    for (const entry of this.scoped) {
       entry.isolated.destroy();
     }
-    this._scoped = [];
+    this.scoped = [];
     super.disconnectedCallback();
   }
 

package/src/teleport/teleport.component.ts

@@ -32,7 +32,7 @@ export class SchmancyTeleportation extends SchmancyElement {
 
 	debugging = Boolean(import.meta.env.DEV)
 
-	get _slottedChildren() {
+	get slottedChildren() {
 		const slot = this.shadowRoot.querySelector('slot')
 		return slot.assignedElements({ flatten: true })
 	}
@@ -101,7 +101,7 @@ export class SchmancyTeleportation extends SchmancyElement {
 									},
 									to: {
 										rect: e,
-										element: this._slottedChildren[0] as HTMLElement,
+										element: this.slottedChildren[0] as HTMLElement,
 									},
 									host: this,
 								})

package/src/theme/theme.service.ts

@@ -63,10 +63,10 @@ class ThemeService {
   private static instance: ThemeService
 
   // Observable properties for theme values
-  private _theme$ = new BehaviorSubject<Partial<TSchmancyTheme>>({})
-  private _themeComponent$ = new BehaviorSubject<SchmancyThemeComponent | null>(null)
-  private _fullscreen$ = new BehaviorSubject<boolean>(false)
-  private _bottomOffset$ = new BehaviorSubject<number>(0)
+  #theme$ = new BehaviorSubject<Partial<TSchmancyTheme>>({})
+  #themeComponent$ = new BehaviorSubject<SchmancyThemeComponent | null>(null)
+  #fullscreen$ = new BehaviorSubject<boolean>(false)
+  #bottomOffset$ = new BehaviorSubject<number>(0)
 
   // Public observables derived from context
   public readonly scheme$ = ThemeContext.$.pipe(
@@ -81,22 +81,22 @@ class ThemeService {
     shareReplay(1)
   )
 
-  public readonly theme$ = this._theme$.asObservable().pipe(
+  public readonly theme$ = this.#theme$.asObservable().pipe(
     distinctUntilChanged((a, b) => JSON.stringify(a) === JSON.stringify(b)),
     shareReplay(1)
   )
 
-  public readonly themeComponent$ = this._themeComponent$.asObservable().pipe(
+  public readonly themeComponent$ = this.#themeComponent$.asObservable().pipe(
     distinctUntilChanged(),
     shareReplay(1)
   )
 
-  public readonly fullscreen$ = this._fullscreen$.asObservable().pipe(
+  public readonly fullscreen$ = this.#fullscreen$.asObservable().pipe(
     distinctUntilChanged(),
     shareReplay(1)
   )
 
-  public readonly bottomOffset$ = this._bottomOffset$.asObservable().pipe(
+  public readonly bottomOffset$ = this.#bottomOffset$.asObservable().pipe(
     distinctUntilChanged(),
     shareReplay(1)
   )
@@ -111,19 +111,19 @@ class ThemeService {
   }
 
   get theme(): Partial<TSchmancyTheme> {
-    return this._theme$.getValue()
+    return this.#theme$.getValue()
   }
 
   get themeComponent(): SchmancyThemeComponent | null {
-    return this._themeComponent$.getValue()
+    return this.#themeComponent$.getValue()
   }
 
   get fullscreen(): boolean {
-    return this._fullscreen$.getValue()
+    return this.#fullscreen$.getValue()
   }
 
   get bottomOffset(): number {
-    return this._bottomOffset$.getValue()
+    return this.#bottomOffset$.getValue()
   }
 
   // Computed observable for actual scheme (resolving 'auto')
@@ -217,11 +217,11 @@ class ThemeService {
    * @internal
    */
   public registerThemeComponent(component: SchmancyThemeComponent): void {
-    this._themeComponent$.next(component)
+    this.#themeComponent$.next(component)
 
     // Update values from the component
     ThemeContext.set({ scheme: component.scheme, color: component.color })
-    this._theme$.next(component.theme)
+    this.#theme$.next(component.theme)
   }
 
   /**
@@ -250,7 +250,7 @@ class ThemeService {
       ThemeContext.set(updates)
     }
     if (values.theme !== undefined) {
-      this._theme$.next(values.theme)
+      this.#theme$.next(values.theme)
     }
   }
 
@@ -415,7 +415,7 @@ class ThemeService {
    * ```
    */
   public setFullscreen(value: boolean): void {
-    this._fullscreen$.next(value)
+    this.#fullscreen$.next(value)
 
     // Emit custom event for components to listen to
     window.dispatchEvent(
@@ -458,7 +458,7 @@ class ThemeService {
    * ```
    */
   public setBottomOffset(value: number): void {
-    this._bottomOffset$.next(value)
+    this.#bottomOffset$.next(value)
   }
 
   /**

package/src/tree/tree.ts

@@ -35,19 +35,19 @@ export class SchmancyTree extends SchmancyElement {
 	// Since it's actually a <schmancy-button>, use HTMLElement or a custom type
 	@query('#chevron') chevron!: HTMLElement
 
-	private readonly _a11yId = `schmancy-tree-${Math.random().toString(36).slice(2, 10)}`
-	private get _contentId() { return `${this._a11yId}-content` }
+	private readonly a11yId = `schmancy-tree-${Math.random().toString(36).slice(2, 10)}`
+	private get contentId() { return `${this.a11yId}-content` }
 
 	/** ElementInternals — broadcasts `:state(open)` for consumer CSS. */
-	private readonly _internals: ElementInternals | undefined = (() => {
+	private readonly internals: ElementInternals | undefined = (() => {
 		try { return this.attachInternals() } catch { return undefined }
 	})()
 
 	updated(changed: Map<string, unknown>) {
 		super.updated?.(changed)
 		if (changed.has('open')) {
-			if (this.open) this._internals?.states.add('open')
-			else this._internals?.states.delete('open')
+			if (this.open) this.internals?.states.add('open')
+			else this.internals?.states.delete('open')
 		}
 	}
 
@@ -137,7 +137,7 @@ export class SchmancyTree extends SchmancyElement {
 					slot="trailing"
 					id="chevron"
 					aria-expanded=${this.open ? 'true' : 'false'}
-					aria-controls=${this._contentId}
+					aria-controls=${this.contentId}
 					aria-label=${this.open ? 'Collapse' : 'Expand'}
 					@click=${(e: Event) => e.stopPropagation()}
 				>
@@ -146,7 +146,7 @@ export class SchmancyTree extends SchmancyElement {
 			</div>
 
 			<!-- The default slot: tree children -->
-			<slot id=${this._contentId}></slot>
+			<slot id=${this.contentId}></slot>
 		`
 	}
 }

package/src/typography/typography.ts

@@ -351,11 +351,11 @@ export class SchmancyTypography extends SchmancyElement {
 	/** Placeholder shown when editable and empty */
 	@property({ type: String }) placeholder = ''
 
-	private _editRef = createRef<HTMLDivElement>()
+	private editRef = createRef<HTMLDivElement>()
 
 	/** Focus and select all text in editable mode */
 	selectAll() {
-		const el = this._editRef.value
+		const el = this.editRef.value
 		if (!el) return
 		el.focus()
 		const sel = window.getSelection()
@@ -373,7 +373,7 @@ export class SchmancyTypography extends SchmancyElement {
 		fromEvent<FocusEvent>(this, 'focusout').pipe(
 			filter(() => this.editable),
 			tap(() => {
-				const el = this._editRef.value
+				const el = this.editRef.value
 				if (!el) return
 				const newValue = el.innerText.trim()
 				if (newValue !== this.value) {
@@ -393,7 +393,7 @@ export class SchmancyTypography extends SchmancyElement {
 		fromEvent(this, 'input').pipe(
 			filter(() => this.editable),
 			tap(() => {
-				const el = this._editRef.value
+				const el = this.editRef.value
 				if (el && !el.innerText.trim()) el.textContent = ''
 			}),
 			takeUntil(this.disconnecting),
@@ -402,7 +402,7 @@ export class SchmancyTypography extends SchmancyElement {
 		fromEvent<KeyboardEvent>(this, 'keydown').pipe(
 			filter(() => this.editable),
 			filter(e => e.key === 'Enter'),
-			tap(e => { e.preventDefault(); (this._editRef.value ?? this).blur() }),
+			tap(e => { e.preventDefault(); (this.editRef.value ?? this).blur() }),
 			takeUntil(this.disconnecting),
 		).subscribe()
 	}
@@ -418,7 +418,7 @@ export class SchmancyTypography extends SchmancyElement {
 			}
 		}
 		if ((changedProperties.has('value') || changedProperties.has('editable')) && this.editable) {
-			const el = this._editRef.value
+			const el = this.editRef.value
 			if (el && document.activeElement !== el) {
 				if (this.value) {
 					el.innerText = this.value
@@ -432,7 +432,7 @@ export class SchmancyTypography extends SchmancyElement {
 	protected render(): unknown {
 		if (this.editable) {
 			return html`<div
-				${ref(this._editRef)}
+				${ref(this.editRef)}
 				class="edit"
 				contenteditable="true"
 				data-placeholder=${this.placeholder ?? ''}

package/src/window/window-manager.ts

@@ -23,13 +23,13 @@ const LEGACY_STORAGE_PREFIX = 'schmancy-float-'
 class WindowManagerService {
 	private static instance: WindowManagerService
 
-	private readonly _state$ = new BehaviorSubject<WindowRegistryState>({
+	readonly #state$ = new BehaviorSubject<WindowRegistryState>({
 		windows: new Map(),
 		focusedId: null,
 		stackOrder: [],
 	})
 
-	readonly state$ = this._state$.asObservable()
+	readonly state$ = this.#state$.asObservable()
 
 	private constructor() {}
 
@@ -43,17 +43,17 @@ class WindowManagerService {
 	// ── Synchronous accessors ──────────────────────────────────────────
 
 	get windows(): Map<string, WindowRecord> {
-		return this._state$.value.windows
+		return this.#state$.value.windows
 	}
 
 	get focusedId(): string | null {
-		return this._state$.value.focusedId
+		return this.#state$.value.focusedId
 	}
 
 	// ── Registration ───────────────────────────────────────────────────
 
 	register(id: string, initialBounds: WindowBounds, snapTarget: SnapTarget): void {
-		const state = this._state$.value
+		const state = this.#state$.value
 		if (state.windows.has(id)) return
 
 		const zIndex = overlayStack.assignZIndex(id)
@@ -70,11 +70,11 @@ class WindowManagerService {
 		windows.set(id, record)
 		const stackOrder = overlayStack.getStackOrder()
 
-		this._state$.next({ ...state, windows, stackOrder })
+		this.#state$.next({ ...state, windows, stackOrder })
 	}
 
 	unregister(id: string): void {
-		const state = this._state$.value
+		const state = this.#state$.value
 		if (!state.windows.has(id)) return
 
 		overlayStack.releaseId(id)
@@ -83,25 +83,25 @@ class WindowManagerService {
 		const stackOrder = overlayStack.getStackOrder()
 		const focusedId = state.focusedId === id ? null : state.focusedId
 
-		this._state$.next({ ...state, windows, stackOrder, focusedId })
+		this.#state$.next({ ...state, windows, stackOrder, focusedId })
 	}
 
 	// ── State mutations ────────────────────────────────────────────────
 
 	updateBounds(id: string, bounds: WindowBounds): void {
-		this._updateRecord(id, { bounds })
+		this.updateRecord(id, { bounds })
 	}
 
 	updateVisualState(id: string, visualState: WindowRecord['visualState']): void {
-		this._updateRecord(id, { visualState })
+		this.updateRecord(id, { visualState })
 	}
 
 	updateOpen(id: string, open: boolean): void {
-		this._updateRecord(id, { open })
+		this.updateRecord(id, { open })
 	}
 
 	focus(id: string): void {
-		const state = this._state$.value
+		const state = this.#state$.value
 		if (!state.windows.has(id)) return
 		if (state.focusedId === id) return
 
@@ -113,14 +113,14 @@ class WindowManagerService {
 		}
 		const stackOrder = overlayStack.getStackOrder()
 
-		this._state$.next({ ...state, windows, stackOrder, focusedId: id })
+		this.#state$.next({ ...state, windows, stackOrder, focusedId: id })
 	}
 
 	// ── Queries ────────────────────────────────────────────────────────
 
 	findOverlaps(bounds: WindowBounds, excludeId: string): WindowRecord[] {
 		const result: WindowRecord[] = []
-		for (const [id, record] of this._state$.value.windows) {
+		for (const [id, record] of this.#state$.value.windows) {
 			if (id === excludeId) continue
 			if (rectsOverlap(bounds, record.bounds)) {
 				result.push(record)
@@ -131,7 +131,7 @@ class WindowManagerService {
 
 	getNeighbors(id: string): WindowRecord[] {
 		const result: WindowRecord[] = []
-		for (const [otherId, record] of this._state$.value.windows) {
+		for (const [otherId, record] of this.#state$.value.windows) {
 			if (otherId !== id) result.push(record)
 		}
 		return result
@@ -140,14 +140,14 @@ class WindowManagerService {
 	// ── Selectors ──────────────────────────────────────────────────────
 
 	selectWindow(id: string): Observable<WindowRecord | undefined> {
-		return this._state$.pipe(
+		return this.#state$.pipe(
 			map(state => state.windows.get(id)),
 			distinctUntilChanged(),
 		)
 	}
 
 	selectFocused(): Observable<string | null> {
-		return this._state$.pipe(
+		return this.#state$.pipe(
 			map(state => state.focusedId),
 			distinctUntilChanged(),
 		)
@@ -185,14 +185,14 @@ class WindowManagerService {
 
 	// ── Internal ───────────────────────────────────────────────────────
 
-	private _updateRecord(id: string, updates: Partial<WindowRecord>): void {
-		const state = this._state$.value
+	private updateRecord(id: string, updates: Partial<WindowRecord>): void {
+		const state = this.#state$.value
 		const record = state.windows.get(id)
 		if (!record) return
 
 		const windows = new Map(state.windows)
 		windows.set(id, { ...record, ...updates })
-		this._state$.next({ ...state, windows })
+		this.#state$.next({ ...state, windows })
 	}
 }