IE8 Position Overflow Issue

Could you please provide a patch for this?

I set the following in my css file in order to make it work:

.page-cart-checkout fieldset#uc_termsofservice_agreement_checkout-pane {
position: relative;
}

.page-cart-checkout fieldset#uc_termsofservice_agreement_checkout-pane .tos-text {
overflow: scroll;
height: 200px;
}

Thanks chris_car, I've included the changes in the last dev release

I'd argue that overflow: auto; is probably more appropriate, so that the TOS element only displays scrollbars when necessary.

You're right, I've changed this in the last dev version. However, you can override the form tpl so you can use the css you want to, the css provided by the module is just an example.

This fix doesn't work for IE7. I am running the latest dev and it still overlaps.

To make this work with IE7 you need to add "position: relative;" not only to the container but also to class "tos-text" itself.

The latest release in file "uc_termsofservice.css" already contains:

#uc_termsofservice_agreement_checkout-pane .tos-text {
  border: 1px solid #D2E6F3;
  overflow: scroll;
  height: 200px;
  padding: 5px;
  margin: 0 0 10px 0;
  display: block;
}

#uc-termsofservice-general-form #edit-submit {
  display: none;
}

#uc_termsofservice_agreement_checkout-pane {
  position: relative;
}

To make it work with IE7, you can either patch this file or add the following in your custom CSS file:

#uc_termsofservice_agreement_checkout-pane .tos-text {
  position: relative;
}

I hope this helps.

Hi, this helped a lot. It is not correct that this is in the latest release: 2011-Feb-25.
Of course because the releasedate is earlier than the remark :)

Thanks for sharing and this helped me a lot on internet explorer.

greetings,
Martijn

Now fixed in -dev.