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.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

OMD created an issue. See original summary.

ivnish’s picture

Status: Active » Postponed (maintainer needs more info)
StatusFileSize
new217.95 KB
new194.92 KB

Hi!

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

screenshot

I have a messages about required fields

screenshot

omd’s picture

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.

ivnish’s picture

Status: Postponed (maintainer needs more info) » Active

Ok, I'll check it

omd’s picture

Status: Active » Postponed (maintainer needs more info)

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.

omd’s picture

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.

AstonVictor made their first commit to this issue’s fork.

astonvictor’s picture

Version: 8.x-2.3 » 8.x-2.x-dev
Status: Postponed (maintainer needs more info) » Fixed

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.

Status: Fixed » Closed (fixed)

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