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

Use different style for field #25

Merged
merged 1 commit into from Jun 9, 2020
Merged

Conversation

nkay08
Copy link
Contributor

@nkay08 nkay08 commented Feb 27, 2020

Use different style than display:none for html field, since it can be detected by some bots.

Credits to https://dev.to/felipperegazio/how-to-create-a-simple-honeypot-to-protect-your-web-forms-from-spammers--25n8

Use different style than `display:none` for html field, since it can be detected by some bots.
@nkay08
Copy link
Contributor Author

nkay08 commented Feb 27, 2020

Resolves #25

@9mido
Copy link

9mido commented Jun 8, 2020

Good idea but for security minded people I would recommend using external style sheets to follow content security policy instead of inline CSS. But people should be able to override the templates to do that so it should be fine. Thank you for the inline CSS it is really helpful!

@jamesturk
Copy link
Owner

Thanks for this, while most people should probably override the template as they desire- this is an improved default that I think will serve most people better.

@jamesturk jamesturk merged commit ced620d into jamesturk:master Jun 9, 2020
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

3 participants