-
Notifications
You must be signed in to change notification settings - Fork 6
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
Avoid error on "verify-email" page under browser auto-translation #42
Conversation
to avoid error on DOM manipulation otherwise we'll get error below: > NotFoundError: Failed to execute 'removeChild' on 'Node': The node > to be removed is not a child of this node.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a comment to describe the "notranslate" class that will prevent Chrome's translation feature. No class name provided tailwindcss
You can now add class="notranslate" to any HTML element to prevent that element from being translated. For example, you may want to do something like:
https://developers.google.com/search/blog/2008/10/helping-you-break-language-barrier
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
yeah you're right. fixed: https://github.com/route06inc/giselle/pull/42/commits/c8dd9b674a429e9385f0a83f881c3567f46d8c64txyo |
thx! |
Summary
Enable signing up in environment with browser auto-translation by preventing unintended change in element class of OTP field
Related Issue
Changes
Disallow translation on OTP "field" on /signup/verify-email
/signup/verify-email
"page"Testing
bun run dev
on localOther Information