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

venusrising’s picture

Title: Popup? » Popup or Collapsible Possible?

I agree, the node takes up the whole page, is it possible to make it collapse?

logicalpat’s picture

I would also like some way to link to the node or something for very long terms of use nodes.

manoloka’s picture

Great module

I subscribe

feconroses’s picture

suscribe!

Chill35’s picture

Component: User interface » Documentation
Assigned: Unassigned » Chill35

I will document how to use a link instead of showing the Terms, or, alternatively how to makes the Terms collapsible or scrollable. Thanks.

Chill35’s picture

Version: 6.x-1.8 » 6.x-1.9
Status: Active » Fixed

On 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.

manoloka’s picture

And what that token would be?

qmaria’s picture

By 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.

qmaria’s picture

Can you please make it a pop-up window? Thank you.

manoloka’s picture

I 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?)

Chill35’s picture

Opening 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.

manoloka’s picture

All these things are currently documented in README.txt

Sorry 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 ???

manoloka’s picture

To make the Terms scrollable, apply a fixed height to your Terms using CSS.

Doesn't this means that the T&C page itself will also be scrollable?

Chill35’s picture

Sorry 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.

Status: Fixed » Closed (fixed)

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

manoloka’s picture

I've managed to do a scrollable Terms of Use div using

#terms-of-use{
	height:250px;
	overflow:auto;
	}

thanks

tsvenson’s picture

Component: Documentation » User interface
Status: Closed (fixed) » Active

It 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.

ingopingo’s picture

why 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...

jennifer.chang’s picture

subscribe

foodbo’s picture

subscribing....

one suggestion, would that be easy to work with "popup api" module? (http://drupal.org/project/popups)

gagarine’s picture

Title: Popup or Collapsible Possible? » Page is to long: use scrool bar to display Terms of Use

popus 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.

HS’s picture

Yup! Solution in #16 works really nicely.

giorgosk’s picture

#16 works great

But links inside a short terms of use can lead to other more detailed terms of use ;-)

iva2k’s picture

subscribe

Solution in #16 makes so much sense.

gagarine’s picture

@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.

iva2k’s picture

Just to let people know - I included fix proposed in #16 in a combined patch #759064: Combined patch to address multiple issues and feature requests

kars-t’s picture

Status: Active » Needs work

CSS should go into an .css file. I make one later so it can be themed more easily.

kars-t’s picture

Assigned: Chill35 » kars-t
joostvdl’s picture

The 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:

  // Adding the Terms of Use to the fieldset. 
  if ($show_terms && $node) {
    $checkbox_label = theme('terms_of_use', $terms = NULL, $node = NULL);
  }

Removed:

    $terms = node_prepare($node);
    $form['terms_of_use']['terms_of_use_text'] = array(
      '#value' => theme('terms_of_use', $terms->body, $node),
    );

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:

function THEMENAME_terms_of_use($terms, $node) {
  $output  = '<span id="terms-of-use">';
  $output .= t('Terms of use of this page are available !here.', array('!here' => l('here', 'node/'."$node->nid")));
  $output .= '</span>';
  return $output;
}
kars-t’s picture

Status: Needs work » Closed (won't fix)