[JS] Pop-up werkt niet (meer) - Softwareontwikkeling

zaterdag 14 november 2009 12:35

Acties:

0 Henk 'm!

Topicstarter

Beste

Ik had mijn site netjes in orde gemaakt en moest enkel nog een foto album invoegen, maar ik had net m'n windows7 CD gekregen en besloot dus alles op een externe harde schijf te zetten en clean install te doen. (wat overigens niet geweldig ging, Acer applicaties die niet installen e.d...) Maar ik heb nu alles dterug op zijn plaats gezet, en eerst kwam ik al tot de ontdekking dat er al watmis was, er ontbraken delen van m'n css file.. Nu heb ik alles ongeveer in orde, maar m'n javascript voor de pop-up werkt niet meer, terwijl het eerst wel werkte, en ik heb geen idee hoe het op te lossen.. Ik weet dat het veel code is, maar ik hoop dat er sommigen eens naar willen kijken!

Head code:

code:

<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<!-- InstanceBeginEditable name="doctitle" -->
<title>Start</title>
<!-- InstanceEndEditable -->

<link href="css/stylesheet.css" rel="stylesheet" type="text/css"/>
<script src="SpryAssets/SpryMenuBar.js" type="text/javascript"></script>
<link href="SpryAssets/SpryMenuBarHorizontal.css" rel="stylesheet" type="text/css"/>
<script type="text/javascript" src="onlyPopupOnce.js"></script>
<!-- InstanceBeginEditable name="head" --><script type="text/javascript" src="onlyPopupOnce.js"></script>
<!-- InstanceEndEditable -->
<style type="text/css">
<!--
a:link {
    color: #999;
}
a:visited {
    color: #666;
}
a:hover {
    color: #FFE500;
}
-->
</style>

<script type="text/javascript">
<!--
function MM_swapImgRestore() { //v3.0
  var i,x,a=document.MM_sr; for(i=0;a&&i<a.length&&(x=a[i])&&x.oSrc;i++) x.src=x.oSrc;
}
function MM_preloadImages() { //v3.0
  var d=document; if(d.images){ if(!d.MM_p) d.MM_p=new Array();
    var i,j=d.MM_p.length,a=MM_preloadImages.arguments; for(i=0; i<a.length; i++)
    if (a[i].indexOf("#")!=0){ d.MM_p[j]=new Image; d.MM_p[j++].src=a[i];}}
}

function MM_findObj(n, d) { //v4.01
  var p,i,x;  if(!d) d=document; if((p=n.indexOf("?"))>0&&parent.frames.length) {
    d=parent.frames[n.substring(p+1)].document; n=n.substring(0,p);}
  if(!(x=d[n])&&d.all) x=d.all[n]; for (i=0;!x&&i<d.forms.length;i++) x=d.forms[i][n];
  for(i=0;!x&&d.layers&&i<d.layers.length;i++) x=MM_findObj(n,d.layers[i].document);
  if(!x && d.getElementById) x=d.getElementById(n); return x;
}

function MM_swapImage() { //v3.0
  var i,j=0,x,a=MM_swapImage.arguments; document.MM_sr=new Array; for(i=0;i<(a.length-2);i+=3)
   if ((x=MM_findObj(a[i]))!=null){document.MM_sr[j++]=x; if(!x.oSrc) x.oSrc=x.src; x.src=a[i+2];}
}
//-->
</script>

</head>

JS

code:

// JavaScript Document

var expDays = 365; // number of days the cookie should last

var page = "popupinfo.html";
var windowprops = "width=560,height=320,location=no,toolbar=no,menubar=no,scrollbars=no,resizable=yes";

function GetCookie (name) {
  var arg = name + "=";
  var alen = arg.length;
  var clen = document.cookie.length;
  var i = 0;
  while (i < clen) {
    var j = i + alen;
    if (document.cookie.substring(i, j) == arg)
    return getCookieVal (j);
    i = document.cookie.indexOf(" ", i) + 1;
    if (i == 0) break;
  }
  return null;
}

function SetCookie (name, value) {
  var argv = SetCookie.arguments;
  var argc = SetCookie.arguments.length;
  var expires = (argc > 2) ? argv[2] : null;
  var path = (argc > 3) ? argv[3] : null;
  var domain = (argc > 4) ? argv[4] : null;
  var secure = (argc > 5) ? argv[5] : false;
  document.cookie = name + "=" + escape (value) +
    ((expires == null) ? "" : ("; expires=" + expires.toGMTString())) +
    ((path == null) ? "" : ("; path=" + path)) +
    ((domain == null) ? "" : ("; domain=" + domain)) +
    ((secure == true) ? "; secure" : "");
}

function DeleteCookie (name) {
  var exp = new Date();
  exp.setTime (exp.getTime() - 1);
  var cval = GetCookie (name);
  document.cookie = name + "=" + cval + "; expires=" + exp.toGMTString();
}

var exp = new Date();
exp.setTime(exp.getTime() + (expDays*24*60*60*1000));

function amt(){
  var count = GetCookie('count')
  if(count == null) {
    SetCookie('count','1')
    return 1
  } else {
    var newcount = parseInt(count) + 1;
    DeleteCookie('count')
    SetCookie('count',newcount,exp)
    return count
  }
}

function getCookieVal(offset) {
  var endstr = document.cookie.indexOf (";", offset);
  if (endstr == -1)
  endstr = document.cookie.length;
  return unescape(document.cookie.substring(offset, endstr));
}

function checkCount() {
  var count = GetCookie('count');
  if (count == null) {
    count=1;
    SetCookie('count', count, exp);
    window.open(page, "", windowprops);
  } else {
    count++;
    SetCookie('count', count, exp);
  }
}

window.onload=checkCount;

popupinfo.html code

code:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Welkom!</title>
<style type="text/css">
<!--
#tekst {
    font-family: "Arial Black", Gadget, sans-serif;
    color: #4F8FBB;
    height: 240px;
    width: 480px;
    vertical-align: middle;
    display: block;
    padding: 0.75em;
    border: 5px solid #4E8EBA;
    background-color: #FFF;
}

a:link {
    color: #999;
}
a:visited {
    color: #666;
}
a:hover {
    color: #FFE500;
}
-->
</style>
</head>

<body>
<div id="tekst">
  <p>Welkom op denotenbalkers.be!    </p>
  <p>Wij raden u ten sterkste volgende browsers aan om onze website op z'n best te kunnen bekijken: <a href="http://www.microsoft.com/netherlands/windows/internet-explorer/" target="_blank">Internet Explorer 8</a> of <a href="http://www.mozilla-europe.org/nl/firefox/" target="_blank">FireFox 3.5</a>.</p>
  <p>Ook is een schermresolutie van 1024 bij 800 of meer pixels aangeraden!</p>
  <p> Nog veel kijkplezier op onze website!</p>
</div>
</body>
</html>

zaterdag 14 november 2009 12:50

Acties:

0 Henk 'm!

IntToStr

Probeer eerst eens te debuggen met bijv. firebug.

Het lijkt erop dat er geen popup komt als de waarde uit GetCookiee null is. Is dat het misschien?
Werkt je window.open functie wel als je hem los aanroept?

zaterdag 14 november 2009 13:38

Acties:

0 Henk 'm!

BramVroy

Topicstarter

Ik krijg het niet aan de praat. Heb het ook al elke file opnieuw gemaakt (de .js en de popup.html) en het werkt nog niet.
Debuggen weet ik niet hoe het moet, ik weet enkel dat je met firebug code kan kijken.

En dat denk ik niet, want de code hééft wel gewerkt..

zaterdag 14 november 2009 14:05

Acties:

0 Henk 'm!

Cartman!

Debuggen is heel simpel, per regeltje even kijken wat de uitkomsten zijn van variabelen, als je al firebug hebt kun je heel makkelijk de console.log() functie gebruiken zodat je niet met alerts zit te rotzooien. Nu gooi je gewoon even je code neer en verwacht je dat wij het wel oplossen, dat is niet de bedoeling.

zondag 15 november 2009 23:38

Acties:

0 Henk 'm!

BramVroy

Topicstarter

OK, ik heb het gefixed.. Het lag blijkbaar aan de roll-over banner. Niet aan dat op zich, maar dat de afbeelding ge-pre-load werd. Ik heb dat pre-loaden dan uit de body-tag gezwierd, en nu werkt het naar behoren. Weet er iemand hoe dit kan? En is het erg dat er niet ge-pre-load wordt?

zondag 15 november 2009 23:49

Acties: