@marianmeres/stuic 3.76.3 → 3.76.5

package/dist/components/EmailVerifyForm/EmailVerifyForm.svelte

@@ -89,6 +89,18 @@
 
 	let t = $derived(tProp ?? t_default);
 
+	// Mirror the form ref into local $state so it survives prop re-application
+	// when the parent re-renders without binding `el`. See LoginForm for the full
+	// rationale — same Svelte 5 `$bindable` + `bind:this` gotcha applies here.
+	// EmailVerifyForm uses `onsubmit={handleFormSubmit}` (declarative attribute)
+	// so the listener wouldn't get torn down the way LoginForm/RegisterForm did,
+	// but the public `el` prop is still unsafe for consumers that bind it after
+	// any parent re-render — same fix for consistency.
+	let formEl = $state<HTMLFormElement | undefined>();
+	$effect(() => {
+		el = formEl;
+	});
+
 	let code = $state("");
 	let cooldownRemaining = $state(0);
 	let resentFlash = $state(false);
@@ -182,7 +194,7 @@
 	let submitDisabled = $derived(code.length !== codeLength || isSubmitting);
 </script>
 
-<form bind:this={el} class={_class} onsubmit={handleFormSubmit} {...rest}>
+<form bind:this={formEl} class={_class} onsubmit={handleFormSubmit} {...rest}>
 	<!-- Heading -->
 	<H level={2} class={unstyled ? undefined : "stuic-email-verify-form-heading"}>
 		{t("email_verify_form.heading")}

package/dist/components/EmailVerifyForm/index.css

@@ -1,6 +1,6 @@
 :root {
 	--stuic-email-verify-form-gap: 1rem;
-	--stuic-email-verify-form-subheading-color: var(--stuic-color-muted-foreground);
+	--stuic-email-verify-form-subheading-color: var(--stuic-color-foreground);
 	--stuic-email-verify-form-attempts-color: var(--stuic-color-muted-foreground);
 	--stuic-email-verify-form-resend-color: var(--stuic-color-muted-foreground);
 	--stuic-email-verify-form-resend-flash-color: var(--stuic-color-primary);
@@ -21,6 +21,7 @@
 		margin: 0;
 		color: var(--stuic-email-verify-form-subheading-color);
 		font-size: var(--text-sm);
+		text-align: center;
 	}
 
 	.stuic-email-verify-form-otp {

package/dist/components/LoginForm/LoginForm.svelte

@@ -115,6 +115,19 @@
 
 	let t = $derived(tProp ?? t_default);
 
+	// Mirror the form ref into local $state so it survives prop re-application
+	// when the parent re-renders without binding `el`. Otherwise the bindable
+	// prop reverts to its default (undefined) on the next parent re-render, the
+	// `$effect` tracking it cleans up the `submit_valid` listener, and `bind:this`
+	// does NOT re-fire (the form element wasn't unmounted) — leaving the form
+	// alive in the DOM with no submit handler. Next click silently goes nowhere.
+	// `el` stays the public API: consumers can still bind it; we mirror formEl
+	// into it so the binding sees a value.
+	let formEl = $state<HTMLFormElement | undefined>();
+	$effect(() => {
+		el = formEl;
+	});
+
 	// Internal validation errors (set on submit)
 	let internalErrors = $state<LoginFormValidationError[]>([]);
 
@@ -145,8 +158,8 @@
 		// pre-clears before the per-field re-dispatch), but doing it here too is
 		// cheap insurance against any future regression that lets a stale flag slip
 		// past the action.
-		if (el) {
-			for (const node of Array.from(el.elements) as HTMLInputElement[]) {
+		if (formEl) {
+			for (const node of Array.from(formEl.elements) as HTMLInputElement[]) {
 				if (typeof node.setCustomValidity === "function") node.setCustomValidity("");
 			}
 		}
@@ -188,9 +201,11 @@
 
 	// The onSubmitValidityCheck action intercepts native submit (capture phase,
 	// stopImmediatePropagation) and dispatches a custom "submit_valid" event.
-	// Listen for it on the form element as a fallback.
+	// Listen for it on the form element as a fallback. Reads `formEl` (local state)
+	// — NOT the `el` prop, which can revert to undefined on parent re-render and
+	// would cause this $effect's cleanup to silently detach the listener.
 	$effect(() => {
-		const node = el;
+		const node = formEl;
 		if (!node) return;
 		node.addEventListener("submit_valid", handleSubmitValid);
 		return () => node.removeEventListener("submit_valid", handleSubmitValid);
@@ -199,7 +214,7 @@
 	let _class = $derived(unstyled ? classProp : twMerge("stuic-login-form", classProp));
 </script>
 
-<form bind:this={el} class={_class} use:onSubmitValidityCheck {...rest}>
+<form bind:this={formEl} class={_class} use:onSubmitValidityCheck {...rest}>
 	<!-- General error alert -->
 	<DismissibleMessage message={error} intent="destructive" />
 

package/dist/components/LoginOrRegisterForm/LoginOrRegisterFormModal.svelte

@@ -168,6 +168,20 @@
 	export function close() {
 		modal.close();
 	}
+
+	// Reset terminal `verify` state when the modal *closes* — otherwise a
+	// register → OTP → logout → re-open cycle resumes on the stale 6-digit-code
+	// screen. Keyed on the visible: true → false transition (not a static
+	// `!visible && mode==="verify"` test) so consumers can still programmatically
+	// open the modal in verify mode by setting `mode = "verify"` before `open()`.
+	let _wasVisible = false;
+	$effect(() => {
+		if (_wasVisible && !visible && mode === "verify") {
+			mode = "login";
+			verifyEmail = "";
+		}
+		_wasVisible = visible;
+	});
 </script>
 
 {#if trigger}

package/dist/components/RegisterForm/RegisterForm.svelte

@@ -132,6 +132,14 @@
 
 	let t = $derived(tProp ?? t_default);
 
+	// Mirror the form ref into local $state so it survives prop re-application
+	// when the parent re-renders without binding `el`. See LoginForm for the full
+	// rationale — same Svelte 5 `$bindable` + `bind:this` gotcha applies here.
+	let formEl = $state<HTMLFormElement | undefined>();
+	$effect(() => {
+		el = formEl;
+	});
+
 	let topFields = $derived(extraFields.filter((f) => f.position === "top"));
 	let bottomFields = $derived(extraFields.filter((f) => f.position !== "top"));
 
@@ -180,9 +188,11 @@
 
 	// The onSubmitValidityCheck action intercepts native submit (capture phase,
 	// stopImmediatePropagation) and dispatches a custom "submit_valid" event.
-	// Listen for it on the form element as a fallback.
+	// Listen for it on the form element as a fallback. Reads `formEl` (local state)
+	// — NOT the `el` prop, which can revert to undefined on parent re-render and
+	// would cause this $effect's cleanup to silently detach the listener.
 	$effect(() => {
-		const node = el;
+		const node = formEl;
 		if (!node) return;
 		node.addEventListener("submit_valid", handleSubmitValid);
 		return () => node.removeEventListener("submit_valid", handleSubmitValid);
@@ -191,7 +201,7 @@
 	let _class = $derived(unstyled ? classProp : twMerge("stuic-register-form", classProp));
 </script>
 
-<form bind:this={el} class={_class} use:onSubmitValidityCheck {...rest}>
+<form bind:this={formEl} class={_class} use:onSubmitValidityCheck {...rest}>
 	<!-- General error alert -->
 	<DismissibleMessage message={error} intent="destructive" />
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@marianmeres/stuic",
-  "version": "3.76.3",
+  "version": "3.76.5",
   "files": [
     "dist",
     "!dist/**/*.test.*",