Skip to content

Disable spellcheck on inputs#2551

Merged
david-crespo merged 1 commit intomainfrom
turn-off-spellcheck-in-inputs
Nov 13, 2024
Merged

Disable spellcheck on inputs#2551
david-crespo merged 1 commit intomainfrom
turn-off-spellcheck-in-inputs

Conversation

@charliepark
Copy link
Contributor

In Safari, input fields currently suggest spellchecked variations of what you've entered:
Screenshot 2024-11-13 at 12 01 55 PM

This PR turns that off:
Screenshot 2024-11-13 at 12 02 27 PM

Note, though, that there are some browser-level preferences that we do not have control over, like this one:
Screenshot 2024-11-13 at 11 58 06 AM
which decides to suggest names from your contacts list to the "Name" field:
Screenshot 2024-11-13 at 11 57 53 AM

I looked into various ways of turning that off, but everything I found was hacky (like adding hidden pseudo form fields to the page to intercept autocompletes, as described in this SO thread).

Nevertheless, field-level spellchecking prefs do seem to be respected by the browser, so we'll apply that here.

Closes #2532

@vercel
Copy link

vercel bot commented Nov 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
console ✅ Ready (Inspect) Visit Preview Nov 13, 2024 8:12pm

@david-crespo david-crespo enabled auto-merge (squash) November 13, 2024 21:52
@david-crespo david-crespo merged commit 7303d9d into main Nov 13, 2024
@david-crespo david-crespo deleted the turn-off-spellcheck-in-inputs branch November 13, 2024 21:58
david-crespo added a commit to oxidecomputer/omicron that referenced this pull request Nov 14, 2024
david-crespo added a commit to oxidecomputer/omicron that referenced this pull request Nov 14, 2024
oxidecomputer/console@9ef82ba...48fc079

* [48fc079a](oxidecomputer/console@48fc079a)
oxidecomputer/console#2542
* [f3d38103](oxidecomputer/console@f3d38103)
oxidecomputer/console#2545
* [eba6626d](oxidecomputer/console@eba6626d)
bump API for moved system timeseries endpoints
* [6052fdbf](oxidecomputer/console@6052fdbf)
oxidecomputer/console#2549
* [b886e4b8](oxidecomputer/console@b886e4b8)
oxidecomputer/console#2550
* [7303d9d7](oxidecomputer/console@7303d9d7)
oxidecomputer/console#2551
* [e239a8e2](oxidecomputer/console@e239a8e2)
oxidecomputer/console#2540
* [31520a2e](oxidecomputer/console@31520a2e)
move Breadcrumbs component definition into TopBar.tsx
* [ad02edec](oxidecomputer/console@ad02edec)
minor: extract UserMenu component for readability

Co-authored-by: iliana etaoin <iliana@oxide.computer>
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.

Disable spellcheck on inputs in Safari

2 participants