fix(auth): hide Turnstile widget container to prevent layout gap#3706
fix(auth): hide Turnstile widget container to prevent layout gap#3706waleedlatif1 merged 1 commit intostagingfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Applies the same layout fix to both Written by Cursor Bugbot for commit 9850cb3. Configure here. |
The invisible Turnstile iframe was taking up space between the password field and submit button. Wrapped in a hidden div.
5c526cc to
9850cb3
Compare
|
@cursor review |
|
@greptile |
Greptile SummaryThis PR fixes a layout gap on the login and signup pages caused by Cloudflare Turnstile's invisible CAPTCHA iframe being rendered in the document flow, by wrapping the Key changes:
Notes:
Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant Form as Login/Signup Form
participant TurnstileDiv as div.h-0.w-0.overflow-hidden
participant Turnstile as Turnstile Widget (invisible)
participant Server as Auth Server
User->>Form: Fill in credentials & submit
Form->>Turnstile: turnstileRef.execute()
Note over TurnstileDiv,Turnstile: Widget lives in zero-sized container,<br/>no layout impact but still in DOM
Turnstile-->>Form: onSuccess(token)
Form->>Server: POST /auth with token
Server-->>Form: Auth response
Form-->>User: Redirect / error message
Last reviewed commit: "fix(auth): hide Turn..." |
Summary
hiddendiv so it doesn't affect layout flowTest plan