Pressing enter on a form step should not click the back button

Working on it!!

vinodhini.e → made their first commit to this issue’s fork.

The problem was occurring because the back button was defined before submit button, so correcting the order fixed the issue for me,
Please review!!

Thanks for the work on this! Tests are failing, but I can also see the problem with back button is on all steps, not just the one. Thanks!

Hello @scott_euser!!,
Thanks for the feedback!
You are correct, the issue was present on multiple steps. I have reviewed the entire form logic in EmailOTPCheck.php and confirmed that my MR addresses the button order in every relevant step.
Like:
-->The fix has been applied to Step 2 (fapiOtpVerificationPageTwo).
-->The fix has been applied to Step 3 (fapiSetPasswordPageThree).
-->Step 1 and Step 4 of the form do not contain a "Back" button, so no changes were needed there.

Also i have made the pipeline green as there were some PHPCS related issues, also if you need changes in any particular file or step that i missed bymistake, then please guide me through it!!
Thanks you!!

@divyansh.gupta I have checked this MR !32 mergeable on drupal version -10.3.1-dev, the patch was applied successfully and is working fine.

Testing Steps-

• add module- Reset Password Email/SMS
• configure this module from Block layout
• Go to the page where block is configured and check- Enter key default focus was on back CTA
• Download and apply patch- MR !32 mergeable
• Go back to the configured page block and reverify.

After applying the patch, the default focus of the Enter key is on the next key.

@divyansh.gupta, Tested MR !32 on Drupal 11.1.7. The patch applied cleanly and and is working fine.

Steps to Test:

1. Install and enable the Reset Password Email/SMS module.
2. Add the corresponding block via Block layout.
3. After applying the patch, the Enter key correctly triggers the Next button by default.

Thanks.