Skip to content

fix(desktop): paint AuthGate above LoginBackdrop - #629

Merged
github-actions[bot] merged 3 commits into
developfrom
cursor/fix-authgate-paint-1a1f
Aug 26, 2026
Merged

fix(desktop): paint AuthGate above LoginBackdrop#629
github-actions[bot] merged 3 commits into
developfrom
cursor/fix-authgate-paint-1a1f

Conversation

@tomymaritano

@tomymaritano tomymaritano commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

Linux AppImage QA on v0.21.0 saw a solid #0a0b0d main window: AuthGate’s magic-link form was covered by an opaque LoginBackdrop canvas, and Settings → Account Sign In did not surface a readable email field.

  • Isolate AuthGate stacking (same pattern as GraphView) so the form sits above the tunnel canvas (z-index 1 vs 0).
  • Stop filling the canvas with opaque --bg-base; stroke the corridor only so a compositor miss cannot hide the form.
  • Use --glass-bg-fallback on the card and drop full-window -webkit-app-region: drag.
  • Portal Settings Account magic-link UI to document.body, drop overlay backdrop-filter, and paint the dialog on --glass-bg-fallback.

No guest path. No continue-locally skip. OFFICIAL_THEMES stays []. No package.json bump, tag, Capture, or GitHub OAuth changes.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Related Issues

QA fail on shipping Dripnex v0.21.0 AppImage (AuthGate does not paint).

Checklist

  • I've read CONTRIBUTING.md
  • Tests pass locally (pnpm test)
  • Build succeeds (pnpm --filter @dripnex/desktop build)
  • PR targets develop branch (not main)

Screenshots

Unauthenticated main window (email field + magic-link action on LoginBackdrop):

AuthGate email form above LoginBackdrop tunnel

Settings → Account → Sign In (email + Send Magic Link):

Settings Account Sign In magic-link dialog

To show artifacts inline, enable in settings.

Open in Web Open in Cursor 

Keep the magic-link form above the tunnel canvas on Linux AppImage,
and portal Settings Account Sign In so the email flow is not clipped.

Co-authored-by:  Tomás Maritano <tomymaritano@gmail.com>
@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 6 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 6218b4ba-6783-4719-874d-48559c62af24

📥 Commits

Reviewing files that changed from the base of the PR and between bb9a9ce and 4da9625.

📒 Files selected for processing (13)
  • apps/desktop/e2e/authgate.spec.ts
  • apps/desktop/src/renderer/components/auth/AuthGate.module.css
  • apps/desktop/src/renderer/components/auth/AuthGate.tsx
  • apps/desktop/src/renderer/components/auth/LoginBackdrop.module.css
  • apps/desktop/src/renderer/components/auth/LoginBackdrop.tsx
  • apps/desktop/src/renderer/components/auth/MagicLinkFlow.module.css
  • apps/desktop/src/renderer/components/auth/MagicLinkFlow.tsx
  • apps/desktop/src/renderer/components/auth/__tests__/authGate.test.ts
  • apps/desktop/src/renderer/components/auth/__tests__/authGateStacking.test.ts
  • apps/desktop/src/renderer/components/auth/__tests__/loginBackdrop.test.ts
  • apps/desktop/src/renderer/components/auth/authGateStacking.ts
  • apps/desktop/src/renderer/components/auth/paintTunnel.ts
  • apps/desktop/src/renderer/pages/settings/sections/__tests__/accountSignIn.test.ts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

cursoragent and others added 2 commits August 26, 2026 19:58
Portal already escaped Settings overflow; drop overlay backdrop-filter
and paint the dialog on glass-bg-fallback so Linux Electron cannot
composite the email field away.

Co-authored-by:  Tomás Maritano <tomymaritano@gmail.com>
Open Settings via globalThis so the evaluate callback typechecks
without a DOM Window lib in the e2e tsconfig.

Co-authored-by:  Tomás Maritano <tomymaritano@gmail.com>
@tomymaritano
tomymaritano marked this pull request as ready for review August 26, 2026 20:00
@github-actions
github-actions Bot enabled auto-merge (squash) August 26, 2026 20:00
@github-actions
github-actions Bot merged commit b2ef2af into develop Aug 26, 2026
22 of 23 checks passed
tomymaritano added a commit that referenced this pull request Aug 26, 2026
AuthGate paint fix so the magic-link form sits above LoginBackdrop (#629).
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 0.21.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

github-actions Bot pushed a commit that referenced this pull request Aug 26, 2026
<!-- CURSOR_AGENT_PR_BODY_BEGIN -->
## Summary

Replace AuthGate’s paper-and-ink tunnel with a vendored [React Bits
Letter Glitch](https://www.reactbits.dev/backgrounds/letter-glitch)
canvas. Dripnex is the hackable AI note taker; the first window now
glitches a matrix-style letter grid behind a readable magic-link form.

No `react-bits` / jsrepo / shadcn dependency. The effect lives in
`letterGlitch.ts`, adapted from
[DavidHDev/react-bits](https://github.com/DavidHDev/react-bits).

Keeps the #629 stacking fix so Linux AppImage cannot cover the form:

- `AUTH_GATE_FORM_Z_INDEX` (1) > `LOGIN_BACKDROP_Z_INDEX` (0)
- canvas `clearRect` only — no opaque `#000` / `--bg-base` `fillRect`
- `.screen` already paints `--bg-base`; outer vignette uses a
transparent mix, not an opaque fill
- `prefers-reduced-motion`: one painted frame, no rAF loop
- `requestAnimationFrame` is invoked as a window method (unbound extract
threw `Illegal invocation`)

No guest path. No continue-locally skip. `OFFICIAL_THEMES` stays `[]`.
No package.json bump, tag, Capture, or GitHub OAuth changes.

## Type of Change

- [ ] Bug fix
- [x] New feature
- [ ] Breaking change
- [ ] Documentation update

## Related Issues

Follows #629 (AuthGate paint / stacking). Tomás asked for Letter Glitch
as the AuthGate background.

## Checklist

- [x] I've read [CONTRIBUTING.md](../CONTRIBUTING.md)
- [x] Tests pass locally (`pnpm test`)
- [x] Build succeeds (`pnpm --filter @dripnex/desktop build`)
- [x] PR targets `develop` branch (not `main`)

## Screenshots

Unauthenticated main window: Letter Glitch behind a readable AuthGate
(email + Email me a link).

[AuthGate letter-glitch backdrop with magic-link
form](https://cursor.com/agents/bc-d9570e0c-a76a-45c7-b89f-a398562d5dab/artifacts?path=%2Fopt%2Fcursor%2Fartifacts%2Fauthgate_letter_glitch.png)


[authgate_letter_glitch_walkthrough.mp4](https://cursor.com/agents/bc-d9570e0c-a76a-45c7-b89f-a398562d5dab/artifacts?path=%2Fopt%2Fcursor%2Fartifacts%2Fauthgate_letter_glitch_walkthrough.mp4)


<sub>To show artifacts inline, <a
href="https://cursor.com/dashboard/cloud-agents#my-pull-requests">enable</a>
in settings.</sub>
<!-- CURSOR_AGENT_PR_BODY_END -->

<div><a
href="https://cursor.com/agents/bc-d9570e0c-a76a-45c7-b89f-a398562d5dab?cursor_ref=pr_footer&cursor_cta=open_in_web"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://cursor.com/assets/images/open-in-web-dark.png"><source
media="(prefers-color-scheme: light)"
srcset="https://cursor.com/assets/images/open-in-web-light.png"><img
alt="Open in Web" width="114" height="28"
src="https://cursor.com/assets/images/open-in-web-dark.png"></picture></a>&nbsp;<a
href="https://cursor.com/background-agent?bcId=bc-d9570e0c-a76a-45c7-b89f-a398562d5dab&cursor_ref=pr_footer&cursor_cta=open_in_cursor"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://cursor.com/assets/images/open-in-cursor-dark.png"><source
media="(prefers-color-scheme: light)"
srcset="https://cursor.com/assets/images/open-in-cursor-light.png"><img
alt="Open in Cursor" width="131" height="28"
src="https://cursor.com/assets/images/open-in-cursor-dark.png"></picture></a>&nbsp;</div>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants