From 2e3e65474863f35dd86e59ce6ddb35b0d6d2bb62 Mon Sep 17 00:00:00 2001
From: Thomas Camlong <thomas@ajnart.fr>
Date: Fri, 14 Aug 2026 21:22:14 +0200
Subject: [PATCH] fix: improve auth form semantics

---
 .../auth/invite/[id]/_registration-form.tsx   |  7 ++++++-
 .../app/[locale]/auth/login/_login-form.tsx   | 20 ++++++++++++++++---
 2 files changed, 23 insertions(+), 4 deletions(-)

diff --git a/apps/nextjs/src/app/[locale]/auth/invite/[id]/_registration-form.tsx b/apps/nextjs/src/app/[locale]/auth/invite/[id]/_registration-form.tsx
index 0ec4f70b9..9bfe3bfd5 100644
--- a/apps/nextjs/src/app/[locale]/auth/invite/[id]/_registration-form.tsx
+++ b/apps/nextjs/src/app/[locale]/auth/invite/[id]/_registration-form.tsx
@@ -62,13 +62,16 @@ export const RegistrationForm = ({ invite }: RegistrationFormProps) => {
 
   return (
     <Stack gap="xl">
-      <form onSubmit={form.onSubmit(handleSubmit)}>
+      <form autoComplete="on" onSubmit={form.onSubmit(handleSubmit)}>
         <Stack gap="lg">
           <TextInput
             label={t("field.username.label")}
             id="username"
             autoComplete="username"
+            autoCapitalize="none"
+            spellCheck={false}
             {...form.getInputProps("username")}
+            name="username"
           />
           <CustomPasswordInput
             withPasswordRequirements
@@ -76,6 +79,7 @@ export const RegistrationForm = ({ invite }: RegistrationFormProps) => {
             id="password"
             autoComplete="new-password"
             {...form.getInputProps("password")}
+            name="password"
           />
 
           <PasswordInput
@@ -83,6 +87,7 @@ export const RegistrationForm = ({ invite }: RegistrationFormProps) => {
             id="password-confirm"
             autoComplete="new-password"
             {...form.getInputProps("confirmPassword")}
+            name="password-confirmation"
           />
           <Button type="submit" fullWidth loading={isPending}>
             {t("action.register.label")}
diff --git a/apps/nextjs/src/app/[locale]/auth/login/_login-form.tsx b/apps/nextjs/src/app/[locale]/auth/login/_login-form.tsx
index 47b8a7f7f..2458b2171 100644
--- a/apps/nextjs/src/app/[locale]/auth/login/_login-form.tsx
+++ b/apps/nextjs/src/app/[locale]/auth/login/_login-form.tsx
@@ -116,19 +116,26 @@ export const LoginForm = ({ providers, oidcClientName, isOidcAutoLoginEnabled, c
       <Stack gap="lg">
         {credentialInputsVisible && (
           <>
-            <form onSubmit={form.onSubmit((credentials) => void signInAsync(credentials.provider, credentials))}>
+            <form
+              autoComplete="on"
+              onSubmit={form.onSubmit((credentials) => void signInAsync(credentials.provider, credentials))}
+            >
               <Stack gap="lg">
                 <TextInput
                   label={t("field.username.label")}
                   id="username"
                   autoComplete="username"
+                  autoCapitalize="none"
+                  spellCheck={false}
                   {...form.getInputProps("name")}
+                  name="username"
                 />
                 <PasswordInput
                   label={t("field.password.label")}
                   id="password"
                   autoComplete="current-password"
                   {...form.getInputProps("password")}
+                  name="password"
                 />
 
                 {providers.includes("credentials") && (
@@ -152,7 +159,13 @@ export const LoginForm = ({ providers, oidcClientName, isOidcAutoLoginEnabled, c
         )}
 
         {providers.includes("oidc") && (
-          <Button fullWidth variant="light" disabled={!isMounted} onClick={async () => await signInAsync("oidc")}>
+          <Button
+            type="button"
+            fullWidth
+            variant="light"
+            disabled={!isMounted || isPending}
+            onClick={() => void signInAsync("oidc")}
+          >
             {t("action.login.labelWith", { provider: oidcClientName })}
           </Button>
         )}
@@ -174,7 +187,8 @@ const SubmitButton = ({ isMounted, isPending, form, provider, children }: PropsW
   return (
     <Button
       type="submit"
-      name={provider}
+      name="provider"
+      value={provider}
       fullWidth
       onClick={() => form.setFieldValue("provider", provider)}
       loading={isPending && isCurrentProviderActive}