IE Overlay Height

In IE, the height of the overlay is set to the view height. If you scroll the page then you can see that the overlay does not cover all of it.

Solution: use the clientHeight instead.

At line 792 of window.js, simply plug in document.body.clientHeight. This does the trick.
Comments and changes to this ticket

niko
niko August 19th, 2008 @ 01:20 AM
    This works for me, too.
satoshi
satoshi August 26th, 2008 @ 01:48 AM

It occurred in IE7.you can see this condition at attached img.
    control_modal.jpg 45.8 KB


    → State changed from “unconfirmed” to “resolved”

There is another aspect of this problem that I am working on, but this is definitely the fix for IE, and I've incorporated the change. Thanks!
gordo11231
gordo11231 November 16th, 2008 @ 06:51 AM

This fix doesn't well work when the document is very short. (In IE 7 at least.) This seems better:

        positionOverlay: function(){
            var dimensions = document.viewport.getDimensions();
            Control.Overlay.container.setStyle({
                width: Math.max(
                            document.body.clientWidth,
                            dimensions.width) + 'px',
                height: Math.max(document.body.clientHeight,
                            dimensions.height) + 'px'
            });
        }

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.