Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add spellcheck="false" to form fields #6744

Merged
merged 1 commit into from
May 16, 2019
Merged

Conversation

pfhayes
Copy link
Contributor

@pfhayes pfhayes commented May 15, 2019

In Safari on OSX, it will replace form values with autocorrected values
unless this field is present

There were a couple fields that already had spellcheck="false", this adds the attribute to all other <input/> fields that currently have autocomplete="off"

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
In Safari on OSX, it will replace form values with autocorrected values
unless this field is present
@meirish
Copy link
Contributor

meirish commented May 16, 2019

Thanks! we need to do a sweep and add these attributes in more places, but this is a great start!

@meirish meirish merged commit cf81bf6 into hashicorp:master May 16, 2019
catsby added a commit that referenced this pull request May 16, 2019

Verified

This commit was signed with the committer’s verified signature.
b1ek blek!
* master: (85 commits)
  UI icon - add size (#6736)
  Add Priority Queue library to sdk (#6664)
  Add spellcheck="false" to form fields (#6744)
  Maximum typo in Vault UI (#6743)
  docs: Fix Markdown formatting error in AWS Auth (#6745)
  changelog++
  Update OIDC Provider Setup docs (#6739)
  Update to use newer sdk
  Copy LogInput from audit package, add OptMarshaler interface  (#6735)
  docs: fixed typo (#6732)
  Fix typo
  changelog++
  Use Go modules in CircleCI (#6729)
  Fix recovery key backup path documentation
  Vendoring updated grpc
  Add link to R client on libraries list (#6722)
  UI ember engines (#6718)
  changelog++
  Update grpc and protos (#6725)
  changelog++
  ...
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.

None yet

2 participants