Closed (won't fix)
Project:
Terms of Use
Version:
6.x-1.9
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
17 Aug 2008 at 17:48 UTC
Updated:
7 Jul 2012 at 17:15 UTC
I think, it doesn't look nice when the terms page is long; so why not use a popup instead? There can be a checkbox and something like "I've read and agree to the Terms of Use" where "Terms of Use" is a link to the popup page.
Comments
Comment #1
venusrising commentedI agree, the node takes up the whole page, is it possible to make it collapse?
Comment #2
logicalpat commentedI would also like some way to link to the node or something for very long terms of use nodes.
Comment #3
manoloka commentedGreat module
I subscribe
Comment #4
feconroses commentedsuscribe!
Comment #5
Chill35 commentedI will document how to use a link instead of showing the Terms, or, alternatively how to makes the Terms collapsible or scrollable. Thanks.
Comment #6
Chill35 commentedOn the Admin page for the module, you may now use a token to place a link in the checkbox label. If you use that token, the Terms are not shown. The User must follow the link to read the Terms.
I have also improved the documentation.
Comment #7
manoloka commentedAnd what that token would be?
Comment #8
qmaria commentedBy using grep, I found this in the help text on the admin page.
I agree with the @link.
However, this is not a pop-up. It takes the user there, who then has to use the back button. If she or he closes the window (which is expected behavior), she will lose the url of your site if she does not remember it.
Comment #9
qmaria commentedCan you please make it a pop-up window? Thank you.
Comment #10
manoloka commentedI agree this solution isn't the most user friendly.
I think it'll be much better make it collapsible or even in a scrollable text area of about 10-20 rows.
What you think?
After all the Terms an Conditions is only a legal requirement, whether the user wants to actually read them or not is up to them (I've some times signed with websites, ticked the T&C but didn't bother to read them, haven't' you?)
Comment #11
Chill35 commentedOpening up the link in a pop-up can be done with JavaScript. Personally I hate pop-ups.
Making the Terms collapsible can be done with JavaScript. However, it does not make sense here to make the Terms collapsible, and I believe that most people would agree. To make content collapsible means: 'we are hiding something that you don't need to read'. I am aware that people check the 'I agree' box without reading, but we have to play along and pretend that they _will_ read. Otherwise, why would we add Terms of use with a required checkbox...?
Making the Terms scrollable can be done with CSS only (just apply a height to the Terms).
All of this can be achieved by theming the output of the module.
All these things are currently documented in README.txt.
Comment #12
manoloka commentedSorry but I don't see any of that documented in the README.txt file (I've just downloaded the latest one).
I must be looking in the wrong place ???
Comment #13
manoloka commentedDoesn't this means that the T&C page itself will also be scrollable?
Comment #14
Chill35 commentedSorry but I don't see any of that documented in the README.txt file (I've just downloaded the latest one).
I have just downloaded the latest version, and it's there.
Doesn't this means that the T&C page itself will also be scrollable?
No. In your CSS selector, you will have to target the div that contains the terms in your registration form.
Comment #16
manoloka commentedI've managed to do a scrollable Terms of Use div using
thanks
Comment #17
tsvenson commentedIt would be so much better if this was moved to the admin UI.
It works great on the live theme using the template.php, but if you have another theme for admin you have to do it there as well. In my case I will have to make a sub theme just to be able to hide this, which I think is a bit overkill.
An option to hide the node text for admins so only the checkbox is visible would be enough and then use the template.php to theme it for the live site.
Comment #18
ingopingo commentedwhy isn't there a clear & simple solution to this issue?
nothing of these work for me and i don't want to show my very terms of use in the registration form either.
for my users should be able to choose from several templates it makes no sense to edit them all...
so i dare to ask again:
how to just show a link to the tou (instead of having this neverending registration form) without hacking the templates?
thx in advance...
Comment #19
jennifer.chang commentedsubscribe
Comment #20
foodbo commentedsubscribing....
one suggestion, would that be easy to work with "popup api" module? (http://drupal.org/project/popups)
Comment #21
gagarine commentedpopus cut the flow.. the solution in #16 is the good solution. You only need to add this CSS code in your theme.
To work of out the box this code need to be include in Terms of Use.
Comment #22
HS commentedYup! Solution in #16 works really nicely.
Comment #23
giorgosk#16 works great
But links inside a short terms of use can lead to other more detailed terms of use ;-)
Comment #24
iva2k commentedsubscribe
Solution in #16 makes so much sense.
Comment #25
gagarine commented@GiorgosK #23 "But links inside a short terms of use can lead to other more detailed terms of use ;-)"
No... in the sign-up process, the only target is to keep user in the page: no external link.
Comment #26
iva2k commentedJust to let people know - I included fix proposed in #16 in a combined patch #759064: Combined patch to address multiple issues and feature requests
Comment #27
kars-t commentedCSS should go into an .css file. I make one later so it can be themed more easily.
Comment #28
kars-t commentedComment #29
joostvdl commentedThe solution mentioned in #11 and the README.txt is not working out of the box.
There are a couple of things that are important to use the theme_override. In the admin interface the "Label for the checkbox' has to be empty.
Second the theming is not working the same way. in the module terms_of_use.module at line 159-165 the code has to be changed to:
Removed:
This has to be removed because of it is not adding the result to the checkbox as a label (see code below this : // Adding the checkbox to the fieldset.)
And the code in the template.php has to be changed. It is better to have this to make the checkbox and label inline:
Comment #30
kars-t commented