- Issue created by @OMD
- Status changed to Postponed: needs info
almost 2 years ago 5:55am 10 March 2023 - 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 10:35am 10 March 2023 - Status changed to Postponed: needs info
almost 2 years ago 10:38am 10 March 2023 - π¨π¦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.
-
AstonVictor β
committed 5214b0e9 on 8.x-2.x
Issue #3346511 - Add wrapper
-
AstonVictor β
committed 5214b0e9 on 8.x-2.x
- Status changed to Fixed
7 months ago 7:26am 10 May 2024 - πΊπ¦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.