Popup Produktbild groß auch für Safari ermöglichen

[Subhash]

Damit das Resize des Popups des Produktbildes auch in dem Apple-Standard-Browser Safari funktioniert, der zur Zeit die 3. stärkste Verbreitung nach IE und Firefox hat und für manche Produkte oder Branchen recht wichtig ist, muss in dem File includes/modules/pages/popup_image/jscript_main.php die Zeilen 25 - 27

  } else {
      i=80; //This is all other browsers including Mozilla on Linux
  }

durch folgendes ersetzt werden:

  } else if (window.navigator.userAgent.indexOf('Safari') != -1) {
      i=5;
  } else {
      i=80; //This is all other browsers including Mozilla on Linux
  }

Übrigens funktioniert dieses PopUp im neuen Internet Explorer 7 gar nicht (es passiert beim Klick aufs Produktbild rein gar nichts außer dass in der Statuszeile "Fehler" angezeigt wird). So bald ich Zeit dafür finde, werde ich das kontrollieren und ggf. hier eine Lösung bekannt geben.

[marino01]

ok danke

[Subhash]

Hat jemand den IE7 installiert und die PopUp-Funktion für die Produktbilder überprüft?

Ich verwende nur die Standalone-Version um auch IE6 zum Testen verwenden zu können und bin mir daher nicht sicher, ob's nicht daran liegt, dass die PopUps bei mir weder in IE 6, noch in IE7 funktionieren. Zumindest mit einem deutschen Shop Zen Cart 1.3.5 (Datenbank Patch Level: 1.3.5) [2006-12-21 22:18:48] (Fresh Installation) und Image Handler.

[webchills]

Wenns drum geht, dass mit installiertem Image Handler im IE7 bei Mouseover kein Popupbild erscheint sondern unten links ein Javascriptfehler mit

Line: 125
Char: 7
Error: 'pageY' is null or not an object
Code: 0

Damits auch mit IE7 funkt muss man die jscript_imagehover.js im Ordner jscript im Templatesordner ändern. Ab Zeile 123:

if (docheight - event.clientY < zoomimg_h + (2 * padding)){
      ycoord += event.clientY - Math.max(0,(0 + zoomimg_h + (5 * padding) + e.pageY - docheight - truebody().scrollTop));
    } else {
      ycoord += truebody().scrollTop + event.clientY;
    }
  }

ersetzen mit:

if (docheight - event.clientY < zoomimg_h + (2 * padding)){
      ycoord += event.clientY - Math.max(0,(0 + zoomimg_h + (5 * padding) + event.clientY - docheight - truebody().scrollTop));
    } else {
      ycoord += truebody().scrollTop + event.clientY;
    }
  }