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

CAPTCHA displays above custom fields #2118

Closed
CMLabay opened this issue May 25, 2022 · 1 comment · Fixed by #2135
Closed

CAPTCHA displays above custom fields #2118

CMLabay opened this issue May 25, 2022 · 1 comment · Fixed by #2135
Labels
bug report This issue reports a suspect bug or issue with the SDK itself

Comments

@CMLabay
Copy link

CMLabay commented May 25, 2022

Describe the problem

When configuring the lock object for the New Universal login, if both the simple CAPTCHA and additionalSignupFields are enabled, the CAPTCHA will display above the additionalSignupFields (but below the email and password fields). This is bad looking UI. If there are additional signup fields defined in the additionalSignupFields array, the CAPTCHA should be placed underneath these fields.

What was the expected behavior?

The expected behavior is that the CAPTCHA would be displayed underneath ALL fields on the the signup screen.

Reproduction

  • Step 1..
  • Step 2..
  • ...
    Enable simple CAPTCHA by navigating to Security > Attack Protection > Bot Detection.
    In the Response section, select Enforce CAPTCHA - Always, and choose Simple CAPTCHA as the provider.
    Save.

Navigate to Branding > Universal Login > Settings
Select New as the Universal Login Experience
In the Login tab next to Settings:
Update the html file by adding the additionalSignUpFields config item, and add at least one field to the array.
image

Save Changes.
View the Signup screen:
image

Environment

  • Version of this library used:
  • Which framework are you using, if applicable:
  • Other modules/plugins/libraries that might be involved:
  • Any other relevant information you think would be useful:
@CMLabay CMLabay added the bug report This issue reports a suspect bug or issue with the SDK itself label May 25, 2022
@stevehobbsdev
Copy link
Contributor

Thanks for raising @CMLabay. I do agree the UI looks a little off in this case. Let me raise a PR to fix it 👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report This issue reports a suspect bug or issue with the SDK itself
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants