Hjælp jeg kan ikke finde fejlen i koden!!!

popupWin = window.open(url, \'send_sms\', \'height=\' + h + \',width=\' + w + \',top=\' + topPos + \',left=\' + leftPos + \'\')
}

<a href=\"java script:PopWin(\'sms.html\',145,380);\">

Sammelign de to... Det ser ud som om, at du mangler et par parametre (top og left positions)

Prøv med:
<a href=\"java script:PopWin(\'sms.html\',145,380,100,100);\">

Se om det virker :o)

popupWin = window.open(url, \'send_sms\', \'height=\' + h + \',width=\' + w + \',top=\' + topPos + \',left=\' + leftPos\')

/per

hvad mener du Per?

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

var popupWin = null;  // erklæres udenfor funktionen så du kan bruge
                        // den sidenhen. (eg: popupWin.close())

function popWin(url,h,w) {
  var ww = 480; wh = 340;  // de to variable bruger du ikke ???
  var leftPos = 50;
  var topPos = 50;    // leftPos og topPos bør aldrig vær null.
  if (top.screen){
    if (screen.width){
      leftPos = Math.round((screen.width-h-10)/2);
      topPos = Math.round((screen.height-w-29)/2);
    } else {    leftPos = 50
      topPos = 50
    }
  };
  if (navigator.appName!=\"Netscape\") {
    var pos = \",left=\"+leftPos+\",top=\"+topPos
  } else {
    var pos = \",screenX=\"+leftPos+\",screenY=\"+topPos
  };
  popupWin = window.open(url, \'send_sms\', \'height=\'+h+\',width=\'+w+top );
  popupWin.focus();  // i tilfælde af vinduer blev åbnet tidligere og
                      // ligger bagved.
};

// --> </script>
osv
osv

mvh JakobA

Jeg kan godt skive det hele igen incl. den linie jeg har rettet for dig.

<head>
<script language=\"JavaScript\">
<!-- Gem koden for gamle browsere

function PopWin(url,h,w) { var ww = 480, wh = 340, leftPos = null, topPos = null;

if (top.screen){
  if (screen.width){
    leftPos = Math.round((screen.width-h-10)/2);
    topPos = Math.round((screen.height-w-29)/2);
  } else {    leftPos = 50
    topPos = 50
  }
}
popupWin = window.open(url,\'send_sms\',\'height=\' + h + \',width=\' + w + \',top=\' + topPos + \',left=\' + leftPos\')
}
// -->
</script>
</head>
<body>
<a href=\"java script:PopWin(\'sms.html\',145,380);\"><h1>Klik her for at sende en sms!</h1></a></body>


/per

tak