-
Notifications
You must be signed in to change notification settings - Fork 68
Email domain typo check #799
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
Merged
dannyvankooten
merged 6 commits into
ibericode:main
from
arnelap:Email-domain-typo-check
Dec 10, 2025
Merged
Email domain typo check #799
dannyvankooten
merged 6 commits into
ibericode:main
from
arnelap:Email-domain-typo-check
Dec 10, 2025
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…to be above the email field instead of below
forgot to undo this local testing change
dannyvankooten
requested changes
Nov 27, 2025
Member
|
Nice! I've added some thoughts, but looks 90% good to me. 👍 Most important for me is if we can load the JS only when a form has email typo check enabled. |
…th typo checker before loading the scripts
when i put it in front of the email field it can in some layouts push the field to the next line where it used to be on the same line with the field label, changing it from 1 line to 3 lines... when i put it behind the field the browser build in suggestions overlap it... I guess for now giving it a class so people can change the position via css if they want, what do you think?
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Was looking at old git issues / feature requests.
#160 Implement mailcheck.js
Seems Mailcheck has been abandoned so i didn't include it.
Uses a Levenshtein distance-based algorithm to suggest the nearest domain.
Suggestion is shown above the email field.
Can be turned on under MC4WP > Form > Settings (Off by default).
Checks these domains, can be overwritten with filter mc4wp_email_typo_checker_domains
[
'gmail.com',
'yahoo.com',
'hotmail.com',
'outlook.com',
'icloud.com',
'aol.com',
'live.com',
'msn.com',
'me.com',
'mac.com',
'googlemail.com',
'ymail.com',
'protonmail.com',
'mail.com',
'gmx.com',
'zoho.com'
]