holli: Problem mit Bildershow im Pop-Up

Hallo,
kann mal jemand den folgenden code durchforsten. irgendwie find ich den fehler nicht. und zwar ist das ganze ein pop-up mit einer javascript bildershow. wenn man auf den pfeil nach rechts klickt sollte ein anderes bild erscheinen und beim pfeil auf links dementsprechend wieder zurück. aber es passiert nix.
hier der code:

<script language="JavaScript">
<!--

function SymError()
{
return true;
}

window.onerror = SymError;

var SymRealWinOpen = window.open;

function SymWinOpen(url, name, attributes)
{
return (new Object());
}

window.open = SymWinOpen;

//-->
</script>

<script language="JavaScript">
<!--

var i = 1
images = new Array
images[1] = "../images/logodesign/devida_thumb.jpg"
images[2] = "../images/thumbs/dokumenten_02.gif"
images[3] = "../images/thumbs/dokumenten_04.gif"
images[4] = "../images/thumbs/dokumenten_05.gif"
images[5] = "../images/thumbs/dokumenten_06.gif"
images[6] = "../images/thumbs/dokumenten_07.gif"

function gallery(){
if (i == 1){
document.gallery.previous.value=""
}
}
function previmg(){
if (i != 1) {
i --
document.img.src = images[i]
document.gallery.next.value = "Nächstes >"
}
if (i == 1) {
document.gallery.previous.value=""
}
}
function nextimg(){
if (i != 6) {
i ++
document.gallery.previous.value="< Vorheriges"
image = images[i]
document.img.src = image}
if (i ==67){
document.gallery.next.value="";
}
}
function pickrand(){
var imagenumber = 6 ;
var randomnumber = Math.random() ;
var rand1 = Math.round( (imagenumber-1) * randomnumber) + 1;
var randimage = images[rand1]
document.img.src = randimage
i = rand1
if (i == 1){
document.gallery.previous.value="";
}
else {
document.gallery.previous.value="< Vorheriges";
}
if (i == 6){
document.gallery.next.value="";
}
else {
document.gallery.next.value="Nächstes >";
}
}
function firstimg(){
i = 1
document.img.src = images[i];
document.gallery.previous.value=" ";
document.gallery.next.value="Nächstes >"
}
function lastimg(){
i = 7
document.img.src = images[i];
document.gallery.next.value=" "
document.gallery.previous.value="< Vorheriges"
}
function stoperror(){
return true
}
window.onerror=stoperror
// -->
</script>
</head>

<body bgcolor="#ffffff">
<table width="320" border="0" cellspacing="0" cellpadding="0" height="250">
<tr height="240">
<td align="center" height="240"><img src="../images/logodesign/devida_brief.jpg" alt="" height="230" width="180" border="0"></td>
</tr>
<tr>
<td align="center" valign="top">
<a href="javascript:onclick=previmg()" class="text"><img src="../images/logodesign/arrow_left.jpg" alt="" height="27" width="26" border="0"></a> 
<a href="javascript:onclick=nextimg()" class="text"><img src="../images/logodesign/arrow_right.jpg" alt="" height="27" width="26" border="0"></a>
</td>
</tr>
<tr>
<td> </td>
</tr>
<tr>
<td align="center"><a href="(EmptyReference!)" onClick="JavaScript:self.close()" class="text">Fenster schliessen</a></td>
</tr>
</table>
</body>

</html>

<script language="JavaScript">
<!--
var SymRealOnLoad;
var SymRealOnUnload;

function SymOnUnload()
{
window.open = SymWinOpen;
if(SymRealOnUnload != null)
SymRealOnUnload();
}

function SymOnLoad()
{
if(SymRealOnLoad != null)
SymRealOnLoad();
window.open = SymRealWinOpen;
SymRealOnUnload = window.onunload;
window.onunload = SymOnUnload;
}

SymRealOnLoad = window.onload;
window.onload = SymOnLoad;

//-->
</script>

gruss und danke
holli

  1. abend,

    also ich weiß zwar nicht wozu du den ganzen anderen kram da mit drin stehen
    hast, aber wahrscheinlich stand der sourcecode irgendwo zum klauen bereit ;)
    ich hab ihn mal auf das nötigste gekürzt.. so, dass dein bildpreview zumindest
    funktioniert... und das ganz ohne etliche, unnütze funktionen:

    <html><head>

    <script language="JavaScript">
    var i = 1;
    images = new Array;
    images[1] = "../images/logodesign/devida_thumb.jpg";
    images[2] = "../images/thumbs/dokumenten_02.gif";
    images[3] = "../images/thumbs/dokumenten_04.gif";
    images[4] = "../images/thumbs/dokumenten_05.gif";
    images[5] = "../images/thumbs/dokumenten_06.gif";
    images[6] = "../images/thumbs/dokumenten_07.gif";

    function nextimg()
    {
     if (i != 6)
     {
      i++;
      nextImage = images[i];
      alert( document.getElementById('prevImage').src );
      document.getElementById('prevImage').src = nextImage;
     } else alert( 'ende der bildschau erreicht' );
    }

    function previmg()
    {
     if (i != 1)
     {
      i--;
      prevImage = images[i];
      alert( document.getElementById('prevImage').src );
      document.getElementById('prevImage').src = prevImage;
     } else alert( 'anfang der bildschau erreicht' );
    }
    </script>

    </head>

    <body bgcolor="#ffffff">
    <table width="320" border="0" cellspacing="0" cellpadding="0" height="250">
    <tr height="240">
    <td align="center" height="240">
     <img id=prevImage src="../images/logodesign/devida_brief.jpg" alt="" height="230" width="180" border="0"></td>
    </tr>
    <tr>
    <td align="center" valign="top">
     <img onclick="previmg()" src="../images/logodesign/arrow_left.jpg" alt="" height="27" width="26" border="0"></a> 
     <img onclick="nextimg()"src="../images/logodesign/arrow_right.jpg" alt="" height="27" width="26" border="0"></a>
    </td>
    </tr>
    <tr>
    <td> </td>
    </tr>
    <tr>
    <td align="center"><a href="(EmptyReference!)" onClick="JavaScript:self.close()" class="text">Fenster schliessen</a></td>
    </tr>
    </table>
    </body>
    </html>

    HIH

    mfg,
    (tanz das)
    Z.N.S.

    --
    <img src="http://www.dmp-web.de/comunicout/neubauten.gif" border="0" alt="">
    1. <td align="center" height="240">
       <img id=prevImage src="../images/logodesign/devida_brief.jpg" alt="" height="230" width="180" border="0"></td>
      </tr>
      <tr>
      <td align="center" valign="top">
       <img onclick="previmg()" src="../images/logodesign/arrow_left.jpg" alt="" height="27" width="26" border="0"></a> 
       <img onclick="nextimg()"src="../images/logodesign/arrow_right.jpg" alt="" height="27" width="26" border="0"></a>

      Hallo,
      danke erst mal. aber ich glaube an dem code stimmt auch was nicht. da fehlt doch was. du schreibst das onclick ... ins image tag aber es fehlt ja noch das <a href ... könntest du den code eventuell vervollständigen.

      gruss
      holli