No message if user fails to check checkbox

Created on 7 March 2023, almost 2 years ago
Updated 24 May 2024, 7 months ago

If you fill out the registration form and forget to check the accept box for the terms, nothing happens, no message, the user thinks the form is broken. For any "required" field there should be a message reminding the user they must check it to continue.

πŸ› Bug report
Status

Fixed

Version

2.0

Component

User interface

Created by

πŸ‡¨πŸ‡¦Canada OMD

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @OMD
  • Status changed to Postponed: needs info almost 2 years ago
  • ivnish Kazakhstan

    Hi!

    I tested on fresh install (Drupal 9.5, terms_of_use 2.3)

    I have a messages about required fields

  • πŸ‡¨πŸ‡¦Canada OMD

    Try inserting a very lengthy terms & conditions text. I have quite a long text and the module wraps it in a closed expandable wrapper with the checkbox inside it. The checkbox for any additional questions added is outside this wrapper and clearly visible, where I think the "accept" checkbox should be as well, otherwise the "please fill out this field" text is hidden inside the closer wrapper.

  • Status changed to Active almost 2 years ago
  • Status changed to Postponed: needs info almost 2 years ago
  • πŸ‡¨πŸ‡¦Canada OMD

    I guess thinking more on this I could expand that wrapper with CSS overriding the default setup but many people many not think of that before they start using the module, until there are complaints about a registration for that seemingly won't submit for no reason, which is how I arrived at this issue.

  • πŸ‡¨πŸ‡¦Canada OMD

    It is nice for the long text to be in the wrapper though to keep the reg. form from being super long. It would be great if the checkbox could just be outside of that wrapper.

  • First commit to issue fork.
  • Merge request !5Issue #3346511 - Add wrapper β†’ (Merged) created by AstonVictor
  • Status changed to Fixed 7 months ago
  • πŸ‡ΊπŸ‡¦Ukraine AstonVictor

    Created and merged a new MR.

    I was able to reproduce the issue with a long text. Probably, you had some additional CSS styles to smth else.
    anyway, I added a wrapper to the text.

  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024