Skip to content

Fix 1Password autofill on site name input - #34

Merged
jasoncrawford merged 4 commits into
mainfrom
fix/site-name-1password
Feb 22, 2026
Merged

jasoncrawford merged 4 commits into
mainfrom
fix/site-name-1password

Conversation

@jasoncrawford

Copy link
Copy Markdown
Collaborator

Summary

  • Rename form input from name="name" to name="site_name" to prevent 1Password from detecting it as a username field
  • Update addSite and editSite server actions to read site_name from form data

Closes #33

Test plan

  • Open site add/edit dialog in Chrome with 1Password extension — autofill should no longer trigger
  • Verify adding and editing sites still works correctly

🤖 Generated with Claude Code

Closes #33

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented Feb 21, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
site-status Ready Ready Preview, Comment Feb 22, 2026 4:51am

Request Review

The name rename alone wasn't enough — 1Password uses multiple signals.
The data-1p-ignore attribute explicitly tells 1Password to skip the field.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The name -> site_name rename was unnecessary; data-1p-ignore and
autocomplete=off are the actual fix for 1Password autofill. Reverting
the rename fixes test failures.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jasoncrawford
jasoncrawford merged commit 24211f3 into main Feb 22, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — 8187ad38 Deployed Feb 22, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Site edit popover triggers 1Password Chrome plugin

1 participant