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.