popup window is cutting off large product picture

[subesdk]

when clicking at a product picture to open it up in a large version - the browser windows / popup frames are cutting of the lower part of the picture.

 

At first I thought it was a browser setting - but it does so at all computeres I have tried. I have only used Microsoft Explore though :-( so whether it does the same in other browsers i do not know atm.

 

you can c the problem at this website

 

www.meganart.dk

 

eg go directly to a product -> http://www.meganart.dk/mobelshop/product_i...products_id=794

 

Ty for any help or comment :thumbsup:

[♥Monika in Germany]

are you on IE 7 alerady? my IE 6 looked fine. I've heard we need a bit of extra padding for the new IE (same for FF) so open the file and add - play around with it, start with 90 instead of the 60 - where you see the width and height parameters ...

 

currently:

document.images[0].width +30, document.images[0].height+60-i

[subesdk]

when clicking at a product picture to open it up in a large version - the browser windows / popup frames are cutting of the lower part of the picture.

 

At first I thought it was a browser setting - but it does so at all computeres I have tried. I have only used Microsoft Explore though :-( so whether it does the same in other browsers i do not know atm.

 

you can c the problem at this website

 

www.meganart.dk

 

eg go directly to a product -> http://www.meganart.dk/mobelshop/product_i...products_id=794

 

Ty for any help or comment :thumbsup:

 

 

Hi Monika

 

you have helped me once before a few months back - ty girl - I love you

 

what file is it Im suppose to open and edit? :'(

[djmonkey1]

This is a problem with many different browsers. Here's something I found that seems to work pretty well:

Open catalog/popup_image.php (standard install 2.2MS2) or catalog/includes/javascript/popup_image.js (BTS)

 

 

###Search for:

 

<script language="javascript">var i=0;
function resize() {
 if (navigator.appName == 'Netscape') i=40;
 if (document.images[0]) window.resizeTo(document.images[0].width +30, document.images[0].height+60-i);
 self.focus();
}
//--></script>

 

 

 

###Replace it with:

 

<script language="javascript" type="text/javascript">var i=0;

function resize() {

 if ( navigator.vendor == 'KDE' || ( document.childNodes && ( !document.all || navigator.accentColorName ) && !navigator.taintEnabled ) ) { //Konqueror et al

if (document.images[0]) window.resizeTo(document.images[0].width +30, document.images[0].height+60-i);
//this actually doesn't work- Konqueror is a mess.  

}  else { 

	if (window.navigator.userAgent.indexOf('MSIE 6.0') != -1 && window.navigator.userAgent.indexOf('SV1') != -1) { 
				   i=23; //IE 6.x on Windows XP SP2
	} else if (window.navigator.userAgent.indexOf('MSIE 6.0') != -1) { 
				   i=50; //IE 6.x
	} else if (window.navigator.userAgent.indexOf('Firefox') != -1 && window.navigator.userAgent.indexOf("Windows") != -1) { 
				   i=38; //Firefox on Windows
	} else if (window.navigator.userAgent.indexOf('Mozilla') != -1 && window.navigator.userAgent.indexOf("Windows") != -1) { 
				   i=45; //Mozilla on Windows (this has been merged with Firefox, no?)
	} else if (window.opera && document.childNodes) {
				   i=50; //Opera 7+
	} else { 
				   i=70; //All other browsers including Mozilla on Linux
	}

if (document.images[0]) {
	imgHeight = document.images[0].height+110-i;
	imgWidth = document.images[0].width+30;
	var height = screen.height;
	var width = screen.width;
	var leftpos = width / 2 - imgWidth / 2;
	var toppos = height / 2 - imgHeight / 2;
	window.moveTo(leftpos, toppos);
	window.resizeTo(imgWidth, imgHeight);
   }

}

 self.focus();

}
//--></script>

[subesdk]

This is a problem with many different browsers. Here's something I found that seems to work pretty well:

 

 

wow djmonkey - you just hit the nail !! it works awesome

 

ty to you both Monika and djmonkey :thumbsup: